Fixed a theoretical memory leak with no practical effects in ae_kqueu…

…e.c and ae_epoll.c, thanks to magicyang87 for reporting it.
henrywoo · Dec 7, 2011 · fb293cc · fb293cc
1 parent 237194b
commit fb293cc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/ae_epoll.c b/src/ae_epoll.c
@@ -14,7 +14,10 @@ static int aeApiCreate(aeEventLoop *eventLoop) {
 
     if (!state) return -1;
     state->epfd = epoll_create(1024); /* 1024 is just an hint for the kernel */
-    if (state->epfd == -1) return -1;
+    if (state->epfd == -1) {
+        zfree(state);
+        return -1;
+    }
     eventLoop->apidata = state;
     return 0;
 }

diff --git a/src/ae_kqueue.c b/src/ae_kqueue.c
@@ -16,7 +16,10 @@ static int aeApiCreate(aeEventLoop *eventLoop) {
 
     if (!state) return -1;
     state->kqfd = kqueue();
-    if (state->kqfd == -1) return -1;
+    if (state->kqfd == -1) {
+        zfree(state);
+        return -1;
+    }
     eventLoop->apidata = state;
 
     return 0;