[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH] secur32/tests: handle skip cleanup in negotiate more cleanly
From:       Aric Stewart <aric () codeweavers ! com>
Date:       2015-09-30 18:11:32
Message-ID: 560C25D4.2010704 () codeweavers ! com
[Download RAW message or body]

On 9/30/15 11:07 AM, Nikolay Sivov wrote:
> On 30.09.2015 19:04, Aric Stewart wrote:
>> Signed-off-by: Aric Stewart <aric@codeweavers.com>
>> ---
>>   dlls/secur32/tests/negotiate.c | 31 +++++++++++++++++++++++--------
>>   1 file changed, 23 insertions(+), 8 deletions(-)
>>
> 
> 
>> -    status_c = pDeleteSecurityContext( &client.ctxt );
>> -    ok( status_c == SEC_E_OK, "DeleteSecurityContext returned %08x\n", status_c );
>> +    if (client.ctxt.dwLower || client.ctxt.dwUpper)
>> +    {
>> +        status_c = pDeleteSecurityContext( &client.ctxt );
>> +        ok( status_c == SEC_E_OK, "DeleteSecurityContext returned %08x\n", status_c );
>> +    }
> 
> Why is this a problem only on macos?
> 
> 
No idea, Again I was trying to turn a crash into a cleaner failure.

-aric


[prev in list] [next in list] [prev in thread] [next in thread]