bug#49782: 28.0.50; set-fontset-font crashes in init (server)

[Benjamin Riefenstahl]

Hi Eli,

Eli Zaretskii writes:
> (I cannot produce such a backtrace myself, because your recipe
> involves 3 different processes, one of which forks, which is hard to
> follow with GDB on a system I have here.)

Same problem here.  I tried 'emacsclient -a "gdbserver emacs"', but than
the problem does not reproduce.  Or maybe I was doing something wrong.

Eli Zaretskii writes:
> But before you embark on that adventure, please try another stab in
> the dark:
>
> diff --git a/src/fontset.c b/src/fontset.c
> index d2d428d..7d4bd65 100644
> --- a/src/fontset.c
> +++ b/src/fontset.c
> @@ -1356,15 +1356,16 @@ check_fontset_name (Lisp_Object name, Lisp_Object 
> *frame)
>    int id;
>    struct frame *f = decode_live_frame (*frame);
>  
> -  if (!FRAME_WINDOW_P (f) && !FRAME_INITIAL_P (f))
> -    error ("Can't use fontsets in non-GUI frames");
> -
>    XSETFRAME (*frame, f);
>  
>    if (EQ (name, Qt))
>      return Vdefault_fontset;
>    if (NILP (name))
> -    id = FRAME_FONTSET (f);
> +    {
> +      if (!FRAME_WINDOW_P (f))
> +     error ("Can't use fontsets in non-GUI frames");
> +      id = FRAME_FONTSET (f);
> +    }
>    else
>      {
>        CHECK_STRING (name);

Yes, this works.  It works for running Emacs directly as well as through
'emacsclient -a ""' and 'emacsclient -a "emacs"'.  I think this is it.

Thank you,
benny