adamwill / 389-ds-base

Forked from 389-ds-base 4 years ago
Clone

3b61c03 coverity - posix winsync mem leaks, null check, deadcode, null ref, use after free

Authored and Committed by rmeggins 11 years ago
    coverity - posix winsync mem leaks, null check, deadcode, null ref, use after free
    
    Fixes the following issues:
    13068 Resource leak
    In posix_winsync_pre_ds_mod_group_cb(): Leak of memory or pointers to system resources.
    
    13067 Resource leak
    In posix_winsync_end_update_cb(): Leak of memory or pointers to system resources
    
    13066 Resource leak
    In modGroupMembership(): Leak of memory or pointers to system resources
    
    13065 Resource leak
    In dn_in_set(): Leak of memory or pointers to system resources
    
    13064 Resource leak
    In dn_in_set(): Leak of memory or pointers to system resources
    
    13063 Dereference after null check
    In windows_plugin_add(): Pointer is checked against null but then dereferenced anyway
    
    13062 Explicit null dereferenced
    In searchUid(): Dereference of an explicit null value
    
    13061 Logically dead code
    In check_account_lock(): Code can never be reached because of a logical contradiction
    
    13059 Use after free
    In windows_plugin_cleanup_agmt(): A pointer to freed memory is dereferenced, used as a function argument, or otherwise used
    Reviewed by: nkinder (Thanks!)
    Branch: 389-ds-base-1.2.11
    (cherry picked from commit 1a9b5ffe655c7ec8dd935106376ad8dabe69b561)
    (cherry picked from commit 0c24dfcff939c6c30d336cc498f074d743daf302)