2005-07-01 Daniel Drake <dsd@gentoo.org>
authorDaniel Drake <dsd@mono-cvs.ximian.com>
Fri, 1 Jul 2005 21:05:50 +0000 (21:05 -0000)
committerDaniel Drake <dsd@mono-cvs.ximian.com>
Fri, 1 Jul 2005 21:05:50 +0000 (21:05 -0000)
* sys-xattr.c: Support FreeBSD's extattr implementation. Slight API rework
  to provide transparency between Linux and FreeBSD EA's.
* map-icalls.h: Update as above

svn path=/trunk/mono/; revision=46863

support/ChangeLog
support/map-icalls.h
support/sys-xattr.c

index 9621e92f4cb19a746d9453f5d211569004c76455..6d6ffe958ae1cf3a6d2fb99f88efbc25f9589427 100644 (file)
@@ -1,3 +1,9 @@
+2005-07-01  Daniel Drake  <dsd@gentoo.org>
+
+       * sys-xattr.c: Support FreeBSD's extattr implementation. Slight API rework
+         to provide transparency between Linux and FreeBSD EA's.
+       * map-icalls.h: Update as above
+
 2005-06-30 Gonzalo Paniagua Javier <gonzalo@ximian.com>
 
        * supportw.c: HeapDestroy.
index 1b67b11665c4f3fdbc1a0c4538918c35783da676..d7d45b198e7acf5880dacc8f0d9cd3f9507a2724 100644 (file)
@@ -108,6 +108,7 @@ int Mono_Posix_Syscall_fgetpwent (void* stream, struct Mono_Posix_Syscall__Passw
 gint64 Mono_Posix_Syscall_fgetxattr (int fd, const char* name, void* value, guint64 size);
 gint64 Mono_Posix_Syscall_flistxattr (int fd, void* list, guint64 size);
 gint64 Mono_Posix_Syscall_fpathconf (int filedes, int name);
+int Mono_Posix_Syscall_fremovexattr (int fd, const char* name);
 int Mono_Posix_Syscall_fsetxattr (int fd, const char* name, void* value, guint64 size, int flags);
 int Mono_Posix_Syscall_fstat (int filedes, struct Mono_Posix_Stat* buf);
 int Mono_Posix_Syscall_fstatvfs (int fd, struct Mono_Posix_Statvfs* buf);
@@ -136,6 +137,7 @@ gint64 Mono_Posix_Syscall_lgetxattr (const char* path, const char* name, void* v
 gint64 Mono_Posix_Syscall_listxattr (const char* path, void* list, guint64 size);
 gint64 Mono_Posix_Syscall_llistxattr (const char* path, void* list, guint64 size);
 int Mono_Posix_Syscall_lockf (int fd, int cmd, gint64 len);
+int Mono_Posix_Syscall_lremovexattr (const char* path, const char* name);
 gint64 Mono_Posix_Syscall_lseek (int fd, gint64 offset, int whence);
 int Mono_Posix_Syscall_lsetxattr (const char* path, const char* name, void* value, guint64 size, int flags);
 int Mono_Posix_Syscall_lstat (const char* file_name, struct Mono_Posix_Stat* buf);
@@ -162,6 +164,7 @@ int Mono_Posix_Syscall_readdir (void* dir, struct Mono_Posix_Syscall__Dirent* de
 int Mono_Posix_Syscall_readdir_r (void* dirp, struct Mono_Posix_Syscall__Dirent* entry, void** result);
 int Mono_Posix_Syscall_readlink (const char* path, char* buf, guint64 bufsiz);
 int Mono_Posix_Syscall_remap_file_pages (void* start, guint64 size, int prot, gint64 pgoff, int flags);
+int Mono_Posix_Syscall_removexattr (const char* path, const char* name);
 int Mono_Posix_Syscall_seekdir (void* dir, gint64 offset);
 gint64 Mono_Posix_Syscall_sendfile (int out_fd, int in_fd, gint64* offset, guint64 count);
 int Mono_Posix_Syscall_setdomainname (const char* name, guint64 len);
index fd2bf257e43287f2e2eed474a8b21ad803617257..352d5941bc59c458b8d838e5b8fb98c269d55eba 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * <sys/xattr.h> wrapper functions.
+ * Wrapper functions for <sys/xattr.h> and <sys/extattr.h>
  *
  * Authors:
  *   Daniel Drake (dsd@gentoo.org)
 
 #include <config.h>
 
-#ifdef HAVE_SYS_XATTR_H
+#if defined(HAVE_SYS_XATTR_H) || defined(HAVE_SYS_EXTATTR_H)
 
 #include <sys/types.h>
+
+#ifdef HAVE_SYS_XATTR_H
 #include <sys/xattr.h>
+#endif
+
+#ifdef HAVE_SYS_EXTATTR_H
+#include <sys/extattr.h>
+#include <sys/uio.h>
+#endif
+
 #include <unistd.h>
 #include <fcntl.h>
 #include <errno.h>
+#include <string.h>
+#include <stdlib.h>
 
 #include "map.h"
 #include "mph.h"
 
+/*
+ * Linux provides extended attributes through the <sys/xattr.h> API.
+ * Any file or link can have attributes assigned to it (provided that they are
+ * supported by the backing filesystem). Each attribute has to be placed in a
+ * namespace, of which "user" is the most common. Namespaces are specified as
+ * a prefix to the attribute name, proceeded by a '.' (e.g. user.myattribute)
+ *
+ * FreeBSD provides extended attributes through the <sys/extattr.h> API.
+ * Behaviour is very similar to Linux EA's, but the namespace is specified
+ * through an enum-style parameter rather than as a prefix to an attribute
+ * name. There are also differences in the behaviour of the "list attributes"
+ * system calls.
+ *
+ * This file merges the two implementations into a single API for use by the
+ * Mono.Unix.Syscall.*xattr methods. No matter which OS you are on, things
+ * should "just work" the same as anywhere else.
+ *
+ * The API provided here leans more towards the Linux implementation. Attribute
+ * namespaces are provided as prefixes to the attribute name (followed by '.').
+ * There is no limit to the namespaces accepted by the Linux side of this
+ * implementation, but you are obviously limited to the ones available to you
+ * on the system.
+ * FreeBSD namespaces have to be converted from the textual prefix into their
+ * relevant number so that they can be used in the FreeBSD system calls.
+ * This means that the only namespaces available are the ones known by in this
+ * file (see bsd_extattr_namespaces). However, you can also specify the
+ * numericalnamespace index yourself, by using an attribute name such as
+ * "5.myattr".
+ * (this will obviously fail on Linux, your code will no longer be 'portable')
+ *
+ * Linux {,l,f}setxattr calls have a flags parameter which allow you to control
+ * what should happen if an attribute with the same name does (or doesn't)
+ * already exist. The 'flags' parameter is available here, but because FreeBSD
+ * does not support this kind of refinement, it will fail on FreeBSD if you
+ * specify anything other than XATTR_AUTO (XATTR_AUTO will create the attribute
+ * if it doesn't already exist, and overwrite the existing attribute if it
+ * already set).
+ * 
+ * For usage and behaviour information, see the monodoc documentation on the
+ * Mono.Unix.Syscall class.
+ */
+
 G_BEGIN_DECLS
 
+//
+// HELPER FUNCTIONS
+//
+
+#ifdef HAVE_SYS_EXTATTR_H
+
+struct BsdNamespaceInfo {
+       const char *name;
+       int value;
+};
+
+static struct BsdNamespaceInfo bsd_extattr_namespaces[] = {
+       {"user"         , EXTATTR_NAMESPACE_USER},
+       {"system"       , EXTATTR_NAMESPACE_SYSTEM}
+};
+
+static int bsd_check_flags (gint32 flags)
+{
+       // BSD doesn't support flags, but always provides the same behaviour as
+       // XATTR_AUTO. So we enforce that here.
+       if (flags != Mono_Posix_XattrFlags_XATTR_AUTO) {
+               errno = EINVAL;
+               return -1;
+       }
+       return 0;
+}
+
+// On FreeBSD, we need to convert "user.blah" into namespace 1 and attribute
+// name "blah", or maybe "6.blah" into namespace 6 attribute "blah"
+static int
+bsd_handle_nsprefix (const char *name, char **_name, int *namespace)
+{
+       int i;
+       gchar **components = g_strsplit (name, ".", 2);
+
+       // Find namespace number from textual representation
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++)
+               if (strcmp (bsd_extattr_namespaces[i].name, components[0]) == 0) {
+                       *namespace = bsd_extattr_namespaces[i].value;
+                       break;
+               }
+
+       if (*namespace == 0) {
+               // Perhaps they specified the namespace number themselves..?
+               char *endptr;
+               *namespace = (int) strtol (components[0], &endptr, 10);
+               if (*endptr != '\0')
+                       return -1;
+       }
+
+       *_name = g_strdup (components[1]);
+       g_strfreev (components);
+       return 0;
+}
+
+static void
+init_attrlists (char *attrlists[])
+{
+       memset (attrlists, 0, G_N_ELEMENTS(bsd_extattr_namespaces) * sizeof(char*));
+}
+
+static void
+free_attrlists (char *attrlists[])
+{
+       int i;
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++)
+               g_free (attrlists[i]);
+}
+
+// Counts the number of attributes in the result of a
+// extattr_list_*() call. Note that the format of the data
+// is: \3one\3two\6eleven where the leading charaters represent the length
+// of the following attribute. (the description in the man-page is wrong)
+static unsigned int
+count_num_attrs (char *attrs, size_t size)
+{
+       size_t i = 0;
+       unsigned int num_attrs = 0;
+
+       if (!attrs || !size)
+               return 0;
+
+       while (i < size) {
+               num_attrs++;
+               i += attrs[i] + 1;
+       }
+
+       return num_attrs;
+}
+
+// Convert a BSD-style list buffer (see the description for count_num_attrs)
+// into a Linux-style NULL-terminated list including namespace prefix.
+static char
+*bsd_convert_list (const char *nsprefix, const char *src, size_t size, char *dest)
+{
+       size_t i = 0;
+       if (src == NULL || dest == NULL || size == 0)
+               return NULL;
+
+       while (i < size) {
+               // Read length
+               int attr_len = (int) src[i];
+               int prefix_len = strlen (nsprefix);
+
+               // Add namespace prefix
+               strncpy (dest, nsprefix, prefix_len);
+               dest[prefix_len] = '.';
+               dest += prefix_len + 1;
+
+               // Copy attribute
+               memcpy(dest, src + ++i, attr_len);
+
+               // NULL-terminate
+               i += attr_len;
+               dest[attr_len] = '\0';
+               dest += attr_len + 1;
+       }
+
+       return dest;
+}
+
+// Combine all the lists of attributes that we know about into a single
+// Linux-style buffer
+static ssize_t
+bsd_combine_lists (char *attrlists[], char *dest, size_t dest_size_needed, size_t dest_size)
+{
+       int i;
+       if (!dest)
+               return dest_size_needed;
+
+       if (dest_size < dest_size_needed) {
+               errno = ERANGE;
+               return -1;
+       }
+
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++)
+               if (attrlists[i])
+                       dest = bsd_convert_list (bsd_extattr_namespaces[i].name, attrlists[i], strlen (attrlists[i]), dest);
+
+       return dest_size_needed;
+}
+
+static mph_ssize_t
+bsd_listxattr (const char *path, void *list, mph_size_t size)
+{
+       size_t full_size = 0;
+       int i;
+       char *attrlists[G_N_ELEMENTS(bsd_extattr_namespaces)];
+
+       init_attrlists (attrlists);
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++) {
+               size_t buf_size;
+               int num_attrs;
+
+               buf_size = (size_t) extattr_list_file (path, i + 1, NULL, 0);
+               if (buf_size == -1)
+                       continue;
+
+               attrlists[i] = g_malloc0 (buf_size + 1);
+               buf_size = (size_t) extattr_list_file (path, i + 1, attrlists[i], buf_size);
+               if (buf_size == -1)
+                       continue;
+
+               num_attrs = count_num_attrs(attrlists[i], buf_size);
+               full_size += buf_size + (num_attrs * (strlen (bsd_extattr_namespaces[i].name) + 1));
+       }
+
+       full_size = bsd_combine_lists (attrlists, (char *) list, full_size, size);
+       free_attrlists (attrlists);
+       return full_size;
+}
+
+static mph_ssize_t
+bsd_llistxattr (const char *path, void *list, mph_size_t size)
+{
+       size_t full_size = 0;
+       int i;
+       char *attrlists[G_N_ELEMENTS(bsd_extattr_namespaces)];
+
+       init_attrlists (attrlists);
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++) {
+               size_t buf_size;
+               int num_attrs;
+
+               buf_size = (size_t) extattr_list_link (path, i + 1, NULL, 0);
+               if (buf_size == -1)
+                       continue;
+
+               attrlists[i] = g_malloc0 (buf_size + 1);
+               buf_size = (size_t) extattr_list_link (path, i + 1, attrlists[i], buf_size);
+               if (buf_size == -1)
+                       continue;
+
+               num_attrs = count_num_attrs(attrlists[i], buf_size);
+               full_size += buf_size + (num_attrs * (strlen (bsd_extattr_namespaces[i].name) + 1));
+       }
+
+       full_size = bsd_combine_lists (attrlists, (char *) list, full_size, size);
+       free_attrlists (attrlists);
+       return full_size;
+}
+
+static mph_ssize_t
+bsd_flistxattr (int fd, void *list, mph_size_t size)
+{
+       size_t full_size = 0;
+       int i;
+       char *attrlists[G_N_ELEMENTS(bsd_extattr_namespaces)];
+
+       init_attrlists (attrlists);
+       for (i = 0; i < G_N_ELEMENTS(bsd_extattr_namespaces); i++) {
+               size_t buf_size;
+               int num_attrs;
+
+               buf_size = (size_t) extattr_list_fd (fd, i + 1, NULL, 0);
+               if (buf_size == -1)
+                       continue;
+
+               attrlists[i] = g_malloc0 (buf_size + 1);
+               buf_size = (size_t) extattr_list_fd (fd, i + 1, attrlists[i], buf_size);
+               if (buf_size == -1)
+                       continue;
+
+               num_attrs = count_num_attrs(attrlists[i], buf_size);
+               full_size += buf_size + (num_attrs * (strlen (bsd_extattr_namespaces[i].name) + 1));
+       }
+
+       full_size = bsd_combine_lists (attrlists, (char *) list, full_size, size);
+       free_attrlists (attrlists);
+       return full_size;
+}
+
+#endif /* HAVE_SYS_EXTATTR_H */
+
+//
+// THE PROVIDED API
+//
+
 gint32
 Mono_Posix_Syscall_setxattr (const char *path, const char *name, void *value, mph_size_t size, gint32 flags)
 {
-       int _flags;
+       gint32 ret;
        mph_return_if_size_t_overflow (size);
 
-       if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
-               return -1;
-
+#ifdef HAVE_SYS_XATTR_H
+       {
+               int _flags;
+               if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
+                       return -1;
 #if __APPLE__
-       return setxattr (path, name, value, size, 0, _flags);
-#else
-       return setxattr (path, name, value, size, _flags);
-#endif
+               ret = setxattr (path, name, value, (size_t) size, 0, _flags);
+#else /* __APPLE__ */
+               ret = setxattr (path, name, value, (size_t) size, _flags);
+#endif /* __APPLE__ */
+       }
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_check_flags (flags) == -1)
+                       return -1;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_set_file (path, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
 }
 
 #if !__APPLE__
 gint32
 Mono_Posix_Syscall_lsetxattr (const char *path, const char *name, void *value, mph_size_t size, gint32 flags)
 {
-       int _flags;
+       gint32 ret;
        mph_return_if_size_t_overflow (size);
 
-       if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
-               return -1;
+#ifdef HAVE_SYS_XATTR_H
+       {
+               int _flags;
+               if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
+                       return -1;
+               ret = lsetxattr (path, name, value, size, _flags);
+       }
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_check_flags (flags) == -1)
+                       return -1;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_set_link (path, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
 
-       return lsetxattr (path, name, value, size, _flags);
+       return ret;
 }
-#endif
+#endif /* !__APPLE__ */
 
 gint32
 Mono_Posix_Syscall_fsetxattr (int fd, const char *name, void *value, mph_size_t size, gint32 flags)
 {
-       int _flags;
+       gint32 ret;
        mph_return_if_size_t_overflow (size);
 
-       if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
-               return -1;
-
+#ifdef HAVE_SYS_XATTR_H
+       {
+               int _flags;
+               if (Mono_Posix_FromXattrFlags (flags, &_flags) == -1)
+                       return -1;
 #if __APPLE__
-       return fsetxattr (fd, name, value, (size_t) size, 0, _flags);
-#else
-       return fsetxattr (fd, name, value, (size_t) size, _flags);
-#endif
+               ret = fsetxattr (fd, name, value, (size_t) size, 0, _flags);
+#else /* __APPLE__ */
+               ret = fsetxattr (fd, name, value, (size_t) size, _flags);
+#endif /* __APPLE__ */
+       }
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_check_flags (flags) == -1)
+                       return -1;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_set_fd (fd, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
 
+       return ret;
 }
 
 mph_ssize_t
 Mono_Posix_Syscall_getxattr (const char *path, const char *name, void *value, mph_size_t size)
 {
+       mph_ssize_t ret;
        mph_return_if_size_t_overflow (size);
+
+#ifdef HAVE_SYS_XATTR_H
 #if __APPLE__
-       return getxattr (path, name, value, (size_t) size, 0, 0);
-#else
-       return getxattr (path, name, value, (size_t) size);
-#endif
+       ret = getxattr (path, name, value, (size_t) size, 0, 0);
+#else /* __APPLE__ */
+       ret = getxattr (path, name, value, (size_t) size);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_get_file (path, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
 }
 
 #if !__APPLE__
 mph_ssize_t
 Mono_Posix_Syscall_lgetxattr (const char *path, const char *name, void *value, mph_size_t size)
 {
+       mph_ssize_t ret;
        mph_return_if_size_t_overflow (size);
-       return lgetxattr (path, name, value, (size_t) size);
+
+#ifdef HAVE_SYS_XATTR_H
+       ret = lgetxattr (path, name, value, (size_t) size);
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_get_link (path, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
 }
-#endif
+#endif /* !__APPLE__ */
 
 mph_ssize_t
 Mono_Posix_Syscall_fgetxattr (int fd, const char *name, void *value, mph_size_t size)
 {
+       mph_ssize_t ret;
        mph_return_if_size_t_overflow (size);
+
+#ifdef HAVE_SYS_XATTR_H
 #if __APPLE__
-       return fgetxattr (fd, name, value, (size_t) size, 0, 0);
-#else
-       return fgetxattr (fd, name, value, (size_t) size);
-#endif
+       ret = fgetxattr (fd, name, value, (size_t) size, 0, 0);
+#else /* __APPLE__ */
+       ret = fgetxattr (fd, name, value, (size_t) size);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_get_fd (fd, namespace, _name, value, (size_t) size);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
 }
 
 mph_ssize_t
 Mono_Posix_Syscall_listxattr (const char *path, void *list, mph_size_t size)
 {
        mph_return_if_size_t_overflow (size);
+
+#ifdef HAVE_SYS_XATTR_H
 #if __APPLE__
-       return listxattr (path, (char *) list, (size_t) size, 0);
-#else
-       return listxattr (path, (char *) list, (size_t) size);
-#endif
+       return listxattr (path, list, (size_t) size, 0);
+#else /* __APPLE__ */
+       return listxattr (path, list, (size_t) size);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       return bsd_listxattr (path, list, size);
+#endif /* HAVE_SYS_XATTR_H */
 }
 
 #if !__APPLE__
@@ -116,24 +507,107 @@ mph_ssize_t
 Mono_Posix_Syscall_llistxattr (const char *path, void *list, mph_size_t size)
 {
        mph_return_if_size_t_overflow (size);
-       return llistxattr (path, (char *) list, (size_t) size);
+
+#ifdef HAVE_SYS_XATTR_H
+       return llistxattr (path, list, (size_t) size);
+#else /* HAVE_SYS_XATTR_H */
+       return bsd_llistxattr (path, list, size);
+#endif /* HAVE_SYS_XATTR_H */
 }
-#endif
+#endif /* !__APPLE__ */
 
 mph_ssize_t
 Mono_Posix_Syscall_flistxattr (int fd, void *list, mph_size_t size)
 {
        mph_return_if_size_t_overflow (size);
+
+#ifdef HAVE_SYS_XATTR_H
 #if __APPLE__
-       return flistxattr (fd, (char *) list, (size_t) size, 0);
-#else
-       return flistxattr (fd, (char *) list, (size_t) size);
-#endif
+       return flistxattr (fd, list, (size_t) size, 0);
+#else /* __APPLE__ */
+       return flistxattr (fd, list, (size_t) size);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       return bsd_flistxattr (fd, list, size);
+#endif /* HAVE_SYS_XATTR_H */
+}
+
+gint32
+Mono_Posix_Syscall_removexattr (const char *path, const char *name)
+{
+       gint32 ret;
+
+#ifdef HAVE_SYS_XATTR_H
+#if __APPLE__
+       ret = removexattr (path, name, 0);
+#else /* __APPLE__ */
+       ret = removexattr (path, name);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_delete_file (path, namespace, _name);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
+}
+
+#if !__APPLE__
+gint32
+Mono_Posix_Syscall_lremovexattr (const char *path, const char *name)
+{
+       gint32 ret;
+
+#ifdef HAVE_SYS_XATTR_H
+       ret = lremovexattr (path, name);
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_delete_link (path, namespace, _name);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
+}
+#endif /* !__APPLE__ */
+
+gint32
+Mono_Posix_Syscall_fremovexattr (int fd, const char *name)
+{
+       gint32 ret;
+
+#ifdef HAVE_SYS_XATTR_H
+#if __APPLE__
+       ret = fremovexattr (fd, name, 0);
+#else /* __APPLE__ */
+       ret = fremovexattr (fd, name);
+#endif /* __APPLE__ */
+#else /* HAVE_SYS_XATTR_H */
+       {
+               char *_name;
+               int namespace;
+               if (bsd_handle_nsprefix (name, &_name, &namespace) == -1)
+                       return -1;
+               ret = extattr_delete_fd (fd, namespace, _name);
+               g_free (_name);
+       }
+#endif /* HAVE_SYS_XATTR_H */
+
+       return ret;
 }
 
 G_END_DECLS
 
-#endif /* def HAVE_ATTR_XATTR_H */
+#endif /* HAVE_SYS_XATTR_H || HAVE_SYS_EXTATTR_H */
 
 /*
  * vim: noexpandtab