Re: [PATCH/RFA] language.c free(0)

[David Taylor <taylor@cygnus.com>]

    Date: Thu, 16 Mar 100 20:26:56 +0100 (CET)
    From: "Philippe De Muyter" <phdm@macqel.be>

    May I inaugurate my ssh connection by applying the following patch ? Here,
    gdb crashes when one attempts to free a null pointer.

Yes.

    Wed Mar 15 07:04:17 2000  Philippe De Muyter  <phdm@macqel.be>

	    * language.c (set_lang_str): Do not call `free' for a null pointer.
	    (set_type_str, set_range_str): Ditto.

Approved.

    --- ./gdb/language.c    Mon Mar 13 20:01:32 2000
    +++ ./gdb/language.c    Sun Feb 20 03:23:37 2000
    @@ -394,7 +394,8 @@ set_lang_str ()
     {
       char *prefix = "";

    -  free (language);
    +  if (language)
    +    free (language);
       if (language_mode == language_mode_auto)
	 prefix = "auto; currently ";

    @@ -406,7 +407,8 @@ set_type_str ()
     {
       char *tmp = NULL, *prefix = "";

    -  free (type);
    +  if (type)
    +    free (type);
       if (type_mode == type_mode_auto)
	 prefix = "auto; currently ";

    @@ -451,7 +453,8 @@ set_range_str ()
	   error ("Unrecognized range check setting.");
	 }

    -  free (range);
    +  if (range)
    +    free (range);
       range = concat (pref, tmp, NULL);
     }