urlmon:sec_mgr failure on XP

[Francois Gouget]

This test now fails on my Widnows XP VMs that have IE >= 7 with 'All 
zones reset to default level':
http://test.winehq.org/data/3b40c03a8f0e3dcbb236d59125b115b2501086f8/xp_fg-w...

sec_mgr.c:1695: Test failed: Expected zone 3, but got 1 for 'file/testing/test.test'

The patch that introduces the failures is 98d5dfb8 which was committed 
on the 5th.

I did some experiments and I can get this test to pass if I do:
 * Go to the Internet Properties control panel
 * Security tab
 * Select 'Local intranet'
 * Click on the 'Sites' button
 * Deselect 'Automatically detect intranet network'
 * Deselect 'Include all local (intranet) sites not listed in other zones'
 * Click on 'OK'

I also checked on Windows 7 and by default it has the first checkbox 
checked but the second one unchecked. Disabling 'Automatically detect 
intranet network' and enabling 'Include all local sites...' to match 
Windows XP causes the test to fail on Windows 7 too.

Finally, on XP even if I deselect all the 'Include XXX' checkboxes but 
select the 'Automatically detect intranet network' one the test fails 
while with the same configuration it passes on Windows 7. So I think 
this reflects a difference in the way the intranet detection works.

So I'm hoping that you can fix the test.

The easiest fix would be to remove the 'file/testing/test.test' check. 
An alternative would be to check what the settings are and adjust the 
expected result or skip the test. But it seems like this would be much 
more complex.

[Thomas Mullaly]

Hi Francois,

On Sat, Sep 10, 2011 at 6:33 AM, Francois Gouget  wrote:
> This test now fails on my Widnows XP VMs that have IE >= 7 with 'All
> zones reset to default level':
> http://test.winehq.org/data/3b40c03a8f0e3dcbb236d59125b115b2501086f8/xp_fg-w...
>
> sec_mgr.c:1695: Test failed: Expected zone 3, but got 1 for 'file/testing/test.test'

I wasn't able to reproduce this problem on my Windows XP VM. I was,
however, able to reproduce this problem using Windows 7.
> So I'm hoping that you can fix the test.
>
> The easiest fix would be to remove the 'file/testing/test.test' check.
> An alternative would be to check what the settings are and adjust the
> expected result or skip the test. But it seems like this would be much
> more complex.

The test in question isn't really relevant with showing how the
function itself should work, so removing it will be fine (I'll be
sending a patch shortly removing it).

Thanks for pointing this issue out.