[PATCH 4/4] server: Add primitive support for setting and getting the security descriptor of files based on their Unix permissions.

[Alexandre Julliard]

Robert Shearman <rob at codeweavers.com> writes:

> +    /* assume we are the owner */
> +    if (st.st_mode & S_IRWXU)
> +    {

Why are you assuming that?  There doesn't seem to be much sense in
checking permissions without checking the owner.

-- 
Alexandre Julliard
julliard at winehq.org