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 /vi.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 'vi.c')
-rw-r--r-- | vi.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -19,7 +19,7 @@ | |||
19 | */ | 19 | */ |
20 | 20 | ||
21 | char *vi_Version = | 21 | char *vi_Version = |
22 | "$Id: vi.c,v 1.4 2001/04/16 15:46:44 andersen Exp $"; | 22 | "$Id: vi.c,v 1.5 2001/04/26 15:56:47 andersen Exp $"; |
23 | 23 | ||
24 | /* | 24 | /* |
25 | * To compile for standalone use: | 25 | * To compile for standalone use: |
@@ -1745,7 +1745,7 @@ static void colon(Byte * buf) | |||
1745 | while (isblnk(*buf)) | 1745 | while (isblnk(*buf)) |
1746 | buf++; | 1746 | buf++; |
1747 | strcpy((char *) args, (char *) buf); | 1747 | strcpy((char *) args, (char *) buf); |
1748 | if (cmd[strlen((char *) cmd) - 1] == '!') { | 1748 | if (last_char_is((char *)cmd,'!')) { |
1749 | useforce = TRUE; | 1749 | useforce = TRUE; |
1750 | cmd[strlen((char *) cmd) - 1] = '\0'; // get rid of ! | 1750 | cmd[strlen((char *) cmd) - 1] = '\0'; // get rid of ! |
1751 | } | 1751 | } |