qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [PATCH 1/1] Coverity: Fix failure path for qemu_accept in m


From: Dr. David Alan Gilbert (git)
Subject: [Qemu-devel] [PATCH 1/1] Coverity: Fix failure path for qemu_accept in migration
Date: Wed, 19 Mar 2014 11:13:44 +0000

From: "Dr. David Alan Gilbert" <address@hidden>

Coverity defects 1005733 & 1005734 complain about passing a -ve value
to closesocket in the error paths on incoming migration.

Stash the error value and print it in the message (previously we gave
no indication of the reason for the failure)

Signed-off-by: Dr. David Alan Gilbert <address@hidden>
---
 migration-tcp.c  | 11 ++++++-----
 migration-unix.c | 11 ++++++-----
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/migration-tcp.c b/migration-tcp.c
index 782572d..5c96cd3 100644
--- a/migration-tcp.c
+++ b/migration-tcp.c
@@ -56,19 +56,20 @@ static void tcp_accept_incoming_migration(void *opaque)
     socklen_t addrlen = sizeof(addr);
     int s = (intptr_t)opaque;
     QEMUFile *f;
-    int c;
+    int c, err;
 
     do {
         c = qemu_accept(s, (struct sockaddr *)&addr, &addrlen);
-    } while (c == -1 && socket_error() == EINTR);
+        err = socket_error();
+    } while (c == -1 && err == EINTR);
     qemu_set_fd_handler2(s, NULL, NULL, NULL, NULL);
     closesocket(s);
 
     DPRINTF("accepted migration\n");
 
-    if (c == -1) {
-        fprintf(stderr, "could not accept migration connection\n");
-        goto out;
+    if (c < 0) {
+        fprintf(stderr, "could not accept migration connection (%d)\n", err);
+        return;
     }
 
     f = qemu_fopen_socket(c, "rb");
diff --git a/migration-unix.c b/migration-unix.c
index 651fc5b..e7cf9a2 100644
--- a/migration-unix.c
+++ b/migration-unix.c
@@ -56,19 +56,20 @@ static void unix_accept_incoming_migration(void *opaque)
     socklen_t addrlen = sizeof(addr);
     int s = (intptr_t)opaque;
     QEMUFile *f;
-    int c;
+    int c, err;
 
     do {
         c = qemu_accept(s, (struct sockaddr *)&addr, &addrlen);
-    } while (c == -1 && errno == EINTR);
+        err = errno;
+    } while (c == -1 && err == EINTR);
     qemu_set_fd_handler2(s, NULL, NULL, NULL, NULL);
     close(s);
 
     DPRINTF("accepted migration\n");
 
-    if (c == -1) {
-        fprintf(stderr, "could not accept migration connection\n");
-        goto out;
+    if (c < 0) {
+        fprintf(stderr, "could not accept migration connection (%d)\n", err);
+        return;
     }
 
     f = qemu_fopen_socket(c, "rb");
-- 
1.8.5.3




reply via email to

[Prev in Thread] Current Thread [Next in Thread]