ucm: fix seg fault in execute_cset()

When there's no space in 'cset', 'pos' gets NULL, dereferencing 'pos' gets a
seg fault. We need to be more robust.

Signed-off-by: Lu Guanqun <guanqun.lu@intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

diff --git a/src/ucm/main.c b/src/ucm/main.c
index 8e9a85d2939f412518529f9a6a20c7978b8007f5..9ecbe4351cf3fcb9bc52686eac28715fac92207c 100644 (file)
--- a/src/ucm/main.c
+++ b/src/ucm/main.c
@@ -196,7 +196,8 @@ static int execute_cset(snd_ctl_t *ctl, char *cset)
                goto __fail;
        err = 0;
       __fail:
-       *pos = ' ';
+       if (pos != NULL)
+               *pos = ' ';
 
        if (id != NULL)
                free(id);