[Libguestfs] [PATCH nbdkit] rust: Drop box ptr

alan somers asomers at gmail.com
Thu Sep 29 15:56:34 UTC 2022


This change is correct.  You can consider it reviewed by me and commit
it.  As for the warning from Mockall, that's a bug in Mockall.  I'll
fix it.

On Thu, Sep 29, 2022 at 8:21 AM Richard W.M. Jones <rjones at redhat.com> wrote:
>
> Because of this recent change to rust, our close function warned that
> we calculate Box::from_raw but never use it.  I added the suggested
> call to drop() around the function.
>
> https://github.com/rust-lang/rust/pull/99270
> ---
>  plugins/rust/src/lib.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/plugins/rust/src/lib.rs b/plugins/rust/src/lib.rs
> index 69be36fb2..128334ef2 100644
> --- a/plugins/rust/src/lib.rs
> +++ b/plugins/rust/src/lib.rs
> @@ -368,7 +368,7 @@ mod ffi {
>
>      pub(super) extern fn close(selfp: *mut c_void) {
>          unsafe {
> -            Box::from_raw(selfp as *mut Box<dyn Server>);
> +            drop(Box::from_raw(selfp as *mut Box<dyn Server>));
>          }
>      }
>
> --
> 2.37.0.rc2
>



More information about the Libguestfs mailing list