diff options
| author | Eric Andersen | 2001-05-04 22:04:24 +0000 |
|---|---|---|
| committer | Eric Andersen | 2001-05-04 22:04:24 +0000 |
| commit | 044a72d0d50bb7380601232d1388453f01fff622 (patch) | |
| tree | 18212b9982894cbb0bafa7a675e9cc7f0fa6e3c3 /libbb/concat_path_file.c | |
| parent | a0ba9f45fb0c9f3e34d5a2b25e45b7e5d5cac9a4 (diff) | |
| download | busybox-044a72d0d50bb7380601232d1388453f01fff622.zip busybox-044a72d0d50bb7380601232d1388453f01fff622.tar.gz | |
Larry suggested using concat_path_file() would be an even safer bet
for 'which'. I ageed, so I whipped this up -- which revealed a bug in
concat_path_file. It turns out that that a '/' can be appended from
either the path _or_ the filename, but only the former was checked.
-Erik
Diffstat (limited to 'libbb/concat_path_file.c')
| -rw-r--r-- | libbb/concat_path_file.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libbb/concat_path_file.c b/libbb/concat_path_file.c index d53dc0e..ce92310 100644 --- a/libbb/concat_path_file.c +++ b/libbb/concat_path_file.c @@ -15,9 +15,11 @@ extern char *concat_path_file(const char *path, const char *filename) int flg_slash = 1; l = strlen(path); - if(l>0 && path[l-1] == '/') + if (l>0 && path[l-1] == '/') flg_slash--; l += strlen(filename); + if (l>0 && filename[0] == '/') + flg_slash--; outbuf = xmalloc(l+1+flg_slash); sprintf(outbuf, (flg_slash ? "%s/%s" : "%s%s"), path, filename); return outbuf; |