Re: [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(a)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(a)winehq.org