summaryrefslogtreecommitdiff
path: root/vi.c
diff options
context:
space:
mode:
authorEric Andersen2001-04-26 15:56:47 +0000
committerEric Andersen2001-04-26 15:56:47 +0000
commitc1bdffe99b6fa6fdb0f43e0dec85d8bbfcab1572 (patch)
tree3cf5b7c0c69bd8c802c4b080b6a5e94a4a401831 /vi.c
parent3c3277f0bda79b66b35e5e965e5340624d29a459 (diff)
downloadbusybox-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 'vi.c')
-rw-r--r--vi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/vi.c b/vi.c
index 6a93fc1..96fc965 100644
--- a/vi.c
+++ b/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 !
}