summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2011-01-11 13:07:22 +0100
committerDenys Vlasenko2011-01-11 13:07:22 +0100
commit5a163b26451c591187482f99659e5fe639a0616a (patch)
treefbaef720c39bbfad1f38531ac15e2959c9716abc
parent0f44c08560a5287fb927c15a3cce54302a0b80e5 (diff)
downloadbusybox-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.c3
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;