On Tue, Mar 23, 2021 at 01:57:05PM +0000, Stefan Hajnoczi wrote: > On Fri, Mar 19, 2021 at 03:25:21PM +0200, Mahmoud Mandour wrote: > > @@ -629,9 +628,6 @@ int fuse_reply_ioctl_retry(fuse_req_t req, const struct > > iovec *in_iov, > > > > res = send_reply_iov(req, 0, iov, count); > > out: > > - free(in_fiov); > > - free(out_fiov); > > - > > return res; > > > > enomem: > > This hunk doesn't seem related to anything in this patch. Was it > included accidentally?
Thanks for explaining that in_fiov/out_fiov are allocated by fuse_ioctl_iovec_copy() earlier in this function. I missed that. Please use Reply-All on mailing list emails so that the mailing like and all other CC email addresses are included in the discussion. Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
signature.asc
Description: PGP signature