diff options
| author | cypherpunks <cypherpunks@torproject.org> | 2018-10-17 00:16:21 +0000 |
|---|---|---|
| committer | cypherpunks <cypherpunks@torproject.org> | 2018-10-17 00:16:21 +0000 |
| commit | 2f0744b3e6f579f25db1ed6e048d0418ac2ab570 (patch) | |
| tree | b722ef373da7828d2c50c9dd2c22eaed27cbd859 | |
| parent | e97adaf8dc13a4f500fab3d70c9c31400a01954f (diff) | |
| download | tor-2f0744b3e6f579f25db1ed6e048d0418ac2ab570.tar.gz tor-2f0744b3e6f579f25db1ed6e048d0418ac2ab570.zip | |
rust/tor_util: drop unsafe block in cstr!
This is unnecessary just to get an empty string, there's Default::default().
Fix on 8fff331bb095dc6f5e2fe2ecfc9ab08ea9e2fe97.
| -rw-r--r-- | changes/ticket28077 | 3 | ||||
| -rw-r--r-- | src/rust/tor_util/strings.rs | 6 |
2 files changed, 4 insertions, 5 deletions
diff --git a/changes/ticket28077 b/changes/ticket28077 new file mode 100644 index 0000000000..2b5afb1678 --- /dev/null +++ b/changes/ticket28077 @@ -0,0 +1,3 @@ + o Code simplification and refactoring: + - Remove unnecessarily unsafe code from the rust macro cstr!. Closes + ticket 28077. diff --git a/src/rust/tor_util/strings.rs b/src/rust/tor_util/strings.rs index d64275e06b..71a908a58c 100644 --- a/src/rust/tor_util/strings.rs +++ b/src/rust/tor_util/strings.rs @@ -105,11 +105,7 @@ macro_rules! cstr { ($($bytes:expr),*) => ( ::std::ffi::CStr::from_bytes_with_nul( concat!($($bytes),*, "\0").as_bytes() - ).unwrap_or( - unsafe{ - ::std::ffi::CStr::from_bytes_with_nul_unchecked(b"\0") - } - ) + ).unwrap_or_default() ) } |