Re: sound issues (which are not cared since 1.3.1)

wylda at volny.cz wylda at volny.cz
Fri Dec 31 10:07:46 CST 2010


> > 
> > int card, err, capcontrols=0
> > 
> > otherwise things like:
> > 
> > mixdev[mixnum].chans += capcontrols;
> > 
> > mixdev[mixnum].lines =
> > HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY,
> > sizeof(line) * mixdev[mixnum].chans);
> > 
> > 
> > goes crazy like in
> > http://bugs.winehq.org/show_bug.cgi?id=25231#c2.

> 
> well spotted, the patch incorrectly removed the = 0.
> 
> I submitted a patch readding it.

Hi again!

Many thanks Marcus for the patch and getting it git-official :)

I wonder if someone could write a Wiki page about different
techniques how beginners like me can spot and easily test for this
kind of wine's code problems.

Thanks to this bug, Wine-1.3.10 + winecfg's audio tab could be taken
as the example and wiki could show how to use different tools
(warn+heap, valgrind, gcc options, ...) and what tools' output to
expect in case of such bugs.


Regards,
W.

PS: Marcus, i noticed "found by my overflow checking patches" in
http://source.winehq.org/patches/data/69925. Are these patches
publicly available? ;)





More information about the wine-devel mailing list