]> git.proxmox.com Git - mirror_qemu.git/commitdiff
event_notifier: avoid dandling file descriptor in event_notifier_cleanup
authorFrediano Ziglio <fziglio@redhat.com>
Wed, 23 Oct 2019 12:26:51 +0000 (13:26 +0100)
committerLaurent Vivier <laurent@vivier.eu>
Thu, 24 Oct 2019 17:26:52 +0000 (19:26 +0200)
If rfd is equal to wfd the file descriptor is closed but
rfd will still have the closed value.
The EventNotifier structure should not be used again after calling
event_notifier_cleanup or should be initialized again but make
sure to not have dandling file descriptors around.

Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20191023122652.2999-2-fziglio@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
util/event_notifier-posix.c

index 73c4046b587184f9110fa1a1e4d6a974f3512c03..00d93204f988c753d1ba308421936c457277ff25 100644 (file)
@@ -80,8 +80,8 @@ void event_notifier_cleanup(EventNotifier *e)
 {
     if (e->rfd != e->wfd) {
         close(e->rfd);
-        e->rfd = -1;
     }
+    e->rfd = -1;
     close(e->wfd);
     e->wfd = -1;
 }