On Tue, Jun 30, 2020 at 11:03:28AM +0200, Markus Armbruster wrote: > Receiving the error in a local variable only to free it is less clear > (and also less efficient) than passing NULL. Clean up. > > Cc: Daniel P. Berrange <berra...@redhat.com> > Cc: Jerome Forissier <jer...@forissier.org> > CC: Greg Kurz <gr...@kaod.org> > Signed-off-by: Markus Armbruster <arm...@redhat.com> > Reviewed-by: Greg Kurz <gr...@kaod.org> > --- > chardev/char-socket.c | 6 ++---- > hw/9pfs/9p.c | 6 ++---- > hw/arm/virt.c | 4 +--- > hw/ppc/spapr_drc.c | 4 +--- > ui/vnc.c | 3 +-- > 5 files changed, 7 insertions(+), 16 deletions(-)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|