Re: [Qemacs-devel] [PATCH] Shut up valgrind

[François Revol]

Hi,

On 24/01/2014 15:05, Charles Gordon wrote:
>> But it frees a little too much maybe:
>>
>> ==451== Invalid read of size 8
>> ==451==    at 0x41B104: main (qe.c:8111)
>> ==451==  Address 0x647b1f0 is 0 bytes inside a block of size 24 free'd
>> ==451==    at 0x4C2A70C: free (vg_replace_malloc.c:468)
>> ==451==    by 0x41B12B: main (qe.c:8112)
> 
> Interesting.  is this the line where I free the key bindings ?
> double free here smells fishy.

looks like it:
8111:                m->first_key = p->next;
8112:                qe_free(&p);

> If you have a reproducible test case for the above, I’m interested.
> 

valgrind ./qe_g --free-all shell.c

but it doesn't seem to be mode-specific.

François.