[PATCH 0/1] MR2746: winepulse: Avoid accessing the product name after freeing it.

[Zebediah Figura (＠zfigura)]

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

[PATCH 1/1] winepulse: Avoid accessing the product name after freeing it.

[Zebediah Figura]

From: Zebediah Figura <zfigura(a)codeweavers.com>

---
 dlls/winepulse.drv/mmdevdrv.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/dlls/winepulse.drv/mmdevdrv.c b/dlls/winepulse.drv/mmdevdrv.c
index 59e2d086d0b..93d392dce21 100644
--- a/dlls/winepulse.drv/mmdevdrv.c
+++ b/dlls/winepulse.drv/mmdevdrv.c
@@ -287,8 +287,13 @@ static WCHAR *get_application_name(BOOL query_app_name)
         }
 
     skip:
+        if (found)
+        {
+            name = wcsdup(productname);
+            free(data);
+            return name;
+        }
         free(data);
-        if (found) return wcsdup(productname);
     }
 
     name = wcsrchr(path, '\\');
-- 
GitLab

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

[PATCH 0/1] MR2746: winepulse: Avoid accessing the product name after freeing it. - approved

[Huw Davies (＠huw)]

This merge request was approved by Huw Davies.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2746