Move some chown / lchown / fchown definitions to syscalls.list (bug 14138).
authorJoseph Myers <joseph@codesourcery.com>
Mon, 6 Oct 2014 22:30:54 +0000 (22:30 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Mon, 6 Oct 2014 22:30:54 +0000 (22:30 +0000)
Continuing the move of syscall definitions to syscalls.list, where the
removal of support for old kernel versions has made this possible,
this patch moves various definitions of chown, lchown and fchown.

In most cases the need for special syscalls.list entries (rather than
existing generic ones) is because these architectures use chown32,
lchown32 and fchown32 as syscall names.  Some architectures also have
symbol versioning compatibility for older versions of chown having
been equivalent to lchown.

In the case of powerpc, chown.c (providing the chown@@GLIBC_2.1
default version) is replaced by a syscalls.list entry (for powerpc32;
powerpc64 has no need for this because of its more recent minimum
symbol version, so can just use the entry in
sysdeps/unix/syscalls.list), but lchown.S is left as-is because it
provides the compat version of chown as an actual alias for __lchown,
which is not yet supported by syscalls.list.  This file can be removed
once such aliases are supported in syscalls.list.

[BZ #14138]
* sysdeps/unix/sysv/linux/arm/fchown.c: Remove file.
* sysdeps/unix/sysv/linux/arm/lchown.c: Likewise.
* sysdeps/unix/sysv/linux/m68k/fchown.c: Likewise.
* sysdeps/unix/sysv/linux/m68k/lchown.c: Likewise.
* sysdeps/unix/sysv/linux/powerpc/chown.c: Likewise.
* sysdeps/unix/sysv/linux/arm/syscalls.list (lchown): Add syscall.
(fchown): Likewise.
* sysdeps/unix/sysv/linux/m68k/syscalls.list (lchown): Likewise.
(fchown): Likewise.
* sysdeps/unix/sysv/linux/powerpc/powerpc32/syscalls.list (chown):
Likewise.

ChangeLog
sysdeps/unix/sysv/linux/arm/fchown.c [deleted file]
sysdeps/unix/sysv/linux/arm/lchown.c [deleted file]
sysdeps/unix/sysv/linux/arm/syscalls.list
sysdeps/unix/sysv/linux/m68k/fchown.c [deleted file]
sysdeps/unix/sysv/linux/m68k/lchown.c [deleted file]
sysdeps/unix/sysv/linux/m68k/syscalls.list
sysdeps/unix/sysv/linux/powerpc/chown.c [deleted file]
sysdeps/unix/sysv/linux/powerpc/powerpc32/syscalls.list

index 0d37530..5a89797 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2014-10-06  Joseph Myers  <joseph@codesourcery.com>
+
+       [BZ #14138]
+       * sysdeps/unix/sysv/linux/arm/fchown.c: Remove file.
+       * sysdeps/unix/sysv/linux/arm/lchown.c: Likewise.
+       * sysdeps/unix/sysv/linux/m68k/fchown.c: Likewise.
+       * sysdeps/unix/sysv/linux/m68k/lchown.c: Likewise.
+       * sysdeps/unix/sysv/linux/powerpc/chown.c: Likewise.
+       * sysdeps/unix/sysv/linux/arm/syscalls.list (lchown): Add syscall.
+       (fchown): Likewise.
+       * sysdeps/unix/sysv/linux/m68k/syscalls.list (lchown): Likewise.
+       (fchown): Likewise.
+       * sysdeps/unix/sysv/linux/powerpc/powerpc32/syscalls.list (chown):
+       Likewise.
+
 2014-10-06  Adhemerval Zanella  <azanella@linux.vnet.ibm.com>
 
        * sysdeps/unix/sysv/linux/powerpc/lowlevellock.h: Remove file.
diff --git a/sysdeps/unix/sysv/linux/arm/fchown.c b/sysdeps/unix/sysv/linux/arm/fchown.c
deleted file mode 100644 (file)
index 3a69ecc..0000000
+++ /dev/null
@@ -1 +0,0 @@
-#include <sysdeps/unix/sysv/linux/i386/fchown.c>
diff --git a/sysdeps/unix/sysv/linux/arm/lchown.c b/sysdeps/unix/sysv/linux/arm/lchown.c
deleted file mode 100644 (file)
index c89de99..0000000
+++ /dev/null
@@ -1 +0,0 @@
-#include <sysdeps/unix/sysv/linux/i386/lchown.c>
index ba8cdd7..f2133bc 100644 (file)
@@ -1,6 +1,8 @@
 # File name    Caller  Syscall name    # args  Strong name     Weak names
 
 chown          -       chown32         i:sii   __chown         chown
+lchown         -       lchown32        i:sii   __lchown        lchown
+fchown         -       fchown32        i:iii   __fchown        fchown
 
 prlimit64      EXTRA   prlimit64       i:iipp  prlimit64
 
diff --git a/sysdeps/unix/sysv/linux/m68k/fchown.c b/sysdeps/unix/sysv/linux/m68k/fchown.c
deleted file mode 100644 (file)
index 3a69ecc..0000000
+++ /dev/null
@@ -1 +0,0 @@
-#include <sysdeps/unix/sysv/linux/i386/fchown.c>
diff --git a/sysdeps/unix/sysv/linux/m68k/lchown.c b/sysdeps/unix/sysv/linux/m68k/lchown.c
deleted file mode 100644 (file)
index c89de99..0000000
+++ /dev/null
@@ -1 +0,0 @@
-#include <sysdeps/unix/sysv/linux/i386/lchown.c>
index aee1011..7896b87 100644 (file)
@@ -1,6 +1,8 @@
 # File name    Caller  Syscall name    Args    Strong name     Weak names
 
 chown          -       chown32         i:sii   __chown         chown
+lchown         -       lchown32        i:sii   __lchown        lchown
+fchown         -       fchown32        i:iii   __fchown        fchown
 waitpid                -       waitpid         Ci:ipi  __waitpid       waitpid __libc_waitpid
 
 cacheflush     EXTRA   cacheflush      i:iiii  __cacheflush    cacheflush
diff --git a/sysdeps/unix/sysv/linux/powerpc/chown.c b/sysdeps/unix/sysv/linux/powerpc/chown.c
deleted file mode 100644 (file)
index 59c6147..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-/* chown() compatibility.
-   Copyright (C) 1998-2014 Free Software Foundation, Inc.
-   This file is part of the GNU C Library.
-
-   The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Lesser General Public
-   License as published by the Free Software Foundation; either
-   version 2.1 of the License, or (at your option) any later version.
-
-   The GNU C Library is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Lesser General Public License for more details.
-
-   You should have received a copy of the GNU Lesser General Public
-   License along with the GNU C Library; if not, see
-   <http://www.gnu.org/licenses/>.  */
-
-#include <errno.h>
-#include <string.h>
-#include <unistd.h>
-#include <limits.h>
-#include <sysdep.h>
-#include <stdlib.h>
-
-/* Consider moving to syscalls.list.  */
-
-/*
-  In Linux 2.1.x the chown functions have been changed.  A new function lchown
-  was introduced.  The new chown now follows symlinks - the old chown and the
-  new lchown do not follow symlinks.
-  This file emulates chown() under the old kernels.
-*/
-
-int
-__chown (const char *file, uid_t owner, gid_t group)
-{
-  return INLINE_SYSCALL (chown, 3, file, owner, group);
-}
-libc_hidden_def (__chown)
-
-#include <shlib-compat.h>
-versioned_symbol (libc, __chown, chown, GLIBC_2_1);
index fb6535a..25487cc 100644 (file)
@@ -1,5 +1,7 @@
 # File name    Caller  Syscall name    # args  Strong name     Weak names
 
+chown          -       chown           i:sii   __chown         chown@@GLIBC_2.1
+
 # System calls with wrappers.
 oldgetrlimit   EXTRA   getrlimit       i:ip    __old_getrlimit getrlimit@GLIBC_2.0
 oldsetrlimit   EXTRA   setrlimit       i:ip    __old_setrlimit setrlimit@GLIBC_2.0