From 25f227b924443a71ee2e15eff342871864b05d01 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 5 Jan 2001 18:10:10 +0000 Subject: [PATCH] Update. * io/ftw.c (ftw_dir): Don't add a second slash at the beginning for searches from the root directory. Reported by loris [PR libc/1991]. * io/Makefile (tests): Add bug-ftw1. * io/bug-ftw1.c: New file. --- ChangeLog | 6 ++++ io/Makefile | 4 +-- io/bug-ftw1.c | 78 +++++++++++++++++++++++++++++++++++++++++++++++++++ io/ftw.c | 7 +++-- 4 files changed, 91 insertions(+), 4 deletions(-) create mode 100644 io/bug-ftw1.c diff --git a/ChangeLog b/ChangeLog index 0bf61dd9d2..7de622f414 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2001-01-05 Ulrich Drepper + * io/ftw.c (ftw_dir): Don't add a second slash at the beginning + for searches from the root directory. + Reported by loris [PR libc/1991]. + * io/Makefile (tests): Add bug-ftw1. + * io/bug-ftw1.c: New file. + * elf/dl-support.c (non_dynamic_init): Don't define DL_FIND_AUXV if it is not available. Instead use it only if it is available. * sysdeps/unix/sysv/linux/ldsodefs.h: New file. diff --git a/io/Makefile b/io/Makefile index bec1549da5..b1bc32d246 100644 --- a/io/Makefile +++ b/io/Makefile @@ -1,4 +1,4 @@ -# Copyright (C) 1992-1999, 2000 Free Software Foundation, Inc. +# Copyright (C) 1992-1999, 2000, 2001 Free Software Foundation, Inc. # This file is part of the GNU C Library. # The GNU C Library is free software; you can redistribute it and/or @@ -56,7 +56,7 @@ static-only-routines = stat fstat lstat mknod stat64 fstat64 lstat64 others := pwd test-srcs := ftwtest tests := test-utime test-stat test-stat2 test-lfs tst-getcwd \ - tst-fcntl + tst-fcntl bug-ftw1 distribute := ftwtest-sh diff --git a/io/bug-ftw1.c b/io/bug-ftw1.c new file mode 100644 index 0000000000..082f6fc83c --- /dev/null +++ b/io/bug-ftw1.c @@ -0,0 +1,78 @@ +/* Test for ftw function searching in root directory. + Copyright (C) 2001 Free Software Foundation, Inc. + This file is part of the GNU C Library. + Contributed by Ulrich Drepper , 2001. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Library General Public License as + published by the Free Software Foundation; either version 2 of the + License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Library General Public License for more details. + + You should have received a copy of the GNU Library General Public + License along with the GNU C Library; see the file COPYING.LIB. If not, + write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330, + Boston, MA 02111-1307, USA. */ + +#include +#include +#include +#include + + +int result; +int cnt; +int sawroot; + + +static int +callback (const char *fname, const struct stat *st, int flag) +{ + if (++cnt >= 10) + return 1; + + printf ("%d: \"%s\" -> ", cnt, fname); + if (strcmp (fname, "/") == 0 && sawroot) + { + puts ("root directory reported twice"); + result = 1; + } + else if (fname[0] != '/') + { + puts ("missing '/' as first character"); + result = 1; + } + else if (fname[1] == '/') + { + puts ("double '/' at beginning"); + result = 1; + } + else + { + puts ("OK"); + sawroot |= strcmp (fname, "/") == 0; + } + + return 0; +} + + +int +main (void) +{ + mtrace (); + + ftw ("/", callback, 10); + + if (! sawroot) + { + puts ("root directory wasn't reported"); + result = 1; + } + + return result; +} diff --git a/io/ftw.c b/io/ftw.c index 01abb63782..18b60519f0 100644 --- a/io/ftw.c +++ b/io/ftw.c @@ -1,5 +1,5 @@ /* File tree walker functions. - Copyright (C) 1996, 1997, 1998, 1999, 2000 Free Software Foundation, Inc. + Copyright (C) 1996,1997,1998,1999,2000,2001 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 1996. @@ -396,7 +396,10 @@ ftw_dir (struct ftw_data *data, struct STAT *st) /* Next, update the `struct FTW' information. */ ++data->ftw.level; startp = strchr (data->dirbuf, '\0'); - *startp++ = '/'; + /* There always must be a directory name. */ + assert (startp != data->dirbuf); + if (startp != data->dirbuf + 1) + *startp++ = '/'; data->ftw.base = startp - data->dirbuf; while (dir.stream != NULL && (d = READDIR (dir.stream)) != NULL)