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 /editors | |
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 'editors')
-rw-r--r-- | editors/vi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/editors/vi.c b/editors/vi.c index 6a93fc1..96fc965 100644 --- a/editors/vi.c +++ b/editors/vi.c @@ -19,7 +19,7 @@ */ char *vi_Version = - "$Id: vi.c,v 1.4 2001/04/16 15:46:44 andersen Exp $"; + "$Id: vi.c,v 1.5 2001/04/26 15:56:47 andersen Exp $"; /* * To compile for standalone use: @@ -1745,7 +1745,7 @@ static void colon(Byte * buf) while (isblnk(*buf)) buf++; strcpy((char *) args, (char *) buf); - if (cmd[strlen((char *) cmd) - 1] == '!') { + if (last_char_is((char *)cmd,'!')) { useforce = TRUE; cmd[strlen((char *) cmd) - 1] = '\0'; // get rid of ! } |