Apparently it is possible to have chm file without #STRINGS in it, so we should not stop processing file at it. Fixes bug #12498

Tomasz Jezierski - Tefnet developers at tefnet.pl
Tue Apr 15 10:07:23 CDT 2008


Dnia 15-04-2008, wto o godzinie 07:14 -0700, Juan Lang pisze:
> >  Have you tried patch without this "if"? In my case it crashes
without it
> >  when I try to open chm from bug #12498... i don't know if strdupW
can
> >  dup NULL as well as null?
> 
> Aaaah...  Sorry, my mistake, I missed this declaration:
>         static const WCHAR null[] = {0};
> That solves that mystery.
> 
> In that case, one fix per patch, please.  This fix and the other are
> unrelated:  one fixes a crash, the other deals with a missing #STRINGS
> stream.  Thanks,

No, I think that crash is only possible with my patch. Right now if
there is no #STRINGS it closes chm and stops processing.





More information about the wine-devel mailing list