Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1287048 > unrolled thread

[PATCH] sh64: fix __NR_fgetxattr

Started by"Dmitry V. Levin" <ldv@altlinux.org>
First post2015-12-09 04:20 +0100
Last post2015-12-09 09:30 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] sh64: fix __NR_fgetxattr "Dmitry V. Levin" <ldv@altlinux.org> - 2015-12-09 04:20 +0100
    Re: [PATCH] sh64: fix __NR_fgetxattr Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-09 09:30 +0100

#1287048 — [PATCH] sh64: fix __NR_fgetxattr

From"Dmitry V. Levin" <ldv@altlinux.org>
Date2015-12-09 04:20 +0100
Subject[PATCH] sh64: fix __NR_fgetxattr
Message-ID<qDDsl-2Lw-9@gated-at.bofh.it>
According to arch/sh/kernel/syscalls_64.S and common sense,
__NR_fgetxattr has to be defined to 259, but it doesn't.
Instead, it's defined to 269, which is of course used
by another syscall, __NR_sched_setaffinity in this case.

This bug was found by strace test suite.

Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Cc: stable@vger.kernel.org
---
 arch/sh/include/uapi/asm/unistd_64.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/sh/include/uapi/asm/unistd_64.h b/arch/sh/include/uapi/asm/unistd_64.h
index e6820c8..47ebd5b 100644
--- a/arch/sh/include/uapi/asm/unistd_64.h
+++ b/arch/sh/include/uapi/asm/unistd_64.h
@@ -278,7 +278,7 @@
 #define __NR_fsetxattr		256
 #define __NR_getxattr		257
 #define __NR_lgetxattr		258
-#define __NR_fgetxattr		269
+#define __NR_fgetxattr		259
 #define __NR_listxattr		260
 #define __NR_llistxattr		261
 #define __NR_flistxattr		262
-- 
ldv
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1287188

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2015-12-09 09:30 +0100
Message-ID<qDIim-5Rn-9@gated-at.bofh.it>
In reply to#1287048
CC akpm

On Wed, Dec 9, 2015 at 4:16 AM, Dmitry V. Levin <ldv@altlinux.org> wrote:
> According to arch/sh/kernel/syscalls_64.S and common sense,
> __NR_fgetxattr has to be defined to 259, but it doesn't.
> Instead, it's defined to 269, which is of course used
> by another syscall, __NR_sched_setaffinity in this case.
>
> This bug was found by strace test suite.
>
> Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
> Cc: stable@vger.kernel.org

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

> ---
>  arch/sh/include/uapi/asm/unistd_64.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/sh/include/uapi/asm/unistd_64.h b/arch/sh/include/uapi/asm/unistd_64.h
> index e6820c8..47ebd5b 100644
> --- a/arch/sh/include/uapi/asm/unistd_64.h
> +++ b/arch/sh/include/uapi/asm/unistd_64.h
> @@ -278,7 +278,7 @@
>  #define __NR_fsetxattr         256
>  #define __NR_getxattr          257
>  #define __NR_lgetxattr         258
> -#define __NR_fgetxattr         269
> +#define __NR_fgetxattr         259
>  #define __NR_listxattr         260
>  #define __NR_llistxattr                261
>  #define __NR_flistxattr                262

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web