Nikolay Sivov : winegstreamer: Get rid of allocated arguments array.

Alexandre Julliard julliard at winehq.org
Tue Mar 26 16:43:13 CDT 2019


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

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Tue Mar 26 08:57:34 2019 +0300

winegstreamer: Get rid of allocated arguments array.

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/winegstreamer/main.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/dlls/winegstreamer/main.c b/dlls/winegstreamer/main.c
index a1b1648..18154fb 100644
--- a/dlls/winegstreamer/main.c
+++ b/dlls/winegstreamer/main.c
@@ -262,7 +262,8 @@ static BOOL CALLBACK init_gstreamer_proc(INIT_ONCE *once, void *param, void **ct
     BOOL *status = param;
     char argv0[] = "wine";
     char argv1[] = "--gst-disable-registry-fork";
-    char **argv = HeapAlloc(GetProcessHeap(), 0, sizeof(char *)*3);
+    char *args[3];
+    char **argv = args;
     int argc = 2;
     GError *err = NULL;
 
@@ -272,12 +273,8 @@ static BOOL CALLBACK init_gstreamer_proc(INIT_ONCE *once, void *param, void **ct
     argv[1] = argv1;
     argv[2] = NULL;
     *status = gst_init_check(&argc, &argv, &err);
-    HeapFree(GetProcessHeap(), 0, argv);
-    if (err) {
-        ERR("Failed to initialize gstreamer: %s\n", err->message);
-        g_error_free(err);
-    }
-    if (*status) {
+    if (*status)
+    {
         HINSTANCE newhandle;
         /* Unloading glib is a bad idea.. it installs atexit handlers,
          * so never unload the dll after loading */
@@ -288,6 +285,11 @@ static BOOL CALLBACK init_gstreamer_proc(INIT_ONCE *once, void *param, void **ct
 
         start_dispatch_thread();
     }
+    else if (err)
+    {
+        ERR("Failed to initialize gstreamer: %s\n", debugstr_a(err->message));
+        g_error_free(err);
+    }
 
     return TRUE;
 }




More information about the wine-cvs mailing list