Re: Emacs 23.1.93 pretest

[Chong Yidong]

Eli Zaretskii <address@hidden> writes:

> "bzr bisect" points to this change as the reason for this bug:
>
>     2010-02-05  Chong Yidong  <address@hidden>
>
>           * charset.c (load_charset_map_from_file): Allocate large
>           charset_map_entries structure on the heap rather than the stack.
>           (Bug#5526).
>
> The revisions before this change works correctly; all revisions after
> it fail as described above.

Hmm, this is strange.  This change (actually the succeeding 2010-02-06
change to the same place) switches from using alloca to SAFE_ALLOCA
(i.e. malloc, since the desired structure is large).  But the only way I
can see for this code to crash is if load_charset_map somehow makes a
pointer into the allocated structure.  But in that case, the old alloca
case should have crashed too.

If you remove the SAFE_FREE () calls, does that prevent the crash?