Changeset de16f89 in mainline for uspace/drv/audio/hdaudio/hdaudio.c

Timestamp:

2014-09-05T08:52:51Z (10 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

089901e

Parents:

3fec817

Message:

Add cleanup code to error paths.

File:

• uspace/drv/audio/hdaudio/hdaudio.c (modified) (5 diffs)

uspace/drv/audio/hdaudio/hdaudio.c

@@ -150 +150 @@
 static int hda_dev_add(ddf_dev_t *dev)
 {
-        ddf_fun_t *fun_pcm;
+        ddf_fun_t *fun_pcm = NULL;
         hda_t *hda = NULL;
         hw_res_list_parsed_t res;
         irq_code_t irq_code;
-        irq_cmd_t *cmds;
+        irq_cmd_t *cmds = NULL;
         size_t ncmds_base;
         size_t ncmds_sdesc;
         size_t ncmds;
         int i;
-        void *regs;
+        void *regs = NULL;
         int rc;
 
         ddf_msg(LVL_NOTE, "hda_dev_add()");
+        hw_res_list_parsed_init(&res);
 
         hda = ddf_dev_data_alloc(dev, sizeof(hda_t));
@@ -181 +182 @@
 
         ddf_msg(LVL_NOTE, "get HW res list");
-        hw_res_list_parsed_init(&res);
         rc = hw_res_get_list_parsed(hda->parent_sess, &res, 0);
         if (rc != EOK) {
@@ -270 +270 @@
         }
 
+        free(cmds);
+        cmds = NULL;
+
         if (hda_ctl_init(hda) == NULL) {
                 rc = EIO;
@@ -295 +298 @@
 
         ddf_fun_add_to_category(fun_pcm, "audio-pcm");
+
+        hw_res_list_parsed_clean(&res);
         return EOK;
 error:
+        if (fun_pcm != NULL)
+                ddf_fun_destroy(fun_pcm);
         if (hda != NULL) {
                 if (hda->ctl != NULL)
                         hda_ctl_fini(hda->ctl);
         }
+        free(cmds);
+        // pio_disable(regs);
+        hw_res_list_parsed_clean(&res);
 
         ddf_msg(LVL_NOTE, "Failing hda_dev_add() -> %d", rc);
@@ -323 +333 @@
         }
 
+        hda_ctl_fini(hda->ctl);
+        // pio_disable(regs);
         return EOK;
 }