commit b7d7df929f6954d8cf5a0638b227393d974e99ab parent c2e5b3d2d5a697fc1fe0ed9af30bd89f3a9679d9 Author: Gabriele Svelto <gsvelto@mozilla.com> Date: Mon, 6 Oct 2025 13:46:44 +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.