On 12/02/2011 11:34 AM, Li Zhi Hui wrote:
Signed-off-by: Li Zhi Hui<zhihu...@linux.vnet.ibm.com>
---
  qemu-nbd.c |    4 +++-
  1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/qemu-nbd.c b/qemu-nbd.c
index 291cba2..ab7fa6c 100644
--- a/qemu-nbd.c
+++ b/qemu-nbd.c
@@ -500,8 +500,10 @@ int main(int argc, char **argv)
          sharing_fds[0] = tcp_socket_incoming(bindto, port);
      }

-    if (sharing_fds[0] == -1)
+    if (sharing_fds[0] == -1) {
+        g_free(sharing_fds);
          return 1;
+    }

      if (device) {
          int ret;
Zhihui,

Kernel should free all memory used by the process after it exits. So there's no memory leak even without explicit free.


Reply via email to