On Mon, Sep 15, 2025 at 10:30:56PM +0300, Vladimir Sementsov-Ogievskiy wrote: > Currently, we just always pass NULL as errp argument. That doesn't > look good. > > Some realizations of interface may actually report errors. > Channel-socket realization actually either ignore or crash on > errors, but we are going to straighten it out to always reporting > an errp in further commits. > > So, convert all callers to either handle the error (where environment > allows) or explicitly use &error_abort. > > Take also a chance to change the return value to more convenient > bool (keeping also in mind, that underlying realizations may > return -1 on failure, not -errno). > > Suggested-by: Daniel P. Berrangé <berra...@redhat.com> > Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@yandex-team.ru> > --- > block/nbd.c | 4 +++- > chardev/char-socket.c | 20 ++++++++++++++++---- > hw/remote/proxy.c | 6 +++++- > hw/remote/remote-obj.c | 6 +++++- > hw/vfio-user/proxy.c | 11 ++++++++--- > include/io/channel.h | 6 +++--- > io/channel.c | 4 ++-- > nbd/server.c | 4 +++- > scsi/qemu-pr-helper.c | 9 ++++++--- > tests/unit/io-channel-helpers.c | 5 +++-- > tests/unit/test-io-channel-tls.c | 4 ++-- > tools/i386/qemu-vmsr-helper.c | 6 ++++-- > ui/vnc.c | 2 +- > util/vhost-user-server.c | 7 ++++++- > 14 files changed, 67 insertions(+), 27 deletions(-)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> With 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 :|
