diff options
author | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2001-04-26 15:56:47 +0000 |
---|---|---|
committer | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2001-04-26 15:56:47 +0000 |
commit | 99a496641bc6647bc66106c9cdab16562c88b0a0 (patch) | |
tree | 3cf5b7c0c69bd8c802c4b080b6a5e94a4a401831 /cut.c | |
parent | b14e3708a3e6b76781330169376184adb63f11e4 (diff) | |
download | busybox-w32-99a496641bc6647bc66106c9cdab16562c88b0a0.tar.gz busybox-w32-99a496641bc6647bc66106c9cdab16562c88b0a0.tar.bz2 busybox-w32-99a496641bc6647bc66106c9cdab16562c88b0a0.zip |
Another nice cleanup from Larry. This adds a new last_char_is() function and
uses it to avoid possible buffer underruns whn strlen is zero, and avoid the
possible space-hogging inline of strlen() in several cases.
-Erik
git-svn-id: svn://busybox.net/trunk/busybox@2450 69ca8d6d-28ef-0310-b511-8ec308f3f277
Diffstat (limited to 'cut.c')
-rw-r--r-- | cut.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -75,7 +75,7 @@ static void decompose_list(const char *list) | |||
75 | /* handle multi-value cases */ | 75 | /* handle multi-value cases */ |
76 | else if (nminus == 1) { | 76 | else if (nminus == 1) { |
77 | /* handle 'N-' case */ | 77 | /* handle 'N-' case */ |
78 | if (list[strlen(list) - 1] == '-') { | 78 | if (last_char_is(list,'-')) { |
79 | startpos = strtol(list, &ptr, 10); | 79 | startpos = strtol(list, &ptr, 10); |
80 | } | 80 | } |
81 | /* handle '-M' case */ | 81 | /* handle '-M' case */ |