diff options
author | Jim Meyering <jim@meyering.net> | 1999-03-13 16:09:00 +0000 |
---|---|---|
committer | Jim Meyering <jim@meyering.net> | 1999-03-13 16:09:00 +0000 |
commit | 2bc2cc2f5421c8557fcd2a537316c251fa8bccd7 (patch) | |
tree | 11ca40f721436d0cc785b16589482064cd5b370b | |
parent | 1905d5266b53cb521747c44c44d1c247fa6b0733 (diff) | |
download | coreutils-2bc2cc2f5421c8557fcd2a537316c251fa8bccd7.tar.xz |
(base_name): If NAME is all slashes, return `/' (in
conformance with the single unix spec). Reported by Peter Moulder.
-rw-r--r-- | lib/basename.c | 24 |
1 files changed, 18 insertions, 6 deletions
diff --git a/lib/basename.c b/lib/basename.c index f4f38f9a6..6038e52aa 100644 --- a/lib/basename.c +++ b/lib/basename.c @@ -1,5 +1,5 @@ /* basename.c -- return the last element in a path - Copyright (C) 1990, 1998 Free Software Foundation, Inc. + Copyright (C) 1990, 1998, 1999 Free Software Foundation, Inc. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -29,16 +29,28 @@ /* In general, we can't use the builtin `basename' function if available, since it has different meanings in different environments. - In some environments the builtin `basename' modifies its argument. */ + In some environments the builtin `basename' modifies its argument. + If NAME is all slashes, be sure to return `/'. + FIXME: what if NAME is the empty string? */ char * base_name (char const *name) { char const *base = name += FILESYSTEM_PREFIX_LEN (name); - - for (; *name; name++) - if (ISSLASH (*name)) - base = name + 1; + int all_slashes = 1; + char const *p; + + for (p = name; *p; p++) + { + if (ISSLASH (*p)) + base = p + 1; + else + all_slashes = 0; + } + + /* If NAME is all slashes, arrange to return `/'. */ + if (*base == '\0' && ISSLASH (*name) && all_slashes) + --base; return (char *) base; } |