On 7/20/21 2:54 PM, Markus Armbruster wrote: > vhost_user_get_config() can fail without setting an error. Unclean. > Its caller vhost_dev_get_config() compensates by substituting a > generic error then. Goes back to commit 50de51387f "vhost: > Distinguish errors in vhost_dev_get_config()". > > Clean up by moving the generic error from vhost_dev_get_config() to > all the failure paths that neglect to set an error. > > Cc: Kevin Wolf <kw...@redhat.com> > Cc: Michael S. Tsirkin <m...@redhat.com> > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > hw/virtio/vhost-user.c | 2 ++ > hw/virtio/vhost.c | 10 ++-------- > 2 files changed, 4 insertions(+), 8 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>