[389-commits] ldap/servers

[Richard Allen Megginson]

 ldap/servers/plugins/replication/windows_private.c |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit ba92e5287970d09c4c280eda56c4e58bc064a2c1
Author: Rich Megginson <rmeggins at redhat.com>
Date:   Tue Aug 28 14:18:54 2012 -0600

    fix coverity resource leak in windows_plugin_add
    
    13056 Resource leak
    In windows_plugin_add(): Leak of memory or pointers to system resources (CWE-404)

diff --git a/ldap/servers/plugins/replication/windows_private.c b/ldap/servers/plugins/replication/windows_private.c
index 3ee6a71..cc007aa 100644
--- a/ldap/servers/plugins/replication/windows_private.c
+++ b/ldap/servers/plugins/replication/windows_private.c
@@ -1251,6 +1251,8 @@ windows_plugin_add(void **theapi, int maxapi)
             PR_INSERT_BEFORE(wpi, elem);
             wpi = NULL; /* owned by list now */
         }
+        /* if we got here and wpi is not NULL we need to free wpi */
+        slapi_ch_free((void **)&wpi);
         return 0;
     }
     return -1;