fix regression that negated some mips syscall error returns
authorRich Felker <dalias@aerifal.cx>
Sun, 20 Jul 2014 16:38:26 +0000 (12:38 -0400)
committerRich Felker <dalias@aerifal.cx>
Mon, 28 Jul 2014 04:28:01 +0000 (00:28 -0400)
due to what was essentially a copy and paste error, the changes made
in commit f61be1f875a2758509d6e9e2cf6f1d9603b28b65 caused syscalls
with 5 or 6 arguments (and syscalls with 2, 3, or 4 arguments when
compiled with clang compatibility) to negate the returned error code a
second time, breaking errno reporting.

(cherry picked from commit 1312930f9bdea47006a8a8c8509c0bed5cf69e85)

arch/mips/syscall_arch.h

index 1aa2c812a1e48455f584cbd222d6517e9cc12294..0f89a1c1e31c60bf14862c6f83c061df9a2966cf 100644 (file)
@@ -116,7 +116,7 @@ static inline long __syscall1(long n, long a)
 static inline long __syscall2(long n, long a, long b)
 {
        long r2 = (__syscall)(n, a, b);
-       if (r2 > -4096UL) return -r2;
+       if (r2 > -4096UL) return r2;
        if (n == SYS_stat64 || n == SYS_fstat64 || n == SYS_lstat64) __stat_fix(b);
        return r2;
 }
@@ -124,7 +124,7 @@ static inline long __syscall2(long n, long a, long b)
 static inline long __syscall3(long n, long a, long b, long c)
 {
        long r2 = (__syscall)(n, a, b, c);
-       if (r2 > -4096UL) return -r2;
+       if (r2 > -4096UL) return r2;
        if (n == SYS_stat64 || n == SYS_fstat64 || n == SYS_lstat64) __stat_fix(b);
        return r2;
 }
@@ -132,7 +132,7 @@ static inline long __syscall3(long n, long a, long b, long c)
 static inline long __syscall4(long n, long a, long b, long c, long d)
 {
        long r2 = (__syscall)(n, a, b, c, d);
-       if (r2 > -4096UL) return -r2;
+       if (r2 > -4096UL) return r2;
        if (n == SYS_stat64 || n == SYS_fstat64 || n == SYS_lstat64) __stat_fix(b);
        if (n == SYS_fstatat) __stat_fix(c);
        return r2;
@@ -143,7 +143,7 @@ static inline long __syscall4(long n, long a, long b, long c, long d)
 static inline long __syscall5(long n, long a, long b, long c, long d, long e)
 {
        long r2 = (__syscall)(n, a, b, c, d, e);
-       if (r2 > -4096UL) return -r2;
+       if (r2 > -4096UL) return r2;
        if (n == SYS_stat64 || n == SYS_fstat64 || n == SYS_lstat64) __stat_fix(b);
        if (n == SYS_fstatat) __stat_fix(c);
        return r2;
@@ -152,7 +152,7 @@ static inline long __syscall5(long n, long a, long b, long c, long d, long e)
 static inline long __syscall6(long n, long a, long b, long c, long d, long e, long f)
 {
        long r2 = (__syscall)(n, a, b, c, d, e, f);
-       if (r2 > -4096UL) return -r2;
+       if (r2 > -4096UL) return r2;
        if (n == SYS_stat64 || n == SYS_fstat64 || n == SYS_lstat64) __stat_fix(b);
        if (n == SYS_fstatat) __stat_fix(c);
        return r2;