Re: Assorted comment, trace and ok() message spelling fixes.

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=22042

Your paranoid android.

=== w864 (32 bit security) ===
security.c:3964: Test failed: MACHINE\Software owner SID (S-1-5-18) != Administrators SID (S-1-5-32-544).
security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0xa != 0x0)

=== w1064 (32 bit security) ===
security.c:3964: Test failed: MACHINE\Software owner SID (S-1-5-18) != Administrators SID (S-1-5-32-544).
security.c:4005: Test failed: Builtin Users ACE has unexpected flags (0x12 != 0xa)
security.c:4009: Test failed: Builtin Users ACE has unexpected mask (0x20019 != 0x80000000)
security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0x12 != 0x0)

=== w1064 (64 bit security) ===
security.c:4005: Test failed: Builtin Users ACE has unexpected flags (0x2 != 0xa)
security.c:4009: Test failed: Builtin Users ACE has unexpected mask (0x20019 != 0x80000000)
security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0x2 != 0x0)

Assorted comment, trace and ok() message spelling fixes.

[Francois Gouget]

On Sun, 10 Apr 2016, Marvin wrote:
[...]
> === w864 (32 bit security) ===
> security.c:3964: Test failed: MACHINE\Software owner SID (S-1-5-18) != Administrators SID (S-1-5-32-544).
> security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0xa != 0x0)
>
> === w1064 (32 bit security) ===
> security.c:3964: Test failed: MACHINE\Software owner SID (S-1-5-18) != Administrators SID (S-1-5-32-544).
> security.c:4005: Test failed: Builtin Users ACE has unexpected flags (0x12 != 0xa)
> security.c:4009: Test failed: Builtin Users ACE has unexpected mask (0x20019 != 0x80000000)
> security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0x12 != 0x0)
>
> === w1064 (64 bit security) ===
> security.c:4005: Test failed: Builtin Users ACE has unexpected flags (0x2 != 0xa)
> security.c:4009: Test failed: Builtin Users ACE has unexpected mask (0x20019 != 0x80000000)
> security.c:4018: Test failed: Builtin Admins ACE has unexpected flags (0x2 != 0x0)

These test failures are not 100% systematic but were already there on 
one test machine or another (for instance on win8_cw2-gtx560-t64 and 
win10_fg-acer64-t32).

https://test.winehq.org/data/tests/advapi32:security.html

It happened on w864 there for instance:
https://test.winehq.org/data/aaddf1365a3371263827206eedf1464a83562c00/win8_n...