[PATCH] virsocket: Don't preserve errno in virSocketRecvFD()

Michal Privoznik mprivozn at redhat.com
Fri Aug 20 08:55:09 UTC 2021


When setting O_CLOEXEC flag on received FD fails the FD is closed
using VIR_FORCE_CLOSE(). But the call is wrapped in errno save
which is not necessary because VIR_FORCE_CLOSE() preservers errno
value.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/util/virsocket.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/util/virsocket.c b/src/util/virsocket.c
index b971da16e3..a6f185114d 100644
--- a/src/util/virsocket.c
+++ b/src/util/virsocket.c
@@ -477,9 +477,7 @@ virSocketRecvFD(int sock, int fdflags)
     /* set close-on-exec flag */
     if (!MSG_CMSG_CLOEXEC && (fdflags & O_CLOEXEC)) {
         if (virSetCloseExec(fd) < 0) {
-            int saved_errno = errno;
             VIR_FORCE_CLOSE(fd);
-            errno = saved_errno;
             return -1;
         }
     }
-- 
2.31.1




More information about the libvir-list mailing list