summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorLaurent Vivier <lvivier@redhat.com>2025-07-17 17:08:05 +0200
committerJason Wang <jasowang@redhat.com>2025-07-21 10:23:17 +0800
commitae9b09972bbf8ff49ae0edf3241fb413391b15ce (patch)
tree4b2665fad2cd91ab35bd7dc5bc99eea9c8fd6f76
parent667ad57b7636881eb79b0b319951f3cba9bbf27e (diff)
downloadfocaccia-qemu-ae9b09972bbf8ff49ae0edf3241fb413391b15ce.tar.gz
focaccia-qemu-ae9b09972bbf8ff49ae0edf3241fb413391b15ce.zip
net/vhost-user: Remove unused "err" from chr_closed_bh() (CID 1612365)
The "err" variable was declared but never used within the
chr_closed_bh() function. This resulted in a dead code
warning (CID 1612365) from Coverity.

Remove the unused variable and the associated error block
to resolve the issue.

Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Jason Wang <jasowang@redhat.com>
-rw-r--r--net/vhost-user.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/net/vhost-user.c b/net/vhost-user.c
index cec83e925f..8b96157145 100644
--- a/net/vhost-user.c
+++ b/net/vhost-user.c
@@ -298,7 +298,6 @@ static void chr_closed_bh(void *opaque)
     const char *name = opaque;
     NetClientState *ncs[MAX_QUEUE_NUM];
     NetVhostUserState *s;
-    Error *err = NULL;
     int queues, i;
 
     queues = qemu_find_net_clients_except(name, ncs,
@@ -317,9 +316,6 @@ static void chr_closed_bh(void *opaque)
     qemu_chr_fe_set_handlers(&s->chr, NULL, NULL, net_vhost_user_event,
                              NULL, opaque, NULL, true);
 
-    if (err) {
-        error_report_err(err);
-    }
     qapi_event_send_netdev_vhost_user_disconnected(name);
 }