Re: wintrust: Implement CryptCATAdminCalcHashFromFileHandle, try 2

[Alexandre Julliard]

Maarten Lankhorst <m.b.lankhorst(a)gmail.com> writes:
> +        curpos.QuadPart = 0;
> +        SetFilePointerEx(hFile, curpos, &oldpos, FILE_CURRENT);
> +        SetFilePointerEx(hFile, curpos, NULL, FILE_BEGIN);

That's not thread-safe.

-- 
Alexandre Julliard
julliard(a)winehq.org

wintrust: Implement CryptCATAdminCalcHashFromFileHandle, try 2

[Maarten Lankhorst]

Hi AJ,

Alexandre Julliard schreef:
> Maarten Lankhorst <m.b.lankhorst(a)gmail.com> writes:
> > +        curpos.QuadPart = 0;
> > +        SetFilePointerEx(hFile, curpos, &oldpos, FILE_CURRENT);
> > +        SetFilePointerEx(hFile, curpos, NULL, FILE_BEGIN);
>
> That's not thread-safe.

My bad, I thought overlapped was equal to asynchronous, but it can also 
be used for synchronous reads it seems, sending a fixed patch.

Cheers,
Maarten.