diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-10 11:54:58 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-09-13 01:24:57 (GMT) |
commit | 39be6b8b0cc84b33cd17bc570e80a25126f87c9c (patch) | |
tree | 7fa6601f9667a769d81c26592cd0b360d97f2ff6 | |
parent | 60aff4ffba3336cd97789b60f086d47b1932147b (diff) | |
download | linux-39be6b8b0cc84b33cd17bc570e80a25126f87c9c.tar.xz |
staging: dgap: fix possible NULL dereference
The return pointer from dgap_getword() is used in strcmp() where it is
dereferenced. But dgap_getword() can return NULL.
Lets put a check there and return 0 as error.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/dgap/dgap.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 9112dd2..20ba258 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dgap.c @@ -349,6 +349,8 @@ static int dgap_gettok(char **in) if (strstr(dgap_cword, "board")) { w = dgap_getword(in); + if (!w) + return 0; snprintf(dgap_cword, MAXCWORD, "%s", w); for (t = dgap_brdtype; t->token != 0; t++) { if (!strcmp(w, t->string)) |