jscript: Do not call memcpy() with a NULL pointer argument

Jacek Caban jacek at codeweavers.com
Sat Dec 20 10:38:44 CST 2008


Hi Andrew,

Andrew Talbot wrote:
> Jacek Caban wrote:
>
>   
>> The string was always zero-terminated without your patch. It's fine to
>> call create_string with NULL str argument as long as len is 0 and
>> current implementation works fine in this case.
>>
>>
>> Jacek
>>     
>
> Hi Jacek,
>
> Technically, behavior is undefined if the pointers do not each point to an
> object, even if the size parameter has a value of zero (see
> http://tinyurl.com/6eqo3n, third post). Though I concede that it would be a
> rare implementation that touched its pointers (or copies of them) in that
> case.
>   

I'm not fan of such fixes, but if you want to fix it, you should check 
len, not str, in your patch and you may move zero-terminating outside 
if..else statement.


Jacek



More information about the wine-devel mailing list