Re: bug in variable.c: fontpath_handler(ACTION_GET)

[Ethan M. <merritt@u.washington.edu>]

On Monday 24 November 2003 07:22, Petr Mikulik wrote:
> There seems to be a bug in file
> =09variable.c
>
> Below is an ad-hoc fix -- and actually two possibilities of the fix.
> Does someone (Harald?) know what that code is doing and propose a
> correct fix?

This is a parallel case to the crash I was getting.
Mine was in loadpath_handler(), yours in fontpath_handler().
I make the correct fix to be the one given below, but I don't
know why this problem is only showing up now:

--- gnuplot/src/variable.c=092002-10-11 09:26:49.000000000 -0700
+++ gnuplot-eam/src/variable.c=092003-11-24 16:32:16.000000000 -0800
@@ -161,7 +161,7 @@
 =09FPRINTF((stderr, "Get loadpath\n"));
 =09if (!loadpath)
 =09    return NULL;
-=09if (!beenhere) {
+=09if (!beenhere || !p) {
 =09    /* init section */
 =09    beenhere =3D 1;
 =09    p =3D loadpath;
@@ -505,7 +505,7 @@
 =09FPRINTF((stderr, "Get fontpath\n"));
 =09if (!fontpath)
 =09    return NULL;
-=09if (!beenhere) {
+=09if (!beenhere || !p) {
 =09    /* init section */
 =09    beenhere =3D 1;
 =09    p =3D fontpath;


--=20
Ethan A Merritt       merritt@u.washington.edu
Biomolecular Structure Center Box 357742
University of Washington, Seattle, WA 98195