[PATCH] wininet/tests: Don't check certificate when secure flag is not set

Jacek Caban jacek at codeweavers.com
Mon Sep 17 09:55:34 CDT 2018


Hi André,

On 09/15/2018 02:47 PM, André Hentschel wrote:
> Signed-off-by: André Hentschel <nerv at dawncrow.de>
> ---
>  dlls/wininet/tests/http.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
> index ace8512399..8bc06b9166 100644
> --- a/dlls/wininet/tests/http.c
> +++ b/dlls/wininet/tests/http.c
> @@ -6025,7 +6025,8 @@ static void test_secure_connection(void)
>      ok(ret, "InternetQueryOption failed: %d\n", GetLastError());
>      ok(flags & SECURITY_FLAG_SECURE, "expected secure flag to be set\n");
>  
> -    test_cert_struct(req, &test_winehq_org_cert);
> +    if (flags & SECURITY_FLAG_SECURE)
> +        test_cert_struct(req, &test_winehq_org_cert);

What do you want to achieve with this patch? Note that we test the flag
just before test_cert_struct call, so tests will fail with your patch
anyway.

Jacek



More information about the wine-devel mailing list