[PATCH 0/1] MR4769: wldap32: Fix a possible memory leak (Coverity).

[Zhiyi Zhang (＠zhiyi)]

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4769

[PATCH 1/1] wldap32: Fix a possible memory leak (Coverity).

[Zhiyi Zhang]

From: Zhiyi Zhang <zzhang(a)codeweavers.com>

---
 dlls/wldap32/ber.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/dlls/wldap32/ber.c b/dlls/wldap32/ber.c
index d3e8adc8433..d38f0119dc4 100644
--- a/dlls/wldap32/ber.c
+++ b/dlls/wldap32/ber.c
@@ -91,8 +91,12 @@ int CDECL WLDAP32_ber_flatten( WLDAP32_BerElement *ber, BERVAL **berval )
     TRACE( "%p, %p\n", ber, berval );
 
     if (ber_flatten( BER(ber), &bervalU )) return WLDAP32_LBER_ERROR;
-    if (!(bervalW = bervalUtoW( bervalU ))) return WLDAP32_LBER_ERROR;
+
+    bervalW = bervalUtoW( bervalU );
     ber_bvfree( bervalU );
+    if (!bervalW)
+        return WLDAP32_LBER_ERROR;
+
     *berval = bervalW;
     return 0;
 }
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/4769

[PATCH 0/1] MR4769: wldap32: Fix a possible memory leak (Coverity). - approved

[Hans Leidekker (＠hans)]

This merge request was approved by Hans Leidekker.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4769