Always use lchmod() if it is available.
authorWayne Davison <wayned@samba.org>
Thu, 31 Dec 2009 22:10:38 +0000 (14:10 -0800)
committerWayne Davison <wayned@samba.org>
Thu, 31 Dec 2009 22:10:38 +0000 (14:10 -0800)
syscall.c

index 945a11d4e208313b6410fd2da0bf552018449f49..aba0009d03642698e22188f1b200b492d37fa9f8 100644 (file)
--- a/syscall.c
+++ b/syscall.c
@@ -195,10 +195,11 @@ int do_chmod(const char *path, mode_t mode)
        int code;
        if (dry_run) return 0;
        RETURN_ERROR_IF_RO_OR_LO;
        int code;
        if (dry_run) return 0;
        RETURN_ERROR_IF_RO_OR_LO;
-       if (S_ISLNK(mode)) {
 #ifdef HAVE_LCHMOD
 #ifdef HAVE_LCHMOD
-               code = lchmod(path, mode & CHMOD_BITS);
-#elif defined HAVE_SETATTRLIST
+       code = lchmod(path, mode & CHMOD_BITS);
+#else
+       if (S_ISLNK(mode)) {
+# if defined HAVE_SETATTRLIST
                struct attrlist attrList;
                uint32_t m = mode & CHMOD_BITS; /* manpage is wrong: not mode_t! */
 
                struct attrlist attrList;
                uint32_t m = mode & CHMOD_BITS; /* manpage is wrong: not mode_t! */
 
@@ -206,11 +207,12 @@ int do_chmod(const char *path, mode_t mode)
                attrList.bitmapcount = ATTR_BIT_MAP_COUNT;
                attrList.commonattr = ATTR_CMN_ACCESSMASK;
                code = setattrlist(path, &attrList, &m, sizeof m, FSOPT_NOFOLLOW);
                attrList.bitmapcount = ATTR_BIT_MAP_COUNT;
                attrList.commonattr = ATTR_CMN_ACCESSMASK;
                code = setattrlist(path, &attrList, &m, sizeof m, FSOPT_NOFOLLOW);
-#else
+# else
                code = 1;
                code = 1;
-#endif
+# endif
        } else
                code = chmod(path, mode & CHMOD_BITS); /* DISCOURAGED FUNCTION */
        } else
                code = chmod(path, mode & CHMOD_BITS); /* DISCOURAGED FUNCTION */
+#endif /* !HAVE_LCHMOD */
        if (code != 0 && (preserve_perms || preserve_executability))
                return code;
        return 0;
        if (code != 0 && (preserve_perms || preserve_executability))
                return code;
        return 0;