From 767f7a1091af3a3dcee2f7a49d0713359a81961c Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Mon, 20 Aug 2018 20:20:31 -0400 Subject: [PATCH] remove erroneous SYMLINK_MAX definition from limits.h, pathconf POSIX requires the symlink function to fail with ENAMETOOLONG if the link contents to be written exceed SYMLINK_MAX in length, but neither Linux nor our syscall wrapper code enforce this. the value 255 for SYMLINK_MAX is not meaningful and does not seem to have been motivated by anything except perhaps a wrong assumption that a definition was mandatory. it has been present (though moving through bits to top-level limits.h) since the beginning of the project history. [f]pathconf is entitled to return -1 as the limit for conf names for which there is no hard limit, with the usual POSIX note that an indefinite limit does not imply an infinite limit. in principle we might should report a limit for filesystems that impose one, but such functionality is not currently present for any of the pathconf limits, and adding it is beyond the scope of fixing the incorrect limit. --- include/limits.h | 1 - src/conf/fpathconf.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/include/limits.h b/include/limits.h index 9cb5426f..ab163a22 100644 --- a/include/limits.h +++ b/include/limits.h @@ -42,7 +42,6 @@ #define PIPE_BUF 4096 #define FILESIZEBITS 64 #define NAME_MAX 255 -#define SYMLINK_MAX 255 #define PATH_MAX 4096 #define NGROUPS_MAX 32 #define ARG_MAX 131072 diff --git a/src/conf/fpathconf.c b/src/conf/fpathconf.c index 8eb037e6..e6aca5cf 100644 --- a/src/conf/fpathconf.c +++ b/src/conf/fpathconf.c @@ -24,7 +24,7 @@ long fpathconf(int fd, int name) [_PC_REC_MIN_XFER_SIZE] = 4096, [_PC_REC_XFER_ALIGN] = 4096, [_PC_ALLOC_SIZE_MIN] = 4096, - [_PC_SYMLINK_MAX] = SYMLINK_MAX, + [_PC_SYMLINK_MAX] = -1, [_PC_2_SYMLINKS] = 1 }; if (name >= sizeof(values)/sizeof(values[0])) { -- 2.25.1