On Fri, Mar 19, 2021 at 03:25:25PM +0200, Mahmoud Mandour wrote: > Changed the allocations of fv_VuDev structs, VuDev structs, and > fv_QueueInfo strcuts from using calloc()/realloc() & free() to using > the equivalent functions from GLib. > > In instances, removed the pair of allocation and assertion for > non-NULL checking with a GLib function that aborts on error. > > Removed NULL-checking for fv_VuDev struct allocation and used > a GLib function that crashes on error; namely, g_new0(). This > is because allocating one struct should not be a problem on an > healthy system. Also following the pattern of aborting-on-null > behaviour that is taken with allocating VuDev structs and > fv_QueueInfo structs. > > Signed-off-by: Mahmoud Mandour <ma.mando...@gmail.com> > --- > tools/virtiofsd/fuse_virtio.c | 19 ++++++------------- > 1 file changed, 6 insertions(+), 13 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
signature.asc
Description: PGP signature
