snd_user_file: fix memory leak

Call wordfree if and only if wordfree returns zero or WRDE_NOSPACE

Signed-off-by: Andreas Schwab <schwab@suse.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

diff --git a/src/userfile.c b/src/userfile.c
index 3a738366f1ea7bbe570b442b77fb9377168ba0c1..72779da4ebafee554a1af1bbf34a03ac7bb66863 100644 (file)
--- a/src/userfile.c
+++ b/src/userfile.c
@@ -44,19 +44,20 @@ int snd_user_file(const char *file, char **result)
        err = wordexp(file, &we, WRDE_NOCMD);
        switch (err) {
        case WRDE_NOSPACE:
+               wordfree(&we);
                return -ENOMEM;
        case 0:
                if (we.we_wordc == 1)
                        break;
+               wordfree(&we);
                /* fall thru */
        default:
-               wordfree(&we);
                return -EINVAL;
        }
        *result = strdup(we.we_wordv[0]);
+       wordfree(&we);
        if (*result == NULL)
                return -ENOMEM;
-       wordfree(&we);
        return 0;
 }