fix for broken kernel side RLIM_INFINITY on mips
authorSzabolcs Nagy <nsz@port70.net>
Fri, 30 May 2014 06:47:35 +0000 (08:47 +0200)
committerRich Felker <dalias@aerifal.cx>
Fri, 6 Jun 2014 21:57:36 +0000 (17:57 -0400)
On 32 bit mips the kernel uses -1UL/2 to mark RLIM_INFINITY (and
this is the definition in the userspace api), but since it is in
the middle of the valid range of limits and limits are often
compared with relational operators, various kernel side logic is
broken if larger than -1UL/2 limits are used. So we truncate the
limits to -1UL/2 in get/setrlimit and prlimit.

Even if the kernel side logic consistently treated -1UL/2 as greater
than any other limit value, there wouldn't be any clean workaround
that allowed using large limits:
* using -1UL/2 as RLIM_INFINITY in userspace would mean different
infinity value for get/setrlimt and prlimit (where infinity is always
-1ULL) and userspace logic could break easily (just like the kernel
is broken now) and more special case code would be needed for mips.
* translating -1UL/2 kernel side value to -1ULL in userspace would
mean that -1UL/2 limit cannot be set (eg. -1UL/2+1 had to be passed
to the kernel instead).

(cherry picked from commit 8258014fd1e34e942a549c88c7e022a00445c352)

arch/mips/syscall_arch.h
src/internal/syscall.h
src/linux/prlimit.c
src/misc/getrlimit.c
src/misc/setrlimit.c

index c52976ebba86b35c9731e2d969a2e20b5a74669d..e62c33985486abf52ea662cff157db73c6647374 100644 (file)
@@ -5,6 +5,8 @@
 
 long (__syscall)(long, ...);
 
+#define SYSCALL_RLIM_INFINITY (-1UL/2)
+
 #ifndef __clang__
 
 #define __asm_syscall(...) do { \
index d0fe107e726c297f46969240a4ee6b5e23b75cc5..1c73d0dec47bc7ac126496c9535de28e0633cd3d 100644 (file)
@@ -4,6 +4,10 @@
 #include <sys/syscall.h>
 #include "syscall_arch.h"
 
+#ifndef SYSCALL_RLIM_INFINITY
+#define SYSCALL_RLIM_INFINITY (~0ULL)
+#endif
+
 #ifndef __scc
 #define __scc(X) ((long) (X))
 typedef long syscall_arg_t;
index d1639ccaf1c0cbeb742e7d214ee9f6e256d48335..0fe28e10664e9047b1ca1a75bdab78c3e0407723 100644 (file)
@@ -3,9 +3,24 @@
 #include "syscall.h"
 #include "libc.h"
 
+#define FIX(x) do{ if ((x)>=SYSCALL_RLIM_INFINITY) (x)=RLIM_INFINITY; }while(0)
+
 int prlimit(pid_t pid, int resource, const struct rlimit *new_limit, struct rlimit *old_limit)
 {
-       return syscall(SYS_prlimit64, pid, resource, new_limit, old_limit);
+       struct rlimit tmp;
+       int r;
+       if (new_limit && SYSCALL_RLIM_INFINITY != RLIM_INFINITY) {
+               tmp = *new_limit;
+               FIX(tmp.rlim_cur);
+               FIX(tmp.rlim_max);
+               new_limit = &tmp;
+       }
+       r = syscall(SYS_prlimit64, pid, resource, new_limit, old_limit);
+       if (!r && old_limit && SYSCALL_RLIM_INFINITY != RLIM_INFINITY) {
+               FIX(old_limit->rlim_cur);
+               FIX(old_limit->rlim_max);
+       }
+       return r;
 }
 
 #undef prlimit64
index b7bbd06253a55ea6e748c2d69e3d0fa6412a15ef..b073677f1a77a7a6062813cba43da1c44528798e 100644 (file)
@@ -3,16 +3,24 @@
 #include "syscall.h"
 #include "libc.h"
 
+#define FIX(x) do{ if ((x)>=SYSCALL_RLIM_INFINITY) (x)=RLIM_INFINITY; }while(0)
+
 int getrlimit(int resource, struct rlimit *rlim)
 {
        unsigned long k_rlim[2];
        int ret = syscall(SYS_prlimit64, 0, resource, 0, rlim);
+       if (!ret) {
+               FIX(rlim->rlim_cur);
+               FIX(rlim->rlim_max);
+       }
        if (!ret || errno != ENOSYS)
                return ret;
        if (syscall(SYS_getrlimit, resource, k_rlim) < 0)
                return -1;
        rlim->rlim_cur = k_rlim[0] == -1UL ? RLIM_INFINITY : k_rlim[0];
        rlim->rlim_max = k_rlim[1] == -1UL ? RLIM_INFINITY : k_rlim[1];
+       FIX(rlim->rlim_cur);
+       FIX(rlim->rlim_max);
        return 0;
 }
 
index ddc13e98bf9e8272c17fe4b074ac0a86cc7cc8d5..8a1b8cc63d3428cda6654b23742dadbe7f740678 100644 (file)
@@ -4,14 +4,22 @@
 #include "libc.h"
 
 #define MIN(a, b) ((a)<(b) ? (a) : (b))
+#define FIX(x) do{ if ((x)>=SYSCALL_RLIM_INFINITY) (x)=RLIM_INFINITY; }while(0)
 
 int __setrlimit(int resource, const struct rlimit *rlim)
 {
        unsigned long k_rlim[2];
+       struct rlimit tmp;
+       if (SYSCALL_RLIM_INFINITY != RLIM_INFINITY) {
+               tmp = *rlim;
+               FIX(tmp.rlim_cur);
+               FIX(tmp.rlim_max);
+               rlim = &tmp;
+       }
        int ret = __syscall(SYS_prlimit64, 0, resource, rlim, 0);
        if (ret != -ENOSYS) return ret;
-       k_rlim[0] = MIN(rlim->rlim_cur, -1UL);
-       k_rlim[1] = MIN(rlim->rlim_max, -1UL);
+       k_rlim[0] = MIN(rlim->rlim_cur, MIN(-1UL, SYSCALL_RLIM_INFINITY));
+       k_rlim[1] = MIN(rlim->rlim_max, MIN(-1UL, SYSCALL_RLIM_INFINITY));
        return __syscall(SYS_setrlimit, resource, k_rlim);
 }