summaryrefslogtreecommitdiff
path: root/findutils/which.c
diff options
context:
space:
mode:
authorEric Andersen2001-05-04 20:47:33 +0000
committerEric Andersen2001-05-04 20:47:33 +0000
commit7526f035f4feea48d4c686a2c5c484c4175976f9 (patch)
tree7e7a5c8f8b620362138967edc83a20fda3e0a346 /findutils/which.c
parente5ffb911e89f244c7edcfe197c71b4b5e5955310 (diff)
downloadbusybox-7526f035f4feea48d4c686a2c5c484c4175976f9.zip
busybox-7526f035f4feea48d4c686a2c5c484c4175976f9.tar.gz
Fix a buffer overflow. Which accounted for the size of the path, plus the the
'/', plus the size of the test command. It did not account for the terminating NULL, which overwrote the end of the string. -Erik
Diffstat (limited to 'findutils/which.c')
-rw-r--r--findutils/which.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/findutils/which.c b/findutils/which.c
index 1d75244..08813c1 100644
--- a/findutils/which.c
+++ b/findutils/which.c
@@ -53,7 +53,7 @@ extern int which_main(int argc, char **argv)
argv++;
found = 0;
for (i = 0; i < count; i++) {
- char buf[strlen(path_n)+1+strlen(*argv)];
+ char buf[strlen(path_n)+strlen(*argv)+2];
strcpy (buf, path_n);
strcat (buf, "/");
strcat (buf, *argv);