rust_binder: only print failure if error has source

The commit that fixes BINDER_GET_EXTENDED_ERROR changed the condition
for printing transaction failures so errors are printed even if the
cause is a dead or frozen process. Undo this change so that the error
is only printed if the failure has an errno associated with it.

Cc: stable@kernel.org
Fixes: 77bfebf110 ("rust_binder: fix BINDER_GET_EXTENDED_ERROR")
Signed-off-by: Alice Ryhl <aliceryhl@google.com>
Link: https://patch.msgid.link/20260708-get-extended-error-fix-printing-v1-1-6e293b213b70@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Alice Ryhl 2026-07-08 12:00:26 +00:00 committed by Greg Kroah-Hartman
parent 761b785a0c
commit bb66b1a345

View File

@ -1273,7 +1273,6 @@ fn transaction(self: &Arc<Self>, cmd: u32, reader: &mut UserSliceReader) -> Resu
inner.extended_error = inner.extended_error =
ExtendedError::new(info.debug_id as u32, err.reply, source.to_errno()); ExtendedError::new(info.debug_id as u32, err.reply, source.to_errno());
} }
}
pr_warn!( pr_warn!(
"{}:{} transaction to {} failed: {err:?}", "{}:{} transaction to {} failed: {err:?}",
@ -1283,6 +1282,7 @@ fn transaction(self: &Arc<Self>, cmd: u32, reader: &mut UserSliceReader) -> Resu
); );
} }
} }
}
Ok(()) Ok(())
} }