GNU bug report logs - #79156
igc: igc_xpalloc_ambig SEGV

Previous Next

Full log

Message #40 received at 79156 <at> debbugs.gnu.org (full text, mbox):

From: Stéphane Marks <shipmints <at> gmail.com>
To: Gerd Möllmann <gerd.moellmann <at> gmail.com>
Cc: Pip Cet <pipcet <at> protonmail.com>, eller.helmut <at> gmail.com,
 Eli Zaretskii <eliz <at> gnu.org>, 79156 <at> debbugs.gnu.org
Subject: Re: bug#79156: igc: igc_xpalloc_ambig SEGV
Date: Sun, 3 Aug 2025 10:05:28 -0400

[Message part 1 (text/plain, inline)]

On Sun, Aug 3, 2025 at 8:53 AM Gerd Möllmann <gerd.moellmann <at> gmail.com>
wrote:

> Gerd Möllmann <gerd.moellmann <at> gmail.com> writes:
>
> > Pip Cet <pipcet <at> protonmail.com> writes:
> >
> >> Gerd Möllmann <gerd.moellmann <at> gmail.com> writes:
> >>
> >>> Eli Zaretskii <eliz <at> gnu.org> writes:
> >>>
> >>>>> Ouch. That seems to me to be a bug in how charset.c calls xpalloc,
> but
> >>>>> I'm not sure whether there are other callers that rely on this
> behavior,
> >>>>> so it's safest to work around it.
> >>>>
> >>>> xpalloc handles this case:
> >>>>
> >>>>   if (! pa)
> >>>>     *nitems = 0;
> >>>
> >>> Yeah, I'd rather check the other igc variants of xpalloc, to make sure
> >>> they are compatible with the original, even if no one else uses that
> >>> particular feature.
> >>
> >> That's what I did, I think?
> >>
> >> The code above doesn't have any effect unless we run out of memory (even
> >> then, it won't have an effect if NITEMS is a stack variable in a frame
> >> that's unwound by memory_full).
> >
> > Sorry, I'm currently to unconcentrated to follow. I just wanted to say
> > that it would be good to make the functions are compatible with xpalloc,
> > whatever that means.
>
> Do you want me to look at your patch? I'd rather just rely on you,
> though :-).
>
> And while everyone is on board: I was making emacs-mac use igc
> in my Emacs. Seems to have worked, I'm writing this from
>
>   ELISP> (and (featurep 'mac) (featurep 'mps))
>   t
>
> Only minimal additions to igc.[ch] were necessary because mac uses a
> Lisp and a GUI thread, and so on, so I couldn't use init_igc for mac.
> Two new functions in äifdef HAVE_MAC_GUI instead, igc_init_mac_early and
> _late.
>

Do you think you'll push that igc merged branch up to JD's git repo?

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.