[PATCH] cygwin: doc: Add keywords for ACE order issues

Yaakov Selkowitz yselkowi@redhat.com
Tue May 12 21:23:26 GMT 2020


On Tue, 2020-05-12 at 22:49 +0200, David Macek via Cygwin-patches
wrote:
> Windows Explorer shows a warning with Cygwin-created DACLs, but putting
> the text of the warning into Google doesn't lead to the relevant Cygwin
> docs.  Let's copy the warning text into the docs in the hopes of helping
> confused users.
> 
> Latest inquiry: <https://cygwin.com/pipermail/cygwin/2020-May/244814.html>
> 
> Signed-off-by: David Macek <david.macek.0@gmail.com>
> ---
>  winsup/doc/ntsec.xml | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/winsup/doc/ntsec.xml b/winsup/doc/ntsec.xml
> index 08a33bdc6c..b94cdd9a97 100644
> --- a/winsup/doc/ntsec.xml
> +++ b/winsup/doc/ntsec.xml
> @@ -2163,7 +2163,10 @@ preferred order.</para>
>  the Windows Explorer insists to rearrange the order of the ACEs to
>  canonical order before you can read them. Thank God, the sort order
>  remains unchanged if one presses the Cancel button.  But don't even
> -<emphasis role='bold'>think</emphasis> of pressing OK...</para>
> +<emphasis role='bold'>think</emphasis> of pressing OK...  For the sake
> +of people searching for this explanation, let's note that the Explorer
> +warning says "The permissions on ... are incorrectly orderer, which may
> +cause some entries to be ineffective."</para>
>  
>  <para>Canonical ACLs are unable to reflect each possible combination
>  of POSIX permissions. Example:</para>

The wording seems awkward.  Why not quote the text of the warning
directly earlier in the paragraph, e.g.:

<para>Unfortunately, the security tab in the file properties dialog of
the Windows Explorer will pop up a warning stating: "The permissions on
... are incorrectly ordered, which may cause some entries to be
ineffective."  Pressing the Cancel button will leave the order
unchanged, but pressing OK will cause Windows to canonicalize the order
of the ACEs, thereby invalidating POSIX compatibility.</para>

-- 
Yaakov Selkowitz
Senior Software Engineer - Platform Enablement
Red Hat, Inc.




More information about the Cygwin-patches mailing list