diff options
| author | Yann E. MORIN <yann.morin.1998@free.fr> | 2013-07-16 20:32:33 +0200 |
|---|---|---|
| committer | Yann E. MORIN <yann.morin.1998@free.fr> | 2013-07-16 20:36:18 +0200 |
| commit | 1407f97aeda5720d6327d69f6058537c0fd469e3 (patch) | |
| tree | e16cffe6b08d38c18db57ef46e96c757c0690f75 /scripts | |
| parent | 803b3519880f255563d8590c1f5870398b9a4ea0 (diff) | |
| download | lwn-1407f97aeda5720d6327d69f6058537c0fd469e3.tar.gz lwn-1407f97aeda5720d6327d69f6058537c0fd469e3.zip | |
kconfig: don't allocate n+1 elements in temporary array
The temporary array that stores the search results is not NULL-terminated,
so there is no reason to allocate n+1 elements.
Reported-by: Jean Delvare <jdelvare@suse.de>
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Reviewed-by: Jean Delvare <jdelvare@suse.de>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/kconfig/symbol.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c index b664d6ed515f..08d4401e646d 100644 --- a/scripts/kconfig/symbol.c +++ b/scripts/kconfig/symbol.c @@ -1010,7 +1010,7 @@ struct symbol **sym_re_search(const char *pattern) continue; if (regexec(&re, sym->name, 1, match, 0)) continue; - if (cnt + 1 >= size) { + if (cnt >= size) { void *tmp; size += 16; tmp = realloc(sym_match_arr, size * sizeof(struct sym_match *)); |