[PATCH 0/2] MR1774: Coverity fixes

[Fabian Maurer (＠DarkShadow44)]

Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de>

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

[PATCH 1/2] urlon: Pass correct parameter to realloc (Coverity)

[Fabian Maurer]

From: Fabian Maurer <dark.shadow4(a)web.de>

Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de>
---
 dlls/urlmon/sec_mgr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/urlmon/sec_mgr.c b/dlls/urlmon/sec_mgr.c
index e6d1864b1e7..0d81e4dfda9 100644
--- a/dlls/urlmon/sec_mgr.c
+++ b/dlls/urlmon/sec_mgr.c
@@ -1276,7 +1276,7 @@ static LPDWORD build_zonemap_from_reg(void)
             if (used == allocated) {
                 LPDWORD new_data;
 
-                new_data = realloc(data, allocated * sizeof(DWORD));
+                new_data = realloc(data, allocated * 2 * sizeof(DWORD));
                 if (!new_data)
                     goto cleanup;
                 memset(new_data + allocated, 0, allocated * sizeof(DWORD));
-- 
GitLab


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

[PATCH 2/2] wldap32: Free resource in error case (Coverity)

[Fabian Maurer]

From: Fabian Maurer <dark.shadow4(a)web.de>

---
 dlls/wldap32/misc.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/dlls/wldap32/misc.c b/dlls/wldap32/misc.c
index edf1f196353..10b506dc338 100644
--- a/dlls/wldap32/misc.c
+++ b/dlls/wldap32/misc.c
@@ -413,10 +413,15 @@ ULONG CDECL WLDAP32_ldap_result( LDAP *ld, ULONG msgid, ULONG all, struct l_time
 
         ret = ldap_result( CTX(ld), msgid, all, timeout ? &timeval : NULL, &msgU );
     }
-    if (msgU && (msg = calloc( 1, sizeof(*msg) )))
+    if (msgU)
     {
-        MSG(msg) = msgU;
-        *res = msg;
+        if (msg = calloc( 1, sizeof(*msg) ))
+        {
+            MSG(msg) = msgU;
+            *res = msg;
+        }
+        else
+            free (msgU);
     }
 
     return ret;
-- 
GitLab

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