19 Aug
2011
19 Aug
'11
8:42 a.m.
At Fri, 19 Aug 2011 16:05:25 +0800, Lu Guanqun wrote:
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
Thanks, applied both now.
Takashi
src/ucm/main.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/ucm/main.c b/src/ucm/main.c index 8e9a85d..9ecbe43 100644 --- 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);