illumos gate

The ACL generated for mode 0077 looks like this:

            owner@:rwx-----------:------:deny
            owner@:------aARWcCos:------:allow
            group@:rwxp--a-R-c--s:------:allow
         everyone@:rwxp--a-R-c--s:------:allow

It should look like this instead (note the "append" permission in the first entry):

            owner@:rwxp----------:------:deny
            owner@:------aARWcCos:------:allow
            group@:rwxp--a-R-c--s:------:allow
         everyone@:rwxp--a-R-c--s:------:allow

Suggested patch looks like this:

--- sys/cddl/contrib/opensolaris/common/acl/acl_common.c        2010-08-30 10:40:14.353057022 +0200
+++ sys/cddl/contrib/opensolaris/common/acl/acl_common.c.new    2010-09-20 22:26:36.588374918 +0200
@@ -1581,21 +1581,21 @@ acl_trivial_access_masks(mode_t mode, ui
        if (!(mode & S_IRUSR) && (mode & (S_IRGRP|S_IROTH)))
                *deny1 |= ACE_READ_DATA;
        if (!(mode & S_IWUSR) && (mode & (S_IWGRP|S_IWOTH)))
-               *deny1 |= ACE_WRITE_DATA;
+               *deny1 |= ACE_WRITE_DATA|ACE_APPEND_DATA;
        if (!(mode & S_IXUSR) && (mode & (S_IXGRP|S_IXOTH)))
                *deny1 |= ACE_EXECUTE;

        if (!(mode & S_IRGRP) && (mode & S_IROTH))
                *deny2 = ACE_READ_DATA;
        if (!(mode & S_IWGRP) && (mode & S_IWOTH))
-               *deny2 |= ACE_WRITE_DATA;
+               *deny2 |= ACE_WRITE_DATA|ACE_APPEND_DATA;
        if (!(mode & S_IXGRP) && (mode & S_IXOTH))
                *deny2 |= ACE_EXECUTE;

        if ((mode & S_IRUSR) && (!(mode & S_IRGRP) && (mode & S_IROTH)))
                *allow0 |= ACE_READ_DATA;
        if ((mode & S_IWUSR) && (!(mode & S_IWGRP) && (mode & S_IWOTH)))
-               *allow0 |= ACE_WRITE_DATA;
+               *allow0 |= ACE_WRITE_DATA|ACE_APPEND_DATA;
        if ((mode & S_IXUSR) && (!(mode & S_IXGRP) && (mode & S_IXOTH)))
                *allow0 |= ACE_EXECUTE;