diff options
author | Laurent Vivier <lvivier@redhat.com> | 2025-07-17 17:08:03 +0200 |
---|---|---|
committer | Jason Wang <jasowang@redhat.com> | 2025-07-21 10:22:57 +0800 |
commit | f74e4f2e60c3b2c6e2f13eb115478df116148fe6 (patch) | |
tree | 58ba608120082d068b5174d9deb2d9925a222064 | |
parent | c40ef7243fdbec816242cb0ded569a61270ea7c3 (diff) | |
download | qemu-f74e4f2e60c3b2c6e2f13eb115478df116148fe6.zip qemu-f74e4f2e60c3b2c6e2f13eb115478df116148fe6.tar.gz qemu-f74e4f2e60c3b2c6e2f13eb115478df116148fe6.tar.bz2 |
net/passt: Check return value of g_remove() in net_passt_cleanup() (CID 1612369)
If g_remove() fails, use warn_report() to log an error.
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/passt.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/passt.c b/net/passt.c index ef59d06..43c336e 100644 --- a/net/passt.c +++ b/net/passt.c @@ -103,7 +103,10 @@ static void net_passt_cleanup(NetClientState *nc) #endif kill(s->pid, SIGTERM); - g_remove(s->pidfile); + if (g_remove(s->pidfile) != 0) { + warn_report("Failed to remove passt pidfile %s: %s", + s->pidfile, strerror(errno)); + } g_free(s->pidfile); g_ptr_array_free(s->args, TRUE); } |