[5/6]wininet: handle cookie expiration

Aric Stewart aric at codeweavers.com
Wed Feb 18 07:40:45 CST 2009


Wonderful, I was looking for a function JUST like that.

Reworked patch coming up shortly.
-aric

Hans Leidekker wrote:
>>> +    LPCWSTR ptr,ptr2;
>>> +    SYSTEMTIME tm;
>>> +    int i;
>>> +
>>> +    memset(&tm,0,sizeof(tm));
>>> +
>>> +    ptr = expiry;
>>> +    for (i = 0; i < 7; i++)
>>> +    {
>>> +        if (strncmpiW(day[i],ptr,3)==0)
>>> +        {
>>> +            tm.wDayOfWeek = i;
>>> +            break;
>>> +        }
>>> +    }
>>> +    ptr += 5;
>>> +    tm.wDay = strtolW(ptr,(LPWSTR*)&ptr2,10);
>>> +    ptr = ptr2+1;
>> You should handle the various possible parsing errors, not just assume
>> that you have a well-formatted string.
> 
> Or use InternetTimeToSystemTimeW which does that already.
> 
>  -Hans
> 



More information about the wine-devel mailing list