Hi Surbhi,

ACK with minor comment.
Short commit msg should be "ntf: ..."

Best Regards,
ThuanTr

-----Original Message-----
From: Surbhi Tripathi <surbhi.tripa...@dektech.com.au> 
Sent: Tuesday, November 24, 2020 10:11 AM
To: Thuan Tran <thuan.t...@dektech.com.au>
Cc: opensaf-devel@lists.sourceforge.net; Surbhi Tripathi 
<surbhi.tripa...@dektech.com.au>
Subject: [PATCH 1/1] clm: fix memleak detected by valgrind [#3238]

---
 src/ntf/ntfd/ntfs_main.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/ntf/ntfd/ntfs_main.c b/src/ntf/ntfd/ntfs_main.c
index 2054205e2..d37d7f8b3 100644
--- a/src/ntf/ntfd/ntfs_main.c
+++ b/src/ntf/ntfd/ntfs_main.c
@@ -343,6 +343,7 @@ int main(int argc, char *argv[])
                if (fds[FD_TERM].revents & POLLIN) {
                        TRACE("Exit via FD_TERM calling stop_ntfimcn()");
                        (void)stop_ntfimcn();
+                        saClmFinalize(ntfs_cb->clm_hdl);
                        daemon_exit();
                }
 
-- 
2.17.1



_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to