Re: MPS make-thread

[Gerd Möllmann]

Gerd Möllmann <gerd.moellmann@gmail.com> writes:

> Helmut Eller <eller.helmut@gmail.com> writes:
>
>>> Just make-thread.
>>> (I am not sure if it is related; if not, I'd better try to prepare a
>>> reproducer later and start a new email thread)
>>
>> This crashes for me:
>>
>>   emacs -Q -batch -eval \
>>    '(thread-join (make-thread (lambda () (igc--collect))))'
>
> On macOS, and in my fork, it says
>
> emacs(38573,0x17071f000) malloc: *** error for object 0x1040c8008: pointer 
> being freed was not allocated
> emacs(38573,0x17071f000) malloc: *** set a breakpoint in malloc_error_break 
> to debug

It's the xfree in run_thread here:

    struct handler *c, *c_next;
    for (c = handlerlist_sentinel; c; c = c_next)
      {
        c_next = c->nextfree;
        xfree (c);
      }

c is an MPS objects. I can fix that later after verifying that all
handlers are now in MPS. Unless someone(tm) is quicker, of course.