diff options
| author | Justus Winter <4winter@informatik.uni-hamburg.de> | 2014-02-04 11:43:41 +0100 |
|---|---|---|
| committer | Justus Winter <4winter@informatik.uni-hamburg.de> | 2014-02-04 15:21:06 +0100 |
| commit | 172f3ea324a3685a62b03edd8b03e5f09e265301 (patch) | |
| tree | 7deb429b44b2f7a7455c52f221468c78ea94a724 /ddb | |
| parent | d7b113dced48aaaba6db262db2ed420b47917896 (diff) | |
ddb: safely copy symbol names into the symtab structure
Use strncpy instead of strcpy to copy the name of a symbol into the
symtab structure. Make sure that the string is properly terminated.
Found using Coverity.
* ddb/db_sym.c (db_add_symbol_table): Use strncpy instead of strcpy,
ensure string termination.
Diffstat (limited to 'ddb')
| -rw-r--r-- | ddb/db_sym.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ddb/db_sym.c b/ddb/db_sym.c index beb4d18e..3c8caf49 100644 --- a/ddb/db_sym.c +++ b/ddb/db_sym.c @@ -75,7 +75,8 @@ db_add_symbol_table(type, start, end, name, ref, map_pointer) st->end = end; st->private = ref; st->map_pointer = (map_pointer == (char *)kernel_map)? 0: map_pointer; - strcpy(st->name, name); + strncpy(st->name, name, sizeof st->name - 1); + st->name[sizeof st->name - 1] = '\0'; db_nsymtab++; |