mirror of
https://git.FreeBSD.org/src.git
synced 2024-10-19 02:29:40 +00:00
48aaad5fbc
fields in FTS and FTSENT structs being too narrow. In addition, the narrow types creep from there into fts.c. As a result, fts(3) consumers, e.g., find(1) or rm(1), can't handle file trees an ordinary user can create, which can have security implications. To fix the historic implementation of fts(3), OpenBSD and NetBSD have already changed <fts.h> in somewhat incompatible ways, so we are free to do so, too. This change is a superset of changes from the other BSDs with a few more improvements. It doesn't touch fts(3) functionality; it just extends integer types used by it to match modern reality and the C standard. Here are its points: o For C object sizes, use size_t unless it's 100% certain that the object will be really small. (Note that fts(3) can construct pathnames _much_ longer than PATH_MAX for its consumers.) o Avoid the short types because on modern platforms using them results in larger and slower code. Change shorts to ints as follows: - For variables than count simple, limited things like states, use plain vanilla `int' as it's the type of choice in C. - For a limited number of bit flags use `unsigned' because signed bit-wise operations are implementation-defined, i.e., unportable, in C. o For things that should be at least 64 bits wide, use long long and not int64_t, as the latter is an optional type. See FTSENT.fts_number aka FTS.fts_bignum. Extending fts_number `to satisfy future needs' is pointless because there is fts_pointer, which can be used to link to arbitrary data from an FTSENT. However, there already are fts(3) consumers that require fts_number, or fts_bignum, have at least 64 bits in it, so we must allow for them. o For the tree depth, use `long'. This is a trade-off between making this field too wide and allowing for 64-bit inode numbers and/or chain-mounted filesystems. On the one hand, `long' is almost enough for 32-bit filesystems on a 32-bit platform (our ino_t is uint32_t now). On the other hand, platforms with a 64-bit (or wider) `long' will be ready for 64-bit inode numbers, as well as for several 32-bit filesystems mounted one under another. Note that fts_level has to be signed because -1 is a magic value for it, FTS_ROOTPARENTLEVEL. o For the `nlinks' local var in fts_build(), use `long'. The logic in fts_build() requires that `nlinks' be signed, but our nlink_t currently is uint16_t. Therefore let's make the signed var wide enough to be able to represent 2^16-1 in pure C99, and even 2^32-1 on a 64-bit platform. Perhaps the logic should be changed just to use nlink_t, but it can be done later w/o breaking fts(3) ABI any more because `nlinks' is just a local var. This commit also inludes supporting stuff for the fts change: o Preserve the old versions of fts(3) functions through libc symbol versioning because the old versions appeared in all our former releases. o Bump __FreeBSD_version just in case. There is a small chance that some ill-written 3-rd party apps may fail to build or work correctly if compiled after this change. o Update the fts(3) manpage accordingly. In particular, remove references to fts_bignum, which was a FreeBSD-specific hack to work around the too narrow types of FTSENT members. Now fts_number is at least 64 bits wide (long long) and fts_bignum is an undocumented alias for fts_number kept around for compatibility reasons. According to Google Code Search, the only big consumers of fts_bignum are in our own source tree, so they can be fixed easily to use fts_number. o Mention the change in src/UPDATING. PR: bin/104458 Approved by: re (quite a while ago) Discussed with: deischen (the symbol versioning part) Reviewed by: -arch (mostly silence); das (generally OK, but we didn't agree on some types used; assuming that no objections on -arch let me to stick to my opinion) |
||
|---|---|---|
| .. | ||
| arpa | ||
| gssapi | ||
| protocols | ||
| rpc | ||
| rpcsvc | ||
| _ctype.h | ||
| a.out.h | ||
| ar.h | ||
| assert.h | ||
| bitstring.h | ||
| complex.h | ||
| cpio.h | ||
| ctype.h | ||
| db.h | ||
| dirent.h | ||
| dlfcn.h | ||
| elf-hints.h | ||
| elf.h | ||
| err.h | ||
| fmtmsg.h | ||
| fnmatch.h | ||
| fstab.h | ||
| fts.h | ||
| ftw.h | ||
| getopt.h | ||
| glob.h | ||
| grp.h | ||
| gssapi.h | ||
| hesiod.h | ||
| histedit.h | ||
| ieeefp.h | ||
| ifaddrs.h | ||
| inttypes.h | ||
| iso646.h | ||
| kenv.h | ||
| langinfo.h | ||
| libgen.h | ||
| limits.h | ||
| link.h | ||
| locale.h | ||
| Makefile | ||
| malloc_np.h | ||
| malloc.h | ||
| memory.h | ||
| monetary.h | ||
| mpool.h | ||
| mqueue.h | ||
| ndbm.h | ||
| netconfig.h | ||
| netdb.h | ||
| nl_types.h | ||
| nlist.h | ||
| nss.h | ||
| nsswitch.h | ||
| paths.h | ||
| printf.h | ||
| proc_service.h | ||
| pthread_np.h | ||
| pthread.h | ||
| pwd.h | ||
| ranlib.h | ||
| readpassphrase.h | ||
| regex.h | ||
| regexp.h | ||
| res_update.h | ||
| resolv.h | ||
| runetype.h | ||
| search.h | ||
| setjmp.h | ||
| sgtty.h | ||
| signal.h | ||
| stab.h | ||
| stdbool.h | ||
| stddef.h | ||
| stdio.h | ||
| stdlib.h | ||
| string.h | ||
| stringlist.h | ||
| strings.h | ||
| sysexits.h | ||
| tar.h | ||
| tgmath.h | ||
| time.h | ||
| timeconv.h | ||
| timers.h | ||
| ttyent.h | ||
| ulimit.h | ||
| unistd.h | ||
| utime.h | ||
| utmp.h | ||
| uuid.h | ||
| varargs.h | ||
| vis.h | ||
| wchar.h | ||
| wctype.h | ||
| wordexp.h | ||