Patrick Lam <plam@xxxxxxx> さんは書きました: > Mike FABIAN wrote: >> + goto bail1; > > I think that should be bail1, because cs isn't freed yet. I've > committed with a goto to bail2. I see. Then what about the two "bail1" a few lines up?: if (!(face->face_flags & FT_FACE_FLAG_SCALABLE)) { for (i = 0; i < face->num_fixed_sizes; i++) if (!FcPatternAddDouble (pat, FC_PIXEL_SIZE, FcGetPixelSize (face, i))) goto bail1; if (!FcPatternAddBool (pat, FC_ANTIALIAS, FcFalse)) goto bail1; # Shouldn't those be "bail2" as well? -- Mike FABIAN <mfabian@xxxxxxx> http://www.suse.de/~mfabian 睡眠不足はいい仕事の敵だ。 _______________________________________________ Fontconfig mailing list Fontconfig@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/fontconfig