[4/6] msvcrt: Added _fread_nolock_s implementation
Iván Matellanes
matellanesivan at gmail.com
Fri Oct 31 07:00:26 CDT 2014
On 31/10/14 07:46, Alexandre Julliard wrote:
> "Iván Matellanes" <matellanesivan at gmail.com> writes:
>
>> {
>> + MSVCRT_size_t ret;
>> +
>> + MSVCRT__lock_file(stream);
>> + ret = MSVCRT__fread_nolock_s(buf, buf_size, elem_size, count, stream);
>> + MSVCRT__unlock_file(stream);
>
> The parameter validation has to be done before locking the file.
>
Hi, thanks for the comment. Should all parameters be validated, or just stream != NULL?
Validating the rest seems redundant to me.
Regards,
Iván.
More information about the wine-devel
mailing list