From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io1-xd32.google.com (mail-io1-xd32.google.com [IPv6:2607:f8b0:4864:20::d32]) by sourceware.org (Postfix) with ESMTPS id AC1D1397300C for ; Mon, 8 Feb 2021 16:33:24 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org AC1D1397300C Received: by mail-io1-xd32.google.com with SMTP id n201so15563919iod.12 for ; Mon, 08 Feb 2021 08:33:24 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=iLr3y6scbCWAyNmRbhT8pcBmhe3DvwXFNJuw6zOh5JE=; b=l4tyyda5aG2l50EDTZnLRKg6f63cR6L+lJ5jTzoNyZSinHrYGnMzkVOwra/vuBn1Cm lxO4tn3xChvrj/m8UBrvlN/zd0adIsfr9r0lPwrWL5EcFje1D0fjaj77u61KO3yvKrWl wnro+BWNdF69hvq1ZI+CYvYuqZDGbEXunr6xynOYDKenzYDrWT1JxBjCdNPX0kTVUbkS DVXA2//vStggCvkF/FKMPrh5G6saCCVa000eQuJHOrO78JXS6zPg3v907QRxF0JlC2HG ykJ0Sz7pTgOghoMTqJOWA+Jz82GQC8S675pZN7r1hvC4i8SAYSE8hzVXLhuiNfs555CF JPjA== X-Gm-Message-State: AOAM531GntFBBRj5adNxMshlmXzUBVK9CFEBreQOAhJWPeZA2rGeAycb IdbHopz8a+4xDYvkGDaOkEdKoWrOqZ9JDqF9BCHwoF4GSko= X-Google-Smtp-Source: ABdhPJySw7f5Ov60U+SDq5VwfhaNvEO8ENM+nGXUdXkc4caeNAP13k2lj6QfXM62kUH/VmHEj7PdkM49MY/lbUo4weg= X-Received: by 2002:a6b:d80f:: with SMTP id y15mr6016287iob.175.1612802003984; Mon, 08 Feb 2021 08:33:23 -0800 (PST) MIME-Version: 1.0 References: <20210120221427.214833-1-eshandhawan51@gmail.com> In-Reply-To: <20210120221427.214833-1-eshandhawan51@gmail.com> From: Eshan Dhawan Date: Mon, 8 Feb 2021 22:03:12 +0530 Message-ID: Subject: Re: [PATCH] FTW Port for Newlib To: Newlib X-Spam-Status: No, score=-9.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, GIT_PATCH_0, HTML_MESSAGE, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: newlib@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Newlib mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Feb 2021 16:33:27 -0000 Ping !! On Thu, Jan 21, 2021 at 3:44 AM Eshan Dhawan wrote: > From: Eshan dhawan > > Signed-off-by: Eshan Dhawan > --- > newlib/libc/include/ftw.h | 66 +++++++++++++++ > newlib/libc/posix/Makefile.am | 2 +- > newlib/libc/posix/ftw.c | 33 ++++++++ > newlib/libc/posix/nftw.c | 154 ++++++++++++++++++++++++++++++++++ > 4 files changed, 254 insertions(+), 1 deletion(-) > create mode 100644 newlib/libc/include/ftw.h > create mode 100644 newlib/libc/posix/ftw.c > create mode 100644 newlib/libc/posix/nftw.c > > diff --git a/newlib/libc/include/ftw.h b/newlib/libc/include/ftw.h > new file mode 100644 > index 000000000..8de1e1e5c > --- /dev/null > +++ b/newlib/libc/include/ftw.h > @@ -0,0 +1,66 @@ > +/* > +* Copyright =C2=A9 2005-2020 Rich Felker, et al. > +* > +* Permission is hereby granted, free of charge, to any person obtaining > +* a copy of this software and associated documentation files (the > +* "Software"), to deal in the Software without restriction, including > +* without limitation the rights to use, copy, modify, merge, publish, > +* distribute, sublicense, and/or sell copies of the Software, and to > +* permit persons to whom the Software is furnished to do so, subject to > +* the following conditions: > +* > +* The above copyright notice and this permission notice shall be > +* included in all copies or substantial portions of the Software. > +* > +* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > +* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF > +* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. > +* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY > +* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, > +* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE > +* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. > +*/ > + > +#ifndef _FTW_H > +#define _FTW_H > + > +#ifdef __cplusplus > +extern "C" { > +#endif > + > +#include > +#include > + > +/* > + * Valid flags for the 3rd argument to the function that is passed as th= e > + * second argument to ftw(3) and nftw(3). Say it three times fast! > + */ > +#define FTW_F 0 /* File. */ > +#define FTW_D 1 /* Directory. */ > +#define FTW_DNR 2 /* Directory without read > permission. */ > +#define FTW_DP 3 /* Directory with subdirectories > visited. */ > +#define FTW_NS 4 /* Unknown type; stat() failed. = */ > +#define FTW_SL 5 /* Symbolic link. */ > +#define FTW_SLN 6 /* Sym link that names a > nonexistent file. */ > + > +/* > + * Flags for use as the 4th argument to nftw(3). These may be ORed > together. > + */ > +#define FTW_PHYS 0x01 /* Physical walk, don't follow sy= m > links. */ > +#define FTW_MOUNT 0x02 /* The walk does not cross a moun= t > point. */ > +#define FTW_DEPTH 0x04 /* Subdirs visited before the dir > itself. */ > +#define FTW_CHDIR 0x08 /* Change to a directory before > reading it. */ > + > +struct FTW { > + int base; > + int level; > +}; > + > +int ftw(const char *, int (*)(const char *, const struct stat *, int), > int); > +int nftw(const char *, int (*)(const char *, const struct stat *, int, > struct FTW *), int, int); > + > +#ifdef __cplusplus > +} > +#endif > + > +#endif > diff --git a/newlib/libc/posix/Makefile.am b/newlib/libc/posix/Makefile.a= m > index 6cdee1df0..5a358f782 100644 > --- a/newlib/libc/posix/Makefile.am > +++ b/newlib/libc/posix/Makefile.am > @@ -10,7 +10,7 @@ GENERAL_SOURCES =3D \ > opendir.c readdir.c readdir_r.c \ > regcomp.c regerror.c regexec.c regfree.c \ > rewinddir.c sleep.c usleep.c \ > - telldir.c > + telldir.c ftw.c nftw.c > > ELIX_2_SOURCES =3D \ > scandir.c seekdir.c > diff --git a/newlib/libc/posix/ftw.c b/newlib/libc/posix/ftw.c > new file mode 100644 > index 000000000..18cfc24cb > --- /dev/null > +++ b/newlib/libc/posix/ftw.c > @@ -0,0 +1,33 @@ > +/* > +* Copyright =C2=A9 2005-2020 Rich Felker, et al. > +* > +* Permission is hereby granted, free of charge, to any person obtaining > +* a copy of this software and associated documentation files (the > +* "Software"), to deal in the Software without restriction, including > +* without limitation the rights to use, copy, modify, merge, publish, > +* distribute, sublicense, and/or sell copies of the Software, and to > +* permit persons to whom the Software is furnished to do so, subject to > +* the following conditions: > +* > +* The above copyright notice and this permission notice shall be > +* included in all copies or substantial portions of the Software. > +* > +* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > +* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF > +* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. > +* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY > +* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, > +* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE > +* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. > +*/ > + > +#include > + > +int ftw(const char *path, int (*fn)(const char *, const struct stat *, > int), int fd_limit) > +{ > + /* The following cast assumes that calling a function with one > + * argument more than it needs behaves as expected. This is > + * actually undefined, but works on all real-world machines. */ > + return nftw(path, (int (*)())fn, fd_limit, FTW_PHYS); > +} > + > diff --git a/newlib/libc/posix/nftw.c b/newlib/libc/posix/nftw.c > new file mode 100644 > index 000000000..f4f55f341 > --- /dev/null > +++ b/newlib/libc/posix/nftw.c > @@ -0,0 +1,154 @@ > +/* > +* Copyright =C2=A9 2005-2020 Rich Felker, et al. > +* > +* Permission is hereby granted, free of charge, to any person obtaining > +* a copy of this software and associated documentation files (the > +* "Software"), to deal in the Software without restriction, including > +* without limitation the rights to use, copy, modify, merge, publish, > +* distribute, sublicense, and/or sell copies of the Software, and to > +* permit persons to whom the Software is furnished to do so, subject to > +* the following conditions: > +* > +* The above copyright notice and this permission notice shall be > +* included in all copies or substantial portions of the Software. > +* > +* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > +* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF > +* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. > +* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY > +* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, > +* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE > +* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. > +*/ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +struct history > +{ > + struct history *chain; > + dev_t dev; > + ino_t ino; > + int level; > + int base; > +}; > + > +#undef dirfd > +#define dirfd(d) (*(int *)d) > + > +static int do_nftw(char *path, int (*fn)(const char *, const struct stat > *, int, struct FTW *), int fd_limit, int flags, struct history *h) > +{ > + size_t l =3D strlen(path), j =3D l && path[l-1]=3D=3D'/' ? l-1 : = l; > + struct stat st; > + struct history new; > + int type; > + int r; > + struct FTW lev; > + > + if ((flags & FTW_PHYS) ? lstat(path, &st) : stat(path, &st) < 0) = { > + if (!(flags & FTW_PHYS) && errno=3D=3DENOENT && !lstat(pa= th, > &st)) > + type =3D FTW_SLN; > + else if (errno !=3D EACCES) return -1; > + else type =3D FTW_NS; > + } else if (S_ISDIR(st.st_mode)) { > + if (access(path, R_OK) < 0) type =3D FTW_DNR; > + else if (flags & FTW_DEPTH) type =3D FTW_DP; > + else type =3D FTW_D; > + } else if (S_ISLNK(st.st_mode)) { > + if (flags & FTW_PHYS) type =3D FTW_SL; > + else type =3D FTW_SLN; > + } else { > + type =3D FTW_F; > + } > + > + if ((flags & FTW_MOUNT) && h && st.st_dev !=3D h->dev) > + return 0; > + > + new.chain =3D h; > + new.dev =3D st.st_dev; > + new.ino =3D st.st_ino; > + new.level =3D h ? h->level+1 : 0; > + new.base =3D j+1; > + > + lev.level =3D new.level; > + if (h) { > + lev.base =3D h->base; > + } else { > + size_t k; > + for (k=3Dj; k && path[k]=3D=3D'/'; k--); > + for (; k && path[k-1]!=3D'/'; k--); > + lev.base =3D k; > + } > + > + if (!(flags & FTW_DEPTH) && (r=3Dfn(path, &st, type, &lev))) > + return r; > + > + for (; h; h =3D h->chain) > + if (h->dev =3D=3D st.st_dev && h->ino =3D=3D st.st_ino) > + return 0; > + > + if ((type =3D=3D FTW_D || type =3D=3D FTW_DP) && fd_limit) { > + DIR *d =3D opendir(path); > + if (d) { > + struct dirent *de; > + while ((de =3D readdir(d))) { > + if (de->d_name[0] =3D=3D '.' > + && (!de->d_name[1] > + || (de->d_name[1]=3D=3D'.' > + && !de->d_name[2]))) continue; > + if (strlen(de->d_name) >=3D PATH_MAX-l) { > + errno =3D ENAMETOOLONG; > + closedir(d); > + return -1; > + } > + path[j]=3D'/'; > + strcpy(path+j+1, de->d_name); > + if ((r=3Ddo_nftw(path, fn, fd_limit-1, > flags, &new))) { > + closedir(d); > + return r; > + } > + } > + closedir(d); > + } else if (errno !=3D EACCES) { > + return -1; > + } > + } > + > + path[l] =3D 0; > + if ((flags & FTW_DEPTH) && (r=3Dfn(path, &st, type, &lev))) > + return r; > + > + return 0; > +} > + > +int nftw(const char *path, int (*fn)(const char *, const struct stat *, > int, struct FTW *), int fd_limit, int flags) > +{ > + int r, cs; > + size_t l; > + char pathbuf[PATH_MAX+1]; > + > + if (fd_limit <=3D 0) return 0; > + > + l =3D strlen(path); > + if (l > PATH_MAX) { > + errno =3D ENAMETOOLONG; > + return -1; > + } > + memcpy(pathbuf, path, l+1); > + > +#ifdef _STDIO_WITH_THREAD_CANCELLATION_SUPPORT > + pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, &cs); > +#endif > + r =3D do_nftw(pathbuf, fn, fd_limit, flags, NULL); > +#ifdef _STDIO_WITH_THREAD_CANCELLATION_SUPPORT > + pthread_setcancelstate(cs, 0); > +#endif > + return r; > +} > + > -- > 2.27.0 > >