Merge pull request #388 from icmccorm/master

Jasper-Bekkers · web-flow · commit eac155e642bd · 2024-12-12T11:03:42.000+01:00
Fixed memory leak in zmq::z85_decode
diff --git a/src/lib.rs b/src/lib.rs
@@ -1343,7 +1343,7 @@ pub fn z85_decode(data: &str) -> result::Result<Vec<u8>, DecodeError> {
     let c_str = ffi::CString::new(data)?;
 
     unsafe {
-        zmq_sys::zmq_z85_decode(dest.as_mut_ptr(), c_str.into_raw());
+        zmq_sys::zmq_z85_decode(dest.as_mut_ptr(), c_str.as_ptr());
     }
 
     Ok(dest)

Original file line number	Diff line number	Diff line change
`@@ -1343,7 +1343,7 @@ pub fn z85_decode(data: &str) -> result::Result<Vec<u8>, DecodeError> {`
`1343`	`1343`	`let c_str = ffi::CString::new(data)?;`
`1344`	`1344`
`1345`	`1345`	`unsafe {`
`1346`		`- zmq_sys::zmq_z85_decode(dest.as_mut_ptr(), c_str.into_raw());`
	`1346`	`+ zmq_sys::zmq_z85_decode(dest.as_mut_ptr(), c_str.as_ptr());`
`1347`	`1347`	`}`
`1348`	`1348`
`1349`	`1349`	`Ok(dest)`