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)