Bugfix: don't leak on WithOrigin (or WithRawSiginfo) destruction

Author: vorner

CHANGELOG.md

@@ -5,6 +5,7 @@
 * Folding the signal-hook-sys inline (but still compiling C code only
   conditionally).
 * `WithRawSiginfo` extractor (to get hands on the raw `siginfo_t`).
+* Bugfix: Don't leak on WithOrigin destruction.
 
 # 0.3.1
 

Cargo.toml

@@ -20,6 +20,7 @@ maintenance = { status = "actively-developed" }
 channel = []
 default = ["channel", "iterator"]
 iterator = []
+# TODO: Unify them on the next breaking release.
 extended-siginfo = ["channel", "iterator", "extended-siginfo-raw"]
 extended-siginfo-raw = ["cc"]
 

src/iterator/exfiltrator/raw.rs

@@ -20,6 +20,15 @@ use crate::low_level::channel::Channel;
 #[derive(Default, Debug)]
 pub struct Slot(AtomicPtr<Channel<siginfo_t>>);
 
+impl Drop for Slot {
+    fn drop(&mut self) {
+        let ptr = self.0.load(Ordering::Acquire);
+        if !ptr.is_null() {
+            drop(unsafe { Box::from_raw(ptr) });
+        }
+    }
+}
+
 /// The [`Exfiltrator`][crate::iterator::exfiltrator::Exfiltrator] that produces the raw
 /// [`libc::siginfo_t`]. Note that it might look differently on different OSes and its API is a
 /// little bit more limited than its C counterpart.