The total count of valgrind warnings in the wine test suite has gone down from about 2000 a month ago http://kegel.com/wine/valgrind/logs/2009-11-16-19.12.errs.txt to about 1000 today http://kegel.com/wine/valgrind/logs/2009-12-15-07.57.errs.txt (as computed by http://kegel.com/wine/valgrind/logs/count.sh.txt ).
Thanks to everyone who has been helping clean this up!
Let's see if we can cut it down another 50% by this time next month...
On Tue, Dec 15, 2009 at 11:54 AM, Dan Kegel dank@kegel.com wrote:
The total count of valgrind warnings in the wine test suite has gone down from about 2000 a month ago... to about 1000 today
http://kegel.com/wine/valgrind/logs/2009-12-16-09.22.errs.txt 1 Use of uninitialised value of size 4 2 Invalid free/ delete / delete[] 2 Source and destination overlap in memcpy 3 Syscall param writev 6 Invalid read of size 1 6 Invalid write of size 4 8 Invalid write of size 1 38 Invalid read of size 2 49 Invalid read of size 4 54 Conditional jump or move depends on uninitialised value 734 XXX bytes in XXX blocks are definitely lost total: 903
Woot! Keep those fixes coming...
Request: could somebody who knows TLS slots review http://source.winehq.org/patches/data/55766 That'd knock out two invalid read errors, but it's stuck in AJ's review queue. - Dan
On Wed, Dec 16, 2009 at 12:56 PM, Dan Kegel dank@kegel.com wrote:
http://kegel.com/wine/valgrind/logs/2009-12-16-09.22.errs.txt 38 Invalid read of size 2 49 Invalid read of size 4 54 Conditional jump or move depends on uninitialised value 734 XXX bytes in XXX blocks are definitely lost total: 903
http://kegel.com/wine/valgrind/logs/2009-12-17-08.55/ http://kegel.com/wine/valgrind/logs/2009-12-17-08.55.errs.txt ... 14 Invalid read of size 4 39 Invalid read of size 2 55 Conditional jump or move depends on uninitialised value 712 XXX bytes in XXX blocks are definitely lost total: 841
Thanks especially to James for continuing to grind out msi fixes in his not-so-copious spare time.
-
Dan Kegel