diff options
author | Denys Vlasenko | 2011-01-11 13:07:22 +0100 |
---|---|---|
committer | Denys Vlasenko | 2011-01-11 13:07:22 +0100 |
commit | 5a163b26451c591187482f99659e5fe639a0616a (patch) | |
tree | fbaef720c39bbfad1f38531ac15e2959c9716abc | |
parent | 0f44c08560a5287fb927c15a3cce54302a0b80e5 (diff) | |
download | busybox-5a163b26451c591187482f99659e5fe639a0616a.zip busybox-5a163b26451c591187482f99659e5fe639a0616a.tar.gz |
lsmod: fox unsafe usage of strlen() - 1
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
-rw-r--r-- | modutils/lsmod.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/modutils/lsmod.c b/modutils/lsmod.c index ab7c11f..d7e1668 100644 --- a/modutils/lsmod.c +++ b/modutils/lsmod.c @@ -87,7 +87,8 @@ int lsmod_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM) // N.B. token[3] is either '-' (module is not used by others) // or comma-separated list ended by comma // so trimming the trailing char is just what we need! - token[3][strlen(token[3])-1] = '\0'; + if (token[3][0]) + token[3][strlen(token[3]) - 1] = '\0'; # if ENABLE_UNICODE_SUPPORT { uni_stat_t uni_stat; |