fix modargs memory leak

git-svn-id: file:///home/lennart/svn/public/pulseaudio/trunk@68 fefdeb5f-60dc-0310-8127-8f9354f1896f

diff --git a/src/module-oss-mmap.c b/src/module-oss-mmap.c
index 020daa2c2cb3aa7cd9824859f45d7dcd66abd289..ceaae4931c7eac3f03166274999bf1452b48fc86 100644 (file)
--- a/src/module-oss-mmap.c
+++ b/src/module-oss-mmap.c
@@ -306,11 +306,16 @@ int pa_module_init(struct pa_core *c, struct pa_module*m) {
     u->mainloop_source = c->mainloop->source_io(c->mainloop, u->fd, (u->source ? PA_MAINLOOP_API_IO_EVENT_INPUT : 0) | (u->sink ? PA_MAINLOOP_API_IO_EVENT_OUTPUT : 0), io_callback, u);
     assert(u->mainloop_source);
 
+    pa_modargs_free(ma);
+    
     return 0;
 
 fail:
     pa_module_done(c, m);
 
+    if (ma)
+        pa_modargs_free(ma);
+    
     return -1;
 }
 

diff --git a/src/module-oss.c b/src/module-oss.c
index 5a6513c39a9c78f17f30f506a2731b99282d2fac..48d104865bfbe706417224039aed775ed1778efe 100644 (file)
--- a/src/module-oss.c
+++ b/src/module-oss.c
@@ -229,6 +229,8 @@ int pa_module_init(struct pa_core *c, struct pa_module*m) {
     
     m->userdata = u;
 
+    pa_modargs_free(ma);
+
     return 0;
 
 fail:

diff --git a/src/module-protocol-stub.c b/src/module-protocol-stub.c
index 4f82d4e02a0296344240d6fa8ab166a83a60000e..7338abc94d6f590b5f5ad50ced3f2a7f993e7a54 100644 (file)
--- a/src/module-protocol-stub.c
+++ b/src/module-protocol-stub.c
@@ -107,23 +107,30 @@ struct pa_socket_server *create_socket_server(struct pa_core *c, struct pa_modar
 
 int pa_module_init(struct pa_core *c, struct pa_module*m) {
     struct pa_socket_server *s;
-    struct pa_modargs *ma;
+    struct pa_modargs *ma = NULL;
+    int ret = -1;
     assert(c && m);
 
     if (!(ma = pa_modargs_new(m->argument, valid_modargs))) {
         fprintf(stderr, "Failed to parse module arguments\n");
-        return -1;
+        goto finish;
     }
 
     if (!(s = create_socket_server(c, ma)))
-        return -1;
+        goto finish;
 
     if (!(m->userdata = protocol_new(c, s, m, ma))) {
         pa_socket_server_free(s);
-        return -1;
+        goto finish;
     }
 
-    return 0;
+    ret = 0;
+
+finish:
+    if (ma)
+        pa_modargs_free(ma);
+
+    return ret;
 }
 
 void pa_module_done(struct pa_core *c, struct pa_module*m) {

diff --git a/src/todo b/src/todo
index 2bec6f183361855495325ef8b5ce0442bfe308a7..5e90dfdae7a8704a1566f1a03d243d958eb9ebe5 100644 (file)
--- a/src/todo
+++ b/src/todo
@@ -1,5 +1,3 @@
-- modargs memory leak
-
 - clean secure directory handling (with username)
 
 - native library/protocol: