Hi Surbhi, ACK with minor comment. Short commit msg should be "ntf: ..."
Best Regards, ThuanTr -----Original Message----- From: Surbhi Tripathi <[email protected]> Sent: Tuesday, November 24, 2020 10:11 AM To: Thuan Tran <[email protected]> Cc: [email protected]; Surbhi Tripathi <[email protected]> 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 [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
