New valgrind results, this time with overrun checks and lots of new errors.
Nikolay Sivov
bunglehead at gmail.com
Thu Nov 19 22:38:43 CST 2009
Dan Kegel wrote:
> On Thu, Nov 19, 2009 at 7:37 AM, Nikolay Sivov <bunglehead at gmail.com> wrote:
>
>>> http://kegel.com/wine/valgrind/logs/2009-11-18-21.51/
>>> is the first full run with the heap tail check enabled.
>>>
>> So you use some private patches for that, why aren't they merged?
>>
>
> 'Cause I just wrote it an hour before that message?
> I already posted it to wine-devel for the curious.
>
Actually I didn't spot that, sorry, I just was interested :).
> I'll try to post to wine-patches today or so.
>
> In the meantime I've updated
> http://wiki.winehq.org/Valgrind
> to link to the patch.
>
> I'm not sure about the user (i.e. developer) interface for turning the heap
> check on. Should it be automatic when one does warn+heap?
> Or should it need an environment variable or registry setting to
> turn it on? How should the user control the size of the redzone
> (a feature Microsoft doesn't seem to offer)?
>
If I got it right it makes sense to enable this only running under
valgrind control?
> For what it's worth, Windows seems to turn the debug heap on
> by default when running apps under a debugger, and lets you disable it
> by setting the environment variable _NO_DEBUG_HEAP=1.
> That makes me think we should turn this feature on automatically
> under warn+heap.
> For finer control, setting _NO_DEBUG_HEAP=1 would disable it,
> and setting WINE_HEAP_REDZONE would let you pick how big a redzone
> to use. (Usually 16 bytes is enough, but I've seen bugs that misses
> but 64 bytes catches.)
>
That's enough maybe, daily script could simply run twice or so with
different zone length, results could
be a diff to default run (16 bytes) results.
> - Dan
>
>
More information about the wine-devel
mailing list