Changeset 3612e9f in mainline

Timestamp:

2018-11-22T20:05:37Z (5 years ago)

Author:

Jakub Jermar <jakub@…>

Parents:

ca21f1e2

Message:

Fix function reference counting

After commit 498ced1, a function is created with an implicit reference.
Adding an extra reference for creation thus adds a reference that will
never be dropped and the function will be leaked.

This commit removes the extra reference.

File:

• uspace/lib/drv/generic/driver.c (modified) (2 diffs)

uspace/lib/drv/generic/driver.c

@@ -733 +733 @@
                 return NULL;
 
-        /* Add one reference that will be dropped by ddf_fun_destroy() */
         fun->dev = dev;
-        fun_add_ref(fun);
+        dev_add_ref(fun->dev);
 
         fun->bound = false;
@@ -743 +742 @@
                 fun->name = str_dup(name);
                 if (fun->name == NULL) {
-                        delete_function(fun);
+                        fun_del_ref(fun);       /* fun is destroyed */
                         return NULL;
                 }