[PATCH] msxml3: Null check url in parseURL.

Jefferson Carpenter jeffersoncarpenter2 at gmail.com
Tue Jun 15 12:32:56 CDT 2021


On 6/15/2021 7:36 AM, Nikolay Sivov wrote:
> This is used in IVB* interface too, which is not tested.

Correct, it looks like only Invoke is tested on that interface.  It 
would be a much bigger change to add comprehensive tests for the IVB 
interface (the interface is not queried in test_saxreader).

> We might want to do the same, there is a 'version' field to account for
> that.
> 

This seems like a place to do the "right thing" instead of maintaining 
bug-for-bug compatibility - unless a program depends on the process 
crashing due to a null pointer access.



More information about the wine-devel mailing list