wininet: Don't perform buffer size tests for components that don't pass buffer in WinHttpCrackUrl.

Jacek Caban jacek at
Tue Aug 30 12:27:16 CDT 2016

This fixes regression in Office 2013 installer caused by
1d07f8987993fb0cb5398e9a45f2435eb771a471. At the moment components
pointers are checked against NULL to test if buffer is copied, they can
be already set by WinHttpCrackUrl via set_component. Existing tests
didn't catch it, because they use ~0u and pass buffer length tests anyway.

Signed-off-by: Jacek Caban <jacek at>
 dlls/winhttp/tests/url.c |  4 ++-
 dlls/winhttp/url.c       | 93
 2 files changed, 30 insertions(+), 67 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-wininet-Don-t-perform-buffer-size-tests-for-component.diff
Type: text/x-patch
Size: 10324 bytes
Desc: not available
URL: <>

More information about the wine-patches mailing list