diff options
| author | Eric Andersen | 2001-04-26 15:56:47 +0000 |
|---|---|---|
| committer | Eric Andersen | 2001-04-26 15:56:47 +0000 |
| commit | c1bdffe99b6fa6fdb0f43e0dec85d8bbfcab1572 (patch) | |
| tree | 3cf5b7c0c69bd8c802c4b080b6a5e94a4a401831 /coreutils | |
| parent | 3c3277f0bda79b66b35e5e965e5340624d29a459 (diff) | |
| download | busybox-c1bdffe99b6fa6fdb0f43e0dec85d8bbfcab1572.zip busybox-c1bdffe99b6fa6fdb0f43e0dec85d8bbfcab1572.tar.gz | |
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
Diffstat (limited to 'coreutils')
| -rw-r--r-- | coreutils/cut.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/coreutils/cut.c b/coreutils/cut.c index 7e9a72e..d852ab3 100644 --- a/coreutils/cut.c +++ b/coreutils/cut.c @@ -75,7 +75,7 @@ static void decompose_list(const char *list) /* handle multi-value cases */ else if (nminus == 1) { /* handle 'N-' case */ - if (list[strlen(list) - 1] == '-') { + if (last_char_is(list,'-')) { startpos = strtol(list, &ptr, 10); } /* handle '-M' case */ |