Re: [PATCH] util: avoid fds leak on virEventPollInit

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

 



On 07/19/2011 05:00 PM, Osier Yang wrote:
于 2011年07月19日 16:25, Alex Jia 写道:
* src/util/event_poll.c: fix file descriptors leak on virEventPollInit.

Detected in valgrind run:
==1254==
==1254== FILE DESCRIPTORS: 6 open at exit.
==1254== Open file descriptor 5:
==1254==    at 0x30736D9D47: pipe2 (syscall-template.S:82)
==1254==    by 0x4DD6267: rpl_pipe2 (pipe2.c:61)
==1254==    by 0x4C4C1C5: virEventPollInit (event_poll.c:648)
==1254==    by 0x4C4AA94: virEventRegisterDefaultImpl (event.c:208)
==1254==    by 0x42150C: main (virsh.c:13790)
==1254==
==1254== Open file descriptor 4:
==1254==    at 0x30736D9D47: pipe2 (syscall-template.S:82)
==1254==    by 0x4DD6267: rpl_pipe2 (pipe2.c:61)
==1254==    by 0x4C4C1C5: virEventPollInit (event_poll.c:648)
==1254==    by 0x4C4AA94: virEventRegisterDefaultImpl (event.c:208)
==1254==    by 0x42150C: main (virsh.c:13790)
==1254==

* how to reproduce?
   % valgrind -v --track-fds=yes virsh list

---
  src/util/event_poll.c |   13 ++++++++++---
  1 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/util/event_poll.c b/src/util/event_poll.c
index 285ba50..1e4ef96 100644
--- a/src/util/event_poll.c
+++ b/src/util/event_poll.c
@@ -639,6 +639,8 @@ static void virEventPollHandleWakeup(int watch ATTRIBUTE_UNUSED,

  int virEventPollInit(void)
  {
+    int ret = -1;
+
      if (virMutexInit(&eventLoop.lock)<  0) {
          virReportSystemError(errno, "%s",
                               _("Unable to initialize mutex"));
@@ -648,7 +650,7 @@ int virEventPollInit(void)
      if (pipe2(eventLoop.wakeupfd, O_CLOEXEC | O_NONBLOCK)<  0) {
          virReportSystemError(errno, "%s",
                               _("Unable to setup wakeup pipe"));
-        return -1;
+        goto cleanup;
      }

      if (virEventPollAddHandle(eventLoop.wakeupfd[0],
@@ -657,10 +659,15 @@ int virEventPollInit(void)
          virEventError(VIR_ERR_INTERNAL_ERROR,
                        _("Unable to add handle %d to event loop"),
                        eventLoop.wakeupfd[0]);
-        return -1;
+        goto cleanup;
      }

-    return 0;
+    ret = 0;
+
+cleanup:
+    close(eventLoop.wakeupfd[0]);
+    close(eventLoop.wakeupfd[1]);

s/close/VIR_FORCE_CLOSE/
If do this, I need also include 'files.h', VIR_FORCE_CLOSE is defined in this header.

Thanks,
Alex

+    return ret;
  }

  static int virEventPollInterruptLocked(void)


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]