commit 2a71866544bb59103d612bec9597775d8b749219 parent e8ae8de3704554192a72a39a0789fc38f333d03c Author: Gabriele Svelto <gsvelto@mozilla.com> Date: Tue, 14 Oct 2025 13:02:49 +0000 Bug 1982016 - Appease clippy r=afranchuk Differential Revision: https://phabricator.services.mozilla.com/D267408 Diffstat:
| M | toolkit/crashreporter/crash_helper_common/src/breakpad/macos.rs | | | 5 | +++++ |
| M | toolkit/crashreporter/process_reader/src/platform/linux.rs | | | 2 | +- |
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/toolkit/crashreporter/crash_helper_common/src/breakpad/macos.rs b/toolkit/crashreporter/crash_helper_common/src/breakpad/macos.rs @@ -23,6 +23,11 @@ pub struct BreakpadData { } impl BreakpadData { + /// Creates a new BreakpadData object from a C string + /// + /// # Safety + /// + /// The `raw` argument must point to a valid NUL-terminated C string pub unsafe fn new(raw: BreakpadRawData) -> BreakpadData { BreakpadData { data: <OsString as BreakpadString>::from_ptr(raw), diff --git a/toolkit/crashreporter/process_reader/src/platform/linux.rs b/toolkit/crashreporter/process_reader/src/platform/linux.rs @@ -66,7 +66,7 @@ impl ProcessReader { .filter_map(Result::ok) .find_map(|(name, address)| { let name = name?; - if &name == module_name { + if name == module_name { return Some(address); } // Check whether the SO_NAME matches the module name.