mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-06 02:50:43 +00:00
wimax/i2400m: fix a memory leak bug
In i2400m_barker_db_init(), 'options_orig' is allocated through kstrdup() to hold the original command line options. Then, the options are parsed. However, if an error occurs during the parsing process, 'options_orig' is not deallocated, leading to a memory leak bug. To fix this issue, free 'options_orig' before returning the error. Signed-off-by: Wenwen Wang <wenwen@cs.uga.edu> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
3434341004
commit
44ef3a0325
1 changed files with 3 additions and 1 deletions
|
@ -351,13 +351,15 @@ int i2400m_barker_db_init(const char *_options)
|
|||
}
|
||||
result = i2400m_barker_db_add(barker);
|
||||
if (result < 0)
|
||||
goto error_add;
|
||||
goto error_parse_add;
|
||||
}
|
||||
kfree(options_orig);
|
||||
}
|
||||
return 0;
|
||||
|
||||
error_parse_add:
|
||||
error_parse:
|
||||
kfree(options_orig);
|
||||
error_add:
|
||||
kfree(i2400m_barker_db);
|
||||
return result;
|
||||
|
|
Loading…
Reference in a new issue