Michael Stefaniuc : wldap32: Remove redundant NULL check before heap_free().

Alexandre Julliard julliard at winehq.org
Mon Mar 15 16:59:11 CDT 2021


Module: wine
Branch: master
Commit: 35ee15eab36eafc660a50284529613ed9dd044da
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=35ee15eab36eafc660a50284529613ed9dd044da

Author: Michael Stefaniuc <mstefani at winehq.org>
Date:   Sun Mar 14 22:54:36 2021 +0100

wldap32: Remove redundant NULL check before heap_free().

Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/wldap32/page.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/dlls/wldap32/page.c b/dlls/wldap32/page.c
index a16ea52409e..07f29be0d50 100644
--- a/dlls/wldap32/page.c
+++ b/dlls/wldap32/page.c
@@ -241,11 +241,8 @@ ULONG CDECL ldap_get_paged_count( WLDAP32_LDAP *ld, PLDAPSearch search,
         return WLDAP32_LDAP_SUCCESS;
     }
 
-    if (search->cookie)
-    {
-        heap_free( search->cookie );
-        search->cookie = NULL;
-    }
+    heap_free( search->cookie );
+    search->cookie = NULL;
 
     ret = ldap_parse_page_controlW( ld, server_ctrls, count, &search->cookie );
     if (ret == WLDAP32_LDAP_SUCCESS)




More information about the wine-cvs mailing list