fontforge/SOURCES/fontforge-20170731-covscan-...

2667 lines
86 KiB
Diff

diff -urN fontforge-20170731.old/fontforge/asmfpst.c fontforge-20170731/fontforge/asmfpst.c
--- fontforge-20170731.old/fontforge/asmfpst.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/asmfpst.c 2018-09-23 14:50:31.558567689 +0530
@@ -810,13 +810,16 @@
match_len = j;
for ( i=0; i<match_len; ++i )
- if ( tables[i]==NULL || tables[i][0]==NULL )
-return( NULL );
-
+ if ( tables[i]==NULL || tables[i][0]==NULL ) {
+ free(tables);
+ return( NULL );
+ }
glyphs = morx_cg_FigureClasses(tables,match_len,
&classes,&class_cnt,&map,&gcnt,fpst,sf,ordered);
- if ( glyphs==NULL )
-return( NULL );
+ if ( glyphs==NULL ) {
+ free(tables);
+ return( NULL );
+ }
for ( i=0; i<match_len; ++i )
free(tables[i]);
diff -urN fontforge-20170731.old/fontforge/autotrace.c fontforge-20170731/fontforge/autotrace.c
--- fontforge-20170731.old/fontforge/autotrace.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/autotrace.c 2018-09-23 14:50:31.558567689 +0530
@@ -474,8 +474,10 @@
++cnt;
for ( start=pt; isspace(*start); ++start);
}
- if ( cnt==0 )
-return( NULL );
+ if ( cnt==0 ) {
+ free(vector);
+ return( NULL );
+ }
if ( vector ) {
vector[cnt] = NULL;
return( vector );
@@ -789,17 +791,17 @@
strcat(arglist[1],filename);
if ( (pid=fork())==0 ) {
/* Child */
- int fd;
+ int fdw, fdr;
chdir(tempdir);
if ( !mf_showerrors ) {
close(1); /* mf generates a lot of verbiage to stdout. Throw it away */
- fd = open("/dev/null",O_WRONLY);
- if ( fd!=1 )
- dup2(fd,1);
+ fdw = open("/dev/null",O_WRONLY);
+ if ( fdw!=1 )
+ dup2(fdw,1);
close(0); /* mf sometimes asks the user questions, but I have no answers... */
- fd = open("/dev/null",O_RDONLY);
- if ( fd!=0 )
- dup2(fd,0);
+ fdr = open("/dev/null",O_RDONLY);
+ if ( fdr!=0 )
+ dup2(fdr,0);
}
exit(execvp(arglist[0],arglist)==-1); /* If exec fails, then die */
} else if ( pid!=-1 ) {
@@ -836,6 +838,7 @@
ff_post_error(_("Can't run mf"),_("Can't run mf"));
free(arglist[1]);
cleantempdir(tempdir);
+ free(tempdir);
return( sf );
#endif
}
diff -urN fontforge-20170731.old/fontforge/bitmapchar.c fontforge-20170731/fontforge/bitmapchar.c
--- fontforge-20170731.old/fontforge/bitmapchar.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/bitmapchar.c 2018-09-23 14:50:31.558567689 +0530
@@ -371,6 +371,8 @@
}
}
}
+ free(bpt);
+ free(pt);
}
BDFProperties *BdfPropsCopy(BDFProperties *props, int cnt ) {
diff -urN fontforge-20170731.old/fontforge/cvimages.c fontforge-20170731/fontforge/cvimages.c
--- fontforge-20170731.old/fontforge/cvimages.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/cvimages.c 2018-09-23 14:50:31.558567689 +0530
@@ -223,11 +223,15 @@
break;
if ( ch!='(' ) {
ff_post_error( _("Not a plate file"), _("This does not seem to be a plate file\nExpected left paren"));
+ free(head);
+ free(spiros);
return;
}
ch = getc(plate);
if ( ch!='v' && ch!='o' && ch!='c' && ch!='[' && ch!=']' && ch!='z' ) {
ff_post_error( _("Not a plate file"), _("This does not seem to be a plate file\nExpected one of 'voc[]z'"));
+ free(head);
+ free(spiros);
return;
}
if ( cnt>=max )
@@ -251,6 +255,8 @@
} else {
if ( fscanf(plate,"%lg %lg )", &spiros[cnt].x, &spiros[cnt].y)!=2 ) {
ff_post_error( _("Not a plate file"), _("This does not seem to be a plate file\nExpected two real numbers"));
+ free(head);
+ free(spiros);
return;
}
++cnt;
diff -urN fontforge-20170731.old/fontforge/cvundoes.c fontforge-20170731/fontforge/cvundoes.c
--- fontforge-20170731.old/fontforge/cvundoes.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/cvundoes.c 2018-09-23 14:50:31.559567674 +0530
@@ -2637,6 +2637,7 @@
adjust->yadv.corrections = malloc(adjust->yadv.last_pixel_size-adjust->yadv.first_pixel_size+1);
memcpy(adjust->yadv.corrections,vr->adjust->yadv.corrections,adjust->yadv.last_pixel_size-adjust->yadv.first_pixel_size+1);
}
+ free(adjust);
}
static void PSTInto(SplineChar *sc,PST *pst,PST *frompst, struct lookup_subtable *sub) {
@@ -3898,6 +3899,7 @@
{
char* str = UndoToString( sc, undo );
printf("\n\n*** undo: %d\n%s\n", idx, str );
+ free(str);
}
printf("dumpUndoChain(end) %s\n", msg );
}
diff -urN fontforge-20170731.old/fontforge/encoding.c fontforge-20170731/fontforge/encoding.c
--- fontforge-20170731.old/fontforge/encoding.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/encoding.c 2018-09-23 14:50:31.560567658 +0530
@@ -757,6 +757,7 @@
if ( item->enc_name==NULL ) {
if ( no_windowing_ui ) {
ff_post_error(_("Bad encoding file format"),_("This file contains an unnamed encoding, which cannot be named in a script"));
+ free(head);
return( NULL );
}
if ( item==head && item->next==NULL )
diff -urN fontforge-20170731.old/fontforge/featurefile.c fontforge-20170731/fontforge/featurefile.c
--- fontforge-20170731.old/fontforge/featurefile.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/featurefile.c 2018-09-23 14:50:31.561567642 +0530
@@ -1620,9 +1620,10 @@
glyphs[lcnt].sc = NULL;
}
- if ( !needsclasses && lcnt==0 && sf->mark_class_cnt==0 )
+ if ( !needsclasses && lcnt==0 && sf->mark_class_cnt==0 ){
+ free(glyphs);
return; /* No anchor positioning, no ligature carets */
-
+ }
if ( sf->mark_class_cnt!=0 ) {
fprintf( out, "#Mark attachment classes (defined in GDEF, used in lookupflags)\n" );
for ( i=1; i<sf->mark_class_cnt; ++i ) {
@@ -1799,9 +1800,12 @@
}
}
}
+ free(langs);
}
}
+ free(scripts);
}
+ free(feats);
}
if (has_DFLT) { dump_header_languagesystem_hash_fe((gpointer)"DFLT dflt", (gpointer)"", (gpointer)out); }
g_tree_foreach( ht, dump_header_languagesystem_hash_fe, out );
@@ -3259,15 +3263,18 @@
LogError(_("Expected '=' in glyph class definition on line %d of %s"), tok->line[tok->inc_depth], tok->filename[tok->inc_depth] );
++tok->err_count;
fea_skip_to_semi(tok);
+ free(classname);
return;
}
fea_ParseTok(tok);
contents = fea_ParseGlyphClass(tok); // Make a list of referenced glyphs.
if ( contents==NULL ) {
fea_skip_to_semi(tok);
+ free(contents);
return;
}
fea_AddClassDef(tok,classname,copy(contents)); // Put the list into a class.
+ free(contents);
fea_end_statement(tok);
}
@@ -3517,6 +3524,7 @@
LogError(_("Expected name in anchor definition on line %d of %s"), tok->line[tok->inc_depth], tok->filename[tok->inc_depth] );
++tok->err_count;
fea_skip_to_semi(tok);
+ free(ap);
return;
}
for ( nap=tok->namedAnchors; nap!=NULL; nap=nap->next )
@@ -3681,6 +3689,7 @@
LogError(_("Expected class name in mark class definition on line %d of %s"), tok->line[tok->inc_depth], tok->filename[tok->inc_depth] );
++tok->err_count;
fea_skip_to_semi(tok);
+ free(ap);
return;
}
gm = chunkalloc(sizeof(*gm));
@@ -7092,6 +7101,7 @@
}
if ( f!=NULL && f->type == ft_feat_end )
f = f->next;
+ free(sl);
return( f );
}
diff -urN fontforge-20170731.old/fontforge/glyphcomp.c fontforge-20170731/fontforge/glyphcomp.c
--- fontforge-20170731.old/fontforge/glyphcomp.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/glyphcomp.c 2018-09-23 14:50:31.562567627 +0530
@@ -1414,7 +1414,10 @@
static void NameCompare(struct font_diff *fd,const char *name1, const char *name2, char *id) {
- if (!name1) name1=""; if (!name2) name2="";
+ if (!name1)
+ name1="";
+ if (!name2)
+ name2="";
if ( strcmp(name1,name2)!=0 ) {
if ( !fd->top_diff )
fprintf( fd->diffs, "Names\n" );
@@ -1442,7 +1445,10 @@
int lang,int strid) {
char strnamebuf[200];
- if (!name1) name1=""; if (!name2) name2="";
+ if (!name1)
+ name1="";
+ if (!name2)
+ name2="";
if ( strcmp(name1,name2)==0 )
return;
sprintf( strnamebuf, "%.90s %.90s", TTFNameIds(strid), MSLangString(lang));
diff -urN fontforge-20170731.old/fontforge/http.c fontforge-20170731/fontforge/http.c
--- fontforge-20170731.old/fontforge/http.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/http.c 2018-09-23 14:50:31.562567627 +0530
@@ -540,7 +540,9 @@
if ( !findFTPhost(&addr, host, port)) {
ff_progress_end_indicator();
ff_post_error(_("Could not find host"),_("Could not find \"%s\"\nAre you connected to the internet?"), host );
- free( host ); free( filename );
+ free( host );
+ free( filename );
+ free(password);
return( false );
}
soc = makeConnection(&addr);
diff -urN fontforge-20170731.old/fontforge/lookups.c fontforge-20170731/fontforge/lookups.c
--- fontforge-20170731.old/fontforge/lookups.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/lookups.c 2018-09-23 14:50:31.562567627 +0530
@@ -4746,20 +4746,26 @@
break;
start = lpt;
if ( *start=='|' ) {
- if ( fpst->type==pst_contextpos || fpst->type==pst_contextsub )
-return( xasprintf( _("Separation marks only meaningful in contextual chaining lookups, starting at: %.20s..."), lpt ));
+ if ( fpst->type==pst_contextpos || fpst->type==pst_contextsub ){
+ free(parsed);
+ return( xasprintf( _("Separation marks only meaningful in contextual chaining lookups, starting at: %.20s..."), lpt ));
+ }
if ( first==-1 )
first = cnt;
else if ( last==-1 )
last = cnt-1;
- else
-return( xasprintf( _("Too many separation marks, starting at: %.20s..."), lpt ));
+ else{
+ free(parsed);
+ return( xasprintf( _("Too many separation marks, starting at: %.20s..."), lpt ));
+ }
++lpt;
continue;
} else if ( *start=='[' ) {
/* A coverage table */
- if ( fpst->format!=pst_coverage && fpst->format!=pst_reversecoverage )
-return( xasprintf( _("A coverage table was found in a glyph or class based contextual lookup, starting at: %.20s..."), lpt ));
+ if ( fpst->format!=pst_coverage && fpst->format!=pst_reversecoverage ){
+ free(parsed);
+ return( xasprintf( _("A coverage table was found in a glyph or class based contextual lookup, starting at: %.20s..."), lpt ));
+ }
++start;
for ( lpt = start; *lpt!='\0' && *lpt!=']'; ++lpt );
if ( *lpt!=']' )
@@ -4788,9 +4794,10 @@
}
free(parsed[cnt].replacements);
parsed[cnt].replacements = newr;
- } else
-return( xasprintf( _("There must be as many replacement glyphs as there are match glyphs: %s => %s"),
+ } else{
+ return( xasprintf( _("There must be as many replacement glyphs as there are match glyphs: %s => %s"),
parsed[cnt].entity, parsed[cnt].replacements));
+ }
continue;
}
} else if ( *start!='@' && *start!='<' && !(*start=='=' && start[1]=='>') ) {
@@ -4802,14 +4809,18 @@
/* A reverse contextual chaining */
if ( fpst->format!=pst_reversecoverage )
return( xasprintf( _("No replacement lists may be specified in this contextual lookup, use a nested lookup instead, starting at: %.20s..."), lpt ));
- if ( do_replacements )
-return( xasprintf( _("Only one replacement list may be specified in a reverse contextual chaining lookup, starting at: %.20s..."), lpt ));
+ if ( do_replacements ) {
+ free(parsed);
+ return( xasprintf( _("Only one replacement list may be specified in a reverse contextual chaining lookup, starting at: %.20s..."), lpt ));
+ }
do_replacements = true;
lpt += 2;
} else {
/* A lookup invocation */
- if ( fpst->format==pst_reversecoverage )
-return( xasprintf( _("No lookups may be specified in a reverse contextual lookup (use a replacement list instead), starting at: %.20s..."), lpt ));
+ if ( fpst->format==pst_reversecoverage ){
+ free(parsed);
+ return( xasprintf( _("No lookups may be specified in a reverse contextual lookup (use a replacement list instead), starting at: %.20s..."), lpt ));
+ }
if ( *start=='@' ) {
for ( lpt=start+1; isspace( *lpt ); ++lpt );
@@ -4867,8 +4878,10 @@
ret = NULL;
if ( !do_replacements && !anylookup ) {
- if ( fpst->format==pst_reversecoverage )
-return( copy( _("A reverse contextual chaining lookup must have a set of replacement glyphs somewhere" )) );
+ if ( fpst->format==pst_reversecoverage ) {
+ free(parsed);
+ return( copy( _("A reverse contextual chaining lookup must have a set of replacement glyphs somewhere" )) );
+ }
else {
*return_is_warning = true;
/* If there are no lookups then this rule matches and does nothing */
@@ -4884,8 +4897,10 @@
if ( first==-1 ) first = i;
}
if ( parsed[i].replacements!=NULL ) {
- if (( first!=-1 && first!=i ) || (last!=-1 && last!=i ))
-return( copy( _("A reverse contextual chaining lookup can only match one coverage table directly" )) );
+ if (( first!=-1 && first!=i ) || (last!=-1 && last!=i )){
+ free(parsed);
+ return( copy( _("A reverse contextual chaining lookup can only match one coverage table directly" )) );
+ }
first = last = i;
}
}
@@ -4977,11 +4992,13 @@
free( rule->u.class.fclasses ); rule->u.class.fclasses = NULL;
rule->u.class.bcnt = rule->u.class.fcnt = rule->u.class.ncnt = 0;
if ( i<first )
-return( xasprintf( _("%s is not a class name for the backtracking classes." ), parsed[i].entity ) );
- else if ( i<=last )
-return( xasprintf( _("%s is not a class name for the matching classes." ), parsed[i].entity ) );
+ return( xasprintf( _("%s is not a class name for the backtracking classes." ), parsed[i].entity ) );
+ else if ( i<=last ) {
+ free(ret);
+ return( xasprintf( _("%s is not a class name for the matching classes." ), parsed[i].entity ) );
+ }
else
-return( xasprintf( _("%s is not a class name for the forward classes." ), parsed[i].entity ) );
+ return( xasprintf( _("%s is not a class name for the forward classes." ), parsed[i].entity ) );
}
if ( i<first )
rule->u.class.bclasses[first-1-i] = j; /* Reverse the backtrack classes */
diff -urN fontforge-20170731.old/fontforge/macbinary.c fontforge-20170731/fontforge/macbinary.c
--- fontforge-20170731.old/fontforge/macbinary.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/macbinary.c 2018-09-23 14:50:31.563567611 +0530
@@ -299,7 +299,8 @@
for (;;) {
if ( getc(pfbfile)!=0x80 ) {
IError("We made a pfb file, but didn't get one. Hunh?" );
-return( NULL );
+ free(resstarts);
+ return( NULL );
}
type = getc(pfbfile);
if ( type==3 ) {
@@ -1833,8 +1834,12 @@
strcpy(pt-1,".fam.bin");
#endif
}
- if ( WriteMacPSFont(tempname,sfi->sf,format,flags,sfi->map,layer)==0 )
-return( 0 );
+ if ( WriteMacPSFont(tempname,sfi->sf,format,flags,sfi->map,layer)==0 ){
+ free(filename);
+ free(tempname);
+ return( 0 );
+ }
+
free(tempname);
}
} else if ( format!=ff_none || bf==bf_sfnt_dfont ) {
@@ -1861,7 +1866,8 @@
if ( res==NULL ) {
for ( sfsub=sfs; sfsub!=NULL; sfsub=sfsub->next )
fclose( sfsub->tempttf );
-return( 0 );
+ free(filename);
+ return( 0 );
}
if ( format==ff_ttfdfont || format==ff_otfdfont || format==ff_otfciddfont ||
@@ -1982,8 +1988,9 @@
if ( pfb==NULL ) {
LogError( _("Can't open temporary file for postscript output\n") );
fseek(f,here,SEEK_SET );
+ free(rsrcids);
free(offsets);
-return(NULL);
+ return(NULL);
}
putc(0x80,pfb);
@@ -2578,7 +2585,9 @@
char *pt, *lparen;
char *find = NULL;
- if ((pt = strrchr(filename,'/'))!=NULL ) pt = filename;
+ if ((pt = strrchr(filename,'/'))!=NULL )
+ pt = filename;
+
if ( (lparen = strchr(filename,'('))!=NULL && strchr(lparen,')')!=NULL ) {
find = copy(lparen+1);
pt = strchr(find,')');
@@ -2588,6 +2597,8 @@
if ( test->psnames[i]!=NULL && strcmp(find,test->psnames[i])==0 ) {
*style = (i&3) | ((i&~3)<<1); /* PS styles skip underline bit */
*name = copy(test->psnames[i]);
+ free(find);
+ free(pt);
return( test );
}
}
@@ -2670,8 +2681,10 @@
SplineChar *sc;
fond = PickFOND(fondlist,filename,&name,&style);
- if ( fond==NULL )
-return( NULL );
+ if ( fond==NULL ){
+ free(name);
+ return( NULL );
+ }
find_id=-1;
if ( flags&ttf_onlyonestrike ) {
@@ -2747,8 +2760,11 @@
SplineChar *sc1, *sc2;
fond = PickFOND(fondlist,filename,&name,&style);
- if ( fond==NULL || into==NULL )
-return( NULL );
+ if ( fond==NULL || into==NULL ){
+ free(name);
+ return( NULL );
+ }
+
for ( i=0; i<fond->stylekerncnt; ++i )
if ( fond->stylekerns[i].style==style )
break;
@@ -2780,6 +2796,7 @@
}
kp->off = offset;
}
+ free(name);
return( into );
}
@@ -2795,7 +2812,11 @@
char **ret;
char *temp = TTFGetFontName(binary,pos,pos);
if ( temp==NULL )
-return( NULL );
+ {
+ free(temp);
+ free(buffer);
+ return( NULL );
+ }
ret = malloc(2*sizeof(char *));
ret[0] = temp;
ret[1] = NULL;
diff -urN fontforge-20170731.old/fontforge/palmfonts.c fontforge-20170731/fontforge/palmfonts.c
--- fontforge-20170731.old/fontforge/palmfonts.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/palmfonts.c 2018-09-23 14:50:31.563567611 +0530
@@ -178,6 +178,7 @@
BCCompressBitmap(bdfc);
}
}
+ free(fontImage);
}
static SplineFont *PalmTestFont(FILE *file,int end, char *family,const char *style) {
@@ -440,6 +441,7 @@
putlong(file,0);
putlong(file,0);
}
+ free(pt1);
return(file);
}
diff -urN fontforge-20170731.old/fontforge/parsepdf.c fontforge-20170731/fontforge/parsepdf.c
--- fontforge-20170731.old/fontforge/parsepdf.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/parsepdf.c 2018-09-23 14:50:31.564567596 +0530
@@ -974,8 +974,11 @@
while ( prev_xref!=-1 ) {
if ( fseek(pdf,prev_xref,SEEK_SET)!=0 ) return( NULL );
pdf_skipobjectheader(pc);
- if ( !pdf_readdict(pc))
-return( NULL );
+ if ( !pdf_readdict(pc)){
+ free(gen);
+ free(ret);
+ return( NULL );
+ }
if ( (pt=PSDictHasEntry(&pc->pdfdict,"Type"))==NULL || strcmp(pt,"/XRef")!=0 )
return( NULL );
if ( (pt=PSDictHasEntry(&pc->pdfdict,"Size"))==NULL )
@@ -2069,6 +2072,7 @@
pcFree(&pc);
fclose(pc.pdf);
switch_to_old_locale(&tmplocale, &oldlocale); // Switch to the cached locale.
+ free(list);
return( NULL );
}
diff -urN fontforge-20170731.old/fontforge/parsettfatt.c fontforge-20170731/fontforge/parsettfatt.c
--- fontforge-20170731.old/fontforge/parsettfatt.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/parsettfatt.c 2018-09-23 14:50:31.564567596 +0530
@@ -790,6 +790,7 @@
if ( feof(ttf) ) {
LogError( _("Bad mark table.\n") );
info->bad_ot = true;
+ free(classes);
return( NULL );
}
@@ -1386,6 +1387,7 @@
if ( glyphs==NULL ) {
/* GT: This continues a multi-line error message, hence the leading space */
LogError( _(" Bad contextual substitution table, ignored\n") );
+ free(class);
free(rules);
return;
}
@@ -1543,6 +1545,7 @@
if ( glyphs==NULL ) {
/* GT: This continues a multi-line error message, hence the leading space */
LogError( _(" Bad contextual chaining substitution table, ignored\n") );
+ free(class);
free(rules);
return;
}
@@ -2443,6 +2446,7 @@
if ( pos+scripts[i].languages[j].offset>=info->g_bounds ) {
LogError(_("Attempt to read script data beyond end of %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(scripts);
return( NULL );
}
fseek(ttf,pos+scripts[i].languages[j].offset,SEEK_SET);
@@ -2452,6 +2456,7 @@
if ( feof(ttf)) {
LogError(_("End of file when reading scripts in %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(scripts);
return( NULL );
}
scripts[i].languages[j].features = malloc(scripts[i].languages[j].fcnt*sizeof(uint16));
@@ -2463,6 +2468,7 @@
if ( feof(ttf)) {
LogError(_("End of file in %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(scripts);
return( NULL );
}
@@ -2502,6 +2508,7 @@
if ( pos+features[i].offset>=info->g_bounds ) {
LogError(_("Attempt to read feature data beyond end of %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(features);
return( NULL );
}
fseek(ttf,pos+features[i].offset,SEEK_SET);
@@ -2518,6 +2525,7 @@
if ( feof(ttf) ) {
LogError(_("End of file when reading features in %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(features);
return( NULL );
}
features[i].lookups = malloc(features[i].lcnt*sizeof(uint16));
@@ -2560,6 +2568,7 @@
LogError(_("Attempt to read lookup data beyond end of %s table"),
isgpos==2? "JSTF" : isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(lookups);
return( NULL );
}
fseek(ttf,pos+lookups[i].offset,SEEK_SET);
@@ -2585,6 +2594,7 @@
if ( feof(ttf) ) {
LogError(_("End of file when reading lookups in %s table"), isgpos ? "GPOS" : "GSUB" );
info->bad_ot = true;
+ free(lookups);
return( NULL );
}
for ( j=0; j<lookups[i].subtabcnt; ++j ) {
@@ -6063,6 +6073,7 @@
LogError( _("JSTF table is too long.\n") );
info->bad_ot = true;
free(soff);
+ free(loff);
return;
}
diff -urN fontforge-20170731.old/fontforge/parsettfbmf.c fontforge-20170731/fontforge/parsettfbmf.c
--- fontforge-20170731.old/fontforge/parsettfbmf.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/parsettfbmf.c 2018-09-23 14:50:31.565567580 +0530
@@ -91,8 +91,8 @@
/* Only use the provided metrics if it hasn't already been set */
if (metrics == NULL)
metrics = &big;
- if ( imageformat==8 )
- /* pad = */ getc(ttf);
+ if ( imageformat==8 )
+ /* pad = */ getc(ttf);
} else if ( imageformat==6 || imageformat==7 || imageformat==9 ) {
big.height = getc(ttf);
big.width = getc(ttf);
diff -urN fontforge-20170731.old/fontforge/parsettf.c fontforge-20170731/fontforge/parsettf.c
--- fontforge-20170731.old/fontforge/parsettf.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/parsettf.c 2018-09-23 14:50:31.566567564 +0530
@@ -1972,7 +1972,8 @@
info->bad_glyph_data = true;
LogError( _("Bad tt font: contour ends make no sense in glyph %d.\n"),
sc->orig_pos );
-return;
+ free(endpt);
+ return;
}
}
if ( path_cnt==0 ) {
diff -urN fontforge-20170731.old/fontforge/parsettfvar.c fontforge-20170731/fontforge/parsettfvar.c
--- fontforge-20170731.old/fontforge/parsettfvar.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/parsettfvar.c 2018-09-23 14:50:31.567567549 +0530
@@ -747,6 +747,7 @@
ti = -1;
if ( k!=info->variations->tuple_count )
ti = k;
+ free(coords);
}
if ( ti!=-1 ) {
here = ftell(ttf);
diff -urN fontforge-20170731.old/fontforge/plugins.c fontforge-20170731/fontforge/plugins.c
--- fontforge-20170731.old/fontforge/plugins.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/plugins.c 2018-09-23 14:50:31.567567549 +0530
@@ -35,6 +35,7 @@
int LoadPlugin(const char *dynamic_lib_name) {
lt_dlhandle plugin;
plugin = load_plugin(dynamic_lib_name, LogError);
+ free(plugin);
return (plugin != NULL);
}
diff -urN fontforge-20170731.old/fontforge/print.c fontforge-20170731/fontforge/print.c
--- fontforge-20170731.old/fontforge/print.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/print.c 2018-09-23 14:50:31.568567533 +0530
@@ -2894,6 +2894,7 @@
}
if ( pt>start && i<sizeof(argv)/sizeof(argv[0])-1 )
argv[i++] = start;
+ free(temp);
}
argv[i] = NULL;
/*for ( i=0; argv[i]!=NULL; ++i ) printf( "%s ", argv[i]); printf("\n" );*/
diff -urN fontforge-20170731.old/fontforge/psread.c fontforge-20170731/fontforge/psread.c
--- fontforge-20170731.old/fontforge/psread.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/psread.c 2018-09-23 14:50:31.569567518 +0530
@@ -295,7 +295,9 @@
wrapper->top = io->prev;
if ( io->isstopped )
wrapper->endedstopped = true;
- if (io->start != NULL) free(io->start); io->start = NULL;
+ if (io->start != NULL)
+ free(io->start);
+ io->start = NULL;
free(io);
io = wrapper->top;
}
@@ -353,7 +355,9 @@
while ( io->prev!=NULL && !io->isstopped ) {
iop = io->prev;
wasloop = io->isloop;
- if (io->start != NULL) free(io->start); io->start = NULL;
+ if (io->start != NULL)
+ free(io->start);
+ io->start = NULL;
free(io);
if ( wasloop ) {
wrapper->top = iop;
@@ -377,7 +381,9 @@
while ( io->prev!=NULL ) {
iop = io->prev;
wasstopped = io->isstopped;
- if (io->start != NULL) free(io->start); io->start = NULL;
+ if (io->start != NULL)
+ free(io->start);
+ io->start = NULL;
free(io);
if ( wasstopped ) {
wrapper->top = iop;
@@ -2292,8 +2298,14 @@
int mx, mn;
real h, s, b;
mx = mn = r;
- if ( mx>g ) mn=g; else mx=g;
- if ( mx<bl ) mx = bl; if ( mn>bl ) mn = bl;
+ if ( mx>g )
+ mn=g;
+ else
+ mx=g;
+ if ( mx<bl )
+ mx = bl;
+ if ( mn>bl )
+ mn = bl;
b = mx/255.;
s = h = 0;
if ( mx>0 )
@@ -3364,8 +3376,10 @@
encname = copy(tokbuf);
tok = nextpstoken(&wrapper,&dval,tokbuf,sizeof(tokbuf));
}
- if ( tok!=pt_openarray && tok!=pt_opencurly )
+ if ( tok!=pt_openarray && tok!=pt_opencurly ){
+ free(encname);
return( head );
+ }
for ( i=0; i<sizeof(names)/sizeof(names[0]); ++i ) {
encs[i] = -1;
names[i]=NULL;
@@ -4675,5 +4689,7 @@
HintsRenumber(ret);
if ( name!=NULL && strcmp(name,".notdef")!=0 )
ret->widthset = true;
+ free(pending_hm);
+ free(pt);
return( ret );
}
diff -urN fontforge-20170731.old/fontforge/python.c fontforge-20170731/fontforge/python.c
--- fontforge-20170731.old/fontforge/python.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/python.c 2018-09-23 14:50:31.572567471 +0530
@@ -1183,6 +1183,7 @@
tuple = PyTuple_New(cnt);
for ( i=0; i<cnt; ++i )
PyTuple_SetItem(tuple,i,Py_BuildValue("s",ret[i]));
+ free(ret);
return( tuple );
}
@@ -1618,6 +1619,7 @@
PyErr_Format(PyExc_ValueError, "Value out of bounds for 4th argument");
free(title);
free(quest);
+ free(answers);
return( NULL );
}
for ( i=0; i<cnt; ++i ) {
@@ -2187,10 +2189,12 @@
ss = SSFromContour(self,NULL);
if ( ss==NULL ) {
PyErr_SetString(PyExc_AttributeError, "Empty Contour");
+ free(ss);
return( NULL );
}
self->spiros = SplineSet2SpiroCP(ss,&cnt);
self->spiro_cnt = cnt;
+ free(ss);
}
spirotuple = PyTuple_New(self->spiro_cnt-1);
for ( i=0; i<self->spiro_cnt-1; ++i ) {
@@ -2893,7 +2897,10 @@
for ( i=0; i<PySequence_Size(args); ++i ) {
pos = PyInt_AsLong(PySequence_GetItem(args,i));
if ( PyErr_Occurred())
-return( NULL );
+ {
+ free(ss);
+ return( NULL );
+ }
SSSelectOnCurve(ss,pos);
}
SplineCharMerge(NULL,&ss,1);
@@ -2960,8 +2967,10 @@
smpl.linefixup = PyFloat_AsDouble( PySequence_GetItem(args,3));
if ( !PyErr_Occurred() && PySequence_Size(args)>=5 )
smpl.linelenmax = PyFloat_AsDouble( PySequence_GetItem(args,4));
- if ( PyErr_Occurred() )
+ if ( PyErr_Occurred() ){
+ free(ss);
return( NULL );
+ }
SplinePointListSimplify(NULL,ss,&smpl);
if ( ss==NULL ) {
for ( i=0; i<self->pt_cnt; ++i )
@@ -3292,6 +3301,7 @@
do_pycall(pen,"qCurveTo",tuple);
} else {
PyErr_Format(PyExc_TypeError, "Wrong number of off-curve points on a cubic contour");
+ free(points);
return( NULL );
}
if ( PyErr_Occurred())
@@ -3324,6 +3334,7 @@
if ( PyErr_Occurred())
return( NULL );
+free(points);
Py_RETURN( self );
}
@@ -3612,6 +3623,7 @@
ss2 = SSFromLayer((PyFF_Layer *) other);
} else {
PyErr_Format(PyExc_TypeError, "Unexpected type");
+ free(ss);
return( -1 );
}
ret = SSsCompare(ss,ss2,pt_err,spline_err,&badpoint);
@@ -3880,8 +3892,10 @@
smpl.linefixup = PyFloat_AsDouble( PySequence_GetItem(args,3));
if ( !PyErr_Occurred() && PySequence_Size(args)>=5 )
smpl.linelenmax = PyFloat_AsDouble( PySequence_GetItem(args,4));
- if ( PyErr_Occurred() )
+ if ( PyErr_Occurred() ){
+ free(ss);
return( NULL );
+ }
ss = SplineCharSimplify(NULL,ss,&smpl);
LayerFromSS(ss,self);
SplinePointListsFree(ss);
@@ -4187,8 +4201,10 @@
SplinePointListsFree(ss);
ss = temp;
}
- if ( !PolyCheck(ss))
+ if ( !PolyCheck(ss)){
+ free(ss);
return( -1 );
+ }
si->poly = ss;
} else if ( width<=0 || minor<=0 ) {
PyErr_Format(PyExc_ValueError, "Stroke width must be positive" );
@@ -4765,7 +4781,9 @@
++nexti;
if ( c->points[nexti]->on_curve ) {
PyErr_Format(PyExc_TypeError, "In cubic splines there must be exactly 2 control points between on curve points");
-return( NULL );
+ free(ss);
+ free(sp);
+ return( NULL );
}
if ( nexti==c->pt_cnt-1 )
nexti = 0;
@@ -4773,7 +4791,9 @@
++nexti;
if ( !c->points[nexti]->on_curve ) {
PyErr_Format(PyExc_TypeError, "In cubic splines there must be exactly 2 control points between on curve points");
-return( NULL );
+ free(sp);
+ free(ss);
+ return( NULL );
}
}
if ( ss->last==NULL ) {
@@ -4785,6 +4805,7 @@
}
if ( ss->last==NULL ) {
PyErr_Format(PyExc_TypeError, "Empty contour");
+ free(ss);
return( NULL );
}
}
@@ -6064,8 +6085,10 @@
PyErr_Format(PyExc_TypeError, "Unexpected type");
return( -1 );
}
- if ( self->sc->layers[self->layer].refs!=NULL )
+ if ( self->sc->layers[self->layer].refs!=NULL ){
+ free(ss2);
return( SS_NoMatch | SS_RefMismatch );
+ }
ret = SSsCompare(self->sc->layers[self->layer].splines,ss2,pt_err,spline_err,&badpoint);
SplinePointListsFree(ss2);
return(ret);
@@ -6274,8 +6297,10 @@
uni = 0; vs = -1; fid = 0;
if ( PyInt_Check(obj))
uni = PyInt_AsLong(obj);
- else if ( !PyArg_ParseTuple(obj,"i|ii", &uni, &vs, &fid))
+ else if ( !PyArg_ParseTuple(obj,"i|ii", &uni, &vs, &fid)){
+ free(head);
return( -1 );
+ }
cur = chunkalloc(sizeof(struct altuni));
if ( vs==0 ) vs=-1; /* convention used in charinfo */
cur->unienc = uni; cur->vs = vs; cur->fid = fid;
@@ -6738,7 +6763,10 @@
return( -1 );
for ( i=0; i<cnt; ++i ) {
if ( !PyArg_ParseTuple(PySequence_GetItem(value,i),"dd", &start, &width ))
+ {
+ free(tail);
return( -1 );
+ }
cur = chunkalloc(sizeof(StemInfo));
if ( width==-20 || width==-21 )
cur->ghost = true;
@@ -7036,8 +7064,10 @@
for ( i=0; i<PySequence_Size(value); ++i ) {
ap = APFromTuple(sc,PySequence_GetItem(value,i));
- if ( ap==NULL )
-return( -1 );
+ if ( ap==NULL ){
+ free(aplast);
+ return( -1 );
+ }
if ( aphead==NULL )
aphead = ap;
else
@@ -7250,6 +7280,7 @@
self->sc->vert_variants = chunkalloc(sizeof(struct glyphvariants));
self->sc->vert_variants->variants = copy(str);
}
+ free(str);
return( 0 );
}
@@ -7276,6 +7307,7 @@
self->sc->horiz_variants = chunkalloc(sizeof(struct glyphvariants));
self->sc->horiz_variants->variants = copy(str);
}
+ free(str);
return( 0 );
}
@@ -7761,6 +7793,7 @@
SCCharChangedUpdate(sc,((PyFF_Glyph *) self)->layer);
+ free(ap);
Py_RETURN( self );
}
@@ -8137,6 +8170,7 @@
}
if ( deltalen==0 ) {
PyErr_Format(PyExc_TypeError,"Glyph name strings may not be empty");
+ free(str);
return( NULL );
}
len += deltalen+1;
@@ -8575,6 +8609,7 @@
old = NULL;
} else {
PyErr_Format(PyExc_KeyError, "Unexpected lookup type: %s",sub->lookup->lookup_name);
+ free(other);
return( NULL );
}
temp.u.subs.variant = other;
@@ -13659,6 +13694,7 @@
ext = locfilename+len-2;
if ( ext[0]!='p' || ext[1]!='k' ) {
PyErr_Format(PyExc_EnvironmentError, "No extension for bitmap font");
+ free(ext);
return( NULL );
}
}
@@ -13673,7 +13709,8 @@
back = true;
} else {
PyErr_Format(PyExc_EnvironmentError, "Bad extension for bitmap font");
-return( NULL );
+ free(ext);
+ return( NULL );
}
if ( to_background!=-1 )
back = to_background;
@@ -13732,8 +13769,10 @@
free(locfilename);
return( NULL );
}
- if ( CheckIfFontClosed(other) )
+ if ( CheckIfFontClosed(other) ){
+ free(locfilename);
return (NULL);
+ }
flags = FlagsFromTuple(flagstuple,compflags,"comparison flag");
if ( flags==FLAG_UNKNOWN ) {
free(locfilename);
@@ -14316,6 +14355,8 @@
if ( offsets!=NULL ) {
if ( cnt1*cnt2 != PySequence_Size(offsets) ) {
PyErr_Format(PyExc_ValueError, "There aren't enough kerning offsets for the number of kerning classes. Should be %d", cnt1*cnt2 );
+ free(class1_strs);
+ free(class2_strs);
return( NULL );
}
offs = malloc(cnt1*cnt2*sizeof(int16));
@@ -14323,7 +14364,9 @@
offs[i] = PyInt_AsLong(PySequence_GetItem(offsets,i));
if ( PyErr_Occurred()) {
free(offs);
-return( NULL );
+ free(class1_strs);
+ free(class2_strs);
+ return( NULL );
}
}
} else
@@ -14333,6 +14376,8 @@
sub = addLookupSubtable(sf, lookup, subtable, after_str);
if ( sub==NULL ) {
free(offs);
+ free(class1_strs);
+ free(class2_strs);
return( NULL );
}
if ( sub->lookup->lookup_type!=gpos_pair ) {
@@ -14407,6 +14452,8 @@
cnt2 = ParseClassNames(class2s,&class2_strs);
if ( cnt1*cnt2 != PySequence_Size(offsets) ) {
PyErr_Format(PyExc_ValueError, "There aren't enough kerning offsets for the number of kerning classes. Should be %d", cnt1*cnt2 );
+ free(class1_strs);
+ free(class2_strs);
return( NULL );
}
offs = malloc(cnt1*cnt2*sizeof(int16));
@@ -15266,6 +15313,7 @@
mcnt = ParseClassNames(mclasses,&matchclasses);
if ( mcnt==-1 ) {
PyErr_Format(PyExc_TypeError, "Bad match class" );
+ free(matchclasses);
return( NULL );
}
if ( mclassnames!=NULL ) {
@@ -15305,6 +15353,7 @@
new_subtable = addLookupSubtable(sf, lookup, subtable, after_str);
if ( new_subtable==NULL ) {
free(backclassnames); free(matchclasses); free(forclasses);
+ free(matchclassnames);
return( NULL );
}
fpst = chunkalloc(sizeof(FPST));
@@ -15495,6 +15544,7 @@
rpl_ss = SSFromContour((PyFF_Contour *) rpl, NULL);
} else {
PyErr_Format(PyExc_TypeError, "Expected a contour or layer");
+ free(srch_ss);
return( NULL );
}
@@ -15939,11 +15989,13 @@
last = cur;
} else {
PyErr_Format(PyExc_TypeError, "Second argument must be either a font or a list of fonts");
+ free(head);
return( NULL );
}
}
} else {
PyErr_Format(PyExc_TypeError, "Second argument must be either a font or a list of fonts");
+ free(head);
return( NULL );
}
@@ -15982,6 +16034,7 @@
otl = SFFindLookup(fv->sf,lookup_name);
if ( otl == NULL ) {
PyErr_Format(PyExc_EnvironmentError, "No lookup named %s", lookup_name );
+ free(locfilename);
return( NULL );
}
}
@@ -16080,6 +16133,7 @@
if ( sf->fv==NULL )
EncMapFree(sf->map);
newfv = SFAdd(InterpolateFont(fv->sf,sf,fraction, fv->map->enc ),false);
+ free(sf);
return( PyFV_From_FV_I(newfv));
}
diff -urN fontforge-20170731.old/fontforge/savefont.c fontforge-20170731/fontforge/savefont.c
--- fontforge-20170731.old/fontforge/savefont.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/savefont.c 2018-09-23 14:50:31.573567455 +0530
@@ -313,8 +313,10 @@
char *buf = malloc(strlen(filename)+12), *pt;
FILE *flog;
- if ( sf->fontlog==NULL || *sf->fontlog=='\0' )
-return( true );
+ if ( sf->fontlog==NULL || *sf->fontlog=='\0' ) {
+ free(buf);
+ return( true );
+ }
strcpy(buf,filename);
pt = strrchr(buf,'/');
diff -urN fontforge-20170731.old/fontforge/scstyles.c fontforge-20170731/fontforge/scstyles.c
--- fontforge-20170731.old/fontforge/scstyles.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/scstyles.c 2018-09-23 14:50:31.575567424 +0530
@@ -2578,7 +2578,8 @@
for ( j=0; j<tot; ++j ) {
if ( overlaps[j].new_start == -10000 ) {
IError( "Hint zone not positioned" );
-return;
+ free(overlaps);
+ return;
}
}
diff -urN fontforge-20170731.old/fontforge/search.c fontforge-20170731/fontforge/search.c
--- fontforge-20170731.old/fontforge/search.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/search.c 2018-09-23 14:50:31.575567424 +0530
@@ -808,7 +808,8 @@
/* Total "fudge" amount should be spread evenly over each point */
FudgeFigure(s,path,&fudge);
if ( s->pointcnt!=s->rpointcnt )
- MinimumDistancesFree(sc->md); sc->md = NULL;
+ MinimumDistancesFree(sc->md);
+ sc->md = NULL;
sc_p = s->matched_sp; p_p = path->first, r_p = rpath->first;
if ( s->endpoints ) {
diff -urN fontforge-20170731.old/fontforge/sfd.c fontforge-20170731/fontforge/sfd.c
--- fontforge-20170731.old/fontforge/sfd.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/sfd.c 2018-09-23 14:50:31.576567408 +0530
@@ -7924,10 +7924,14 @@
{
char* sname = SFDReadUTF7Str(sfd);
if (sf->pfminfo.stylemap == -1) {
- if (strcmp(sname,"bold italic")==0) sf->pfminfo.stylemap = 0x21;
- else if (strcmp(sname,"bold")==0) sf->pfminfo.stylemap = 0x20;
- else if (strcmp(sname,"italic")==0) sf->pfminfo.stylemap = 0x01;
- else if (strcmp(sname,"regular")==0) sf->pfminfo.stylemap = 0x40;
+ if (strcmp(sname,"bold italic")==0)
+ sf->pfminfo.stylemap = 0x21;
+ else if (strcmp(sname,"bold")==0)
+ sf->pfminfo.stylemap = 0x20;
+ else if (strcmp(sname,"italic")==0)
+ sf->pfminfo.stylemap = 0x01;
+ else if (strcmp(sname,"regular")==0)
+ sf->pfminfo.stylemap = 0x40;
}
free(sname);
}
@@ -8010,7 +8014,8 @@
{
/* Class 0 is unused */
int temp;
- while ( (temp=nlgetc(sfd))=='\n' || temp=='\r' ); ungetc(temp,sfd);
+ while ( (temp=nlgetc(sfd))=='\n' || temp=='\r' );
+ ungetc(temp,sfd);
sf->mark_class_names[i] = SFDReadUTF7Str(sfd);
getint(sfd,&temp);
sf->mark_classes[i] = malloc(temp+1); sf->mark_classes[i][temp] = '\0';
@@ -8027,10 +8032,12 @@
{
/* Set 0 is used */
int temp;
- while ( (temp=nlgetc(sfd))=='\n' || temp=='\r' ); ungetc(temp,sfd);
+ while ( (temp=nlgetc(sfd))=='\n' || temp=='\r' );
+ ungetc(temp,sfd);
sf->mark_set_names[i] = SFDReadUTF7Str(sfd);
getint(sfd,&temp);
- sf->mark_sets[i] = malloc(temp+1); sf->mark_sets[i][temp] = '\0';
+ sf->mark_sets[i] = malloc(temp+1);
+ sf->mark_sets[i][temp] = '\0';
nlgetc(sfd); /* skip space */
fread(sf->mark_sets[i],1,temp,sfd);
}
@@ -8042,7 +8049,8 @@
int kernclassversion = 0;
int isv = tok[0]=='V';
int kcvoffset = (isv ? 10 : 9); //Offset to read kerning class version
- if (isdigit(tok[kcvoffset])) kernclassversion = tok[kcvoffset] - '0';
+ if (isdigit(tok[kcvoffset]))
+ kernclassversion = tok[kcvoffset] - '0';
int temp, classstart=1;
int old = (kernclassversion == 0);
@@ -8088,35 +8096,51 @@
for ( i=classstart; i<kc->first_cnt; ++i ) {
if (kernclassversion < 3) {
getint(sfd,&temp);
- kc->firsts[i] = malloc(temp+1); kc->firsts[i][temp] = '\0';
+ kc->firsts[i] = malloc(temp+1);
+ kc->firsts[i][temp] = '\0';
nlgetc(sfd); /* skip space */
fread(kc->firsts[i],1,temp,sfd);
} else {
getint(sfd,&kc->firsts_flags[i]);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd); if (ch == '\n' || ch == EOF) continue;
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
+ if (ch == '\n' || ch == EOF)
+ continue;
kc->firsts_names[i] = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd); if (ch == '\n' || ch == EOF) continue;
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
+ if (ch == '\n' || ch == EOF)
+ continue;
kc->firsts[i] = SFDReadUTF7Str(sfd);
if (kc->firsts[i] == NULL) kc->firsts[i] = copy(""); // In certain places, this must be defined.
- while ((ch=nlgetc(sfd)) == ' ' || ch == '\n'); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ' || ch == '\n');
+ ungetc(ch, sfd);
}
}
kc->seconds[0] = NULL;
for ( i=1; i<kc->second_cnt; ++i ) {
if (kernclassversion < 3) {
getint(sfd,&temp);
- kc->seconds[i] = malloc(temp+1); kc->seconds[i][temp] = '\0';
+ kc->seconds[i] = malloc(temp+1);
+ kc->seconds[i][temp] = '\0';
nlgetc(sfd); /* skip space */
fread(kc->seconds[i],1,temp,sfd);
} else {
getint(sfd,&temp);
kc->seconds_flags[i] = temp;
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd); if (ch == '\n' || ch == EOF) continue;
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
+ if (ch == '\n' || ch == EOF)
+ continue;
kc->seconds_names[i] = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd); if (ch == '\n' || ch == EOF) continue;
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
+ if (ch == '\n' || ch == EOF)
+ continue;
kc->seconds[i] = SFDReadUTF7Str(sfd);
if (kc->seconds[i] == NULL) kc->seconds[i] = copy(""); // In certain places, this must be defined.
- while ((ch=nlgetc(sfd)) == ' ' || ch == '\n'); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ' || ch == '\n');
+ ungetc(ch, sfd);
}
}
for ( i=0; i<kc->first_cnt*kc->second_cnt; ++i ) {
@@ -8173,39 +8197,50 @@
}
else if ( strmatch(tok,"Group:")==0 ) {
struct ff_glyphclasses *grouptmp = calloc(1, sizeof(struct ff_glyphclasses));
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
grouptmp->classname = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
grouptmp->glyphs = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' ' || ch == '\n'); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ' || ch == '\n');
+ ungetc(ch, sfd);
if (d->lastgroup != NULL) d->lastgroup->next = grouptmp; else sf->groups = grouptmp;
d->lastgroup = grouptmp;
}
else if ( strmatch(tok,"GroupKern:")==0 ) {
int temp = 0;
struct ff_rawoffsets *kerntmp = calloc(1, sizeof(struct ff_rawoffsets));
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
kerntmp->left = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
kerntmp->right = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
getint(sfd,&temp);
kerntmp->offset = temp;
- while ((ch=nlgetc(sfd)) == ' ' || ch == '\n'); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ' || ch == '\n');
+ ungetc(ch, sfd);
if (d->lastgroupkern != NULL) d->lastgroupkern->next = kerntmp; else sf->groupkerns = kerntmp;
d->lastgroupkern = kerntmp;
}
else if ( strmatch(tok,"GroupVKern:")==0 ) {
int temp = 0;
struct ff_rawoffsets *kerntmp = calloc(1, sizeof(struct ff_rawoffsets));
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
kerntmp->left = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
kerntmp->right = SFDReadUTF7Str(sfd);
- while ((ch=nlgetc(sfd)) == ' '); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ');
+ ungetc(ch, sfd);
getint(sfd,&temp);
kerntmp->offset = temp;
- while ((ch=nlgetc(sfd)) == ' ' || ch == '\n'); ungetc(ch, sfd);
+ while ((ch=nlgetc(sfd)) == ' ' || ch == '\n');
+ ungetc(ch, sfd);
if (d->lastgroupvkern != NULL) d->lastgroupvkern->next = kerntmp; else sf->groupvkerns = kerntmp;
d->lastgroupvkern = kerntmp;
}
@@ -8710,6 +8746,7 @@
else if ( lastnamedinstance !=NULL )
lastnamedinstance->names = names;
pushedbacktok = true;
+ free(names);
} else if ( strmatch(tok,"MMCDV:")==0 ) {
MMSet *mm = sf->mm;
if ( mm!=NULL )
@@ -8833,6 +8870,7 @@
if (sf->map == NULL) sf->map = EncMapNew(sf->glyphcnt,sf->glyphcnt,&custom);
SFDFixupUndoRefs(sf);
+ free(remap);
return( sf );
}
diff -urN fontforge-20170731.old/fontforge/sflayout.c fontforge-20170731/fontforge/sflayout.c
--- fontforge-20170731.old/fontforge/sflayout.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/sflayout.c 2018-09-23 14:50:31.576567408 +0530
@@ -1285,6 +1285,7 @@
LayoutInfo_Destroy(li);
if ( freeme!=NULL )
arrayfree(freeme);
+ free(li);
}
#include <stdlib.h>
@@ -1415,5 +1416,6 @@
}
LI_MetaChangeCleanup(li,start,end,width);
+ free(fl);
return( true );
}
diff -urN fontforge-20170731.old/fontforge/splinefont.c fontforge-20170731/fontforge/splinefont.c
--- fontforge-20170731.old/fontforge/splinefont.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/splinefont.c 2018-09-23 14:50:31.577567393 +0530
@@ -999,8 +999,10 @@
unlink(spuriousname); free(spuriousname);
} else
strippedname = Unarchive(strippedname,&archivedir);
- if ( strippedname==NULL )
- return NULL;
+ if ( strippedname==NULL ){
+ free(archivedir);
+ return NULL;
+ }
if ( strippedname!=fname && paren!=NULL ) {
fullname = malloc(strlen(strippedname)+strlen(paren)+1);
strcpy(fullname,strippedname);
@@ -1160,7 +1162,8 @@
checked = 'i';
sf = SFReadIkarus(fullname);
} /* Too hard to figure out a valid mark for a mac resource file */
- if ( file!=NULL ) fclose(file);
+ if ( file!=NULL )
+ fclose(file);
}
if ( sf!=NULL )
@@ -1280,7 +1283,9 @@
sf = NULL;
}
}
- if (fname != NULL && fname != filename) free(fname); fname = NULL;
+ if (fname != NULL && fname != filename)
+ free(fname);
+ fname = NULL;
return sf;
}
diff -urN fontforge-20170731.old/fontforge/splineoverlap.c fontforge-20170731/fontforge/splineoverlap.c
--- fontforge-20170731.old/fontforge/splineoverlap.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/splineoverlap.c 2018-09-23 14:50:31.577567393 +0530
@@ -494,6 +494,7 @@
}
current_pointer = &((*current_pointer)->next);
}
+ free(tmp_pointer);
return;
}
diff -urN fontforge-20170731.old/fontforge/splinesaveafm.c fontforge-20170731/fontforge/splinesaveafm.c
--- fontforge-20170731.old/fontforge/splinesaveafm.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/splinesaveafm.c 2018-09-23 14:55:32.054958121 +0530
@@ -1285,8 +1285,10 @@
ch = start[4]; start[4] = '\0';
unicode[u++] = strtol(start,&end,16);
start[4] = ch;
- if ( *end!='\0' )
-return( -1 );
+ if ( *end!='\0' ){
+ free(start);
+ return( -1 );
+ }
start += 4;
}
return( u );
@@ -3424,6 +3426,7 @@
free( italicindex );
} else
free( lkarray );
+ free(kerns);
return( !ferror(tfm));
}
diff -urN fontforge-20170731.old/fontforge/splinestroke.c fontforge-20170731/fontforge/splinestroke.c
--- fontforge-20170731.old/fontforge/splinestroke.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/splinestroke.c 2018-09-23 14:50:31.578567377 +0530
@@ -3847,6 +3847,7 @@
c.resolution = c.radius/3;
if (c.resolution == 0) {
ff_post_notice(_("Invalid stroke parameters"), _("Stroke resolution is zero"));
+ free(first);
return SplinePointListCopy(ss);
}
cur = SplineSets_Stroke(ss,&c,order2);
diff -urN fontforge-20170731.old/fontforge/splineutil.c fontforge-20170731/fontforge/splineutil.c
--- fontforge-20170731.old/fontforge/splineutil.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/splineutil.c 2018-09-23 14:50:31.579567361 +0530
@@ -224,7 +224,8 @@
next = spline->to->next; // Cache the location of the next spline.
SplinePointFree(spline->to); // Free the destination point.
SplineFree(spline); // Free the spline.
- if ( first==NULL ) first = spline; // We want to avoid repeating the circuit.
+ if ( first==NULL )
+ first = spline; // We want to avoid repeating the circuit.
}
// If the path is open or has no splines, free the starting point.
if ( spl->last!=spl->first || nonext )
@@ -262,7 +263,8 @@
next = spline->to->next;
SplinePointMDFree(sc,spline->to);
SplineFree(spline);
- if ( first==NULL ) first = spline;
+ if ( first==NULL )
+ first = spline;
}
if ( freefirst )
SplinePointMDFree(sc,spl->first);
@@ -1404,7 +1406,8 @@
last = cur;
i = j;
}
-return( head );
+ free(list);
+ return( head );
}
SplinePointList *SplinePointListCopy(const SplinePointList *base) {
diff -urN fontforge-20170731.old/fontforge/strlist.c fontforge-20170731/fontforge/strlist.c
--- fontforge-20170731.old/fontforge/strlist.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/strlist.c 2018-09-23 14:50:31.579567361 +0530
@@ -34,6 +34,7 @@
list->next = NULL;
list->str = str2;
}
+ free(str2);
return list;
}
diff -urN fontforge-20170731.old/fontforge/svg.c fontforge-20170731/fontforge/svg.c
--- fontforge-20170731.old/fontforge/svg.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/svg.c 2018-09-23 14:50:31.580567346 +0530
@@ -2634,8 +2634,11 @@
if ( name!=NULL ) {
int hide = xmlStrcmp(name,(xmlChar *) "none")==0;
xmlFree(name);
- if ( hide )
-return( NULL );
+ if ( hide ){
+ free(fill_colour_source);
+ free(stroke_colour_source);
+ return( NULL );
+ }
}
name = xmlGetProp(svg,(xmlChar *) "visibility");
if ( name!=NULL ) {
@@ -2772,7 +2775,9 @@
if ( !st.isvisible ) {
SvgStateFree(&st);
-return( NULL );
+ free(fill_colour_source);
+ free(stroke_colour_source);
+ return( NULL );
}
/* basic shapes */
@@ -2798,8 +2803,11 @@
return( eret );
} else
return( NULL );
- if ( head==NULL )
-return( NULL );
+ if ( head==NULL ){
+ free(fill_colour_source);
+ free(stroke_colour_source);
+ return( NULL );
+ }
SPLCategorizePoints(head);
@@ -3056,6 +3064,7 @@
}
free(u);
}
+ free(chars);
}
static char *SVGGetNames(SplineFont *sf,xmlChar *g,xmlChar *utf8,SplineChar **sc) {
diff -urN fontforge-20170731.old/fontforge/tottfaat.c fontforge-20170731/fontforge/tottfaat.c
--- fontforge-20170731.old/fontforge/tottfaat.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/tottfaat.c 2018-09-23 14:50:31.580567346 +0530
@@ -1483,6 +1483,7 @@
}
free(scripts);
}
+ free(sm);
return( features );
}
diff -urN fontforge-20170731.old/fontforge/tottf.c fontforge-20170731/fontforge/tottf.c
--- fontforge-20170731.old/fontforge/tottf.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/tottf.c 2018-09-23 14:50:31.581567330 +0530
@@ -4394,8 +4394,10 @@
(subheadcnt-i)*sizeof(struct subhead) + sizeof(uint16);
sub = tmpfile();
- if ( sub==NULL )
-return( NULL );
+ if ( sub==NULL ){
+ free(glyphs);
+ return( NULL );
+ }
putshort(sub,2); /* 8/16 format */
putshort(sub,0); /* Subtable length, we'll come back and fix this */
diff -urN fontforge-20170731.old/fontforge/tottfgpos.c fontforge-20170731/fontforge/tottfgpos.c
--- fontforge-20170731.old/fontforge/tottfgpos.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/tottfgpos.c 2018-09-23 14:54:53.300546119 +0530
@@ -452,8 +452,10 @@
++cnt;
}
}
- if ( cnt==0 )
-return( NULL );
+ if ( cnt==0 ){
+ free(array);
+ return( NULL );
+ }
if ( j==1 )
break;
array = malloc((cnt+1)*sizeof(SplineChar *));
@@ -583,13 +585,14 @@
// And we null-terminate.
glyphs[i] = NULL;
}
- for ( i=0; i<cnt-1; ++i )
+ for ( i=0; i<cnt-1; ++i ){
if (glyphs[i]->ttf_glyph==glyphs[i+1]->ttf_glyph) {
// fprintf(stderr, "Duplicate glyph.\n");
memmove(glyphs+i, glyphs+i+1, (cnt-i)*sizeof(SplineChar *));
--cnt;
}
- glyphs[cnt] = NULL;
+ }
+ glyphs[cnt] = NULL;
return( glyphs );
}
@@ -3476,8 +3479,10 @@
glyphs = malloc((lcnt+1)*sizeof(SplineChar *));
glyphs[lcnt] = NULL;
}
- if ( !needsclass && lcnt==0 && sf->mark_class_cnt==0 && sf->mark_set_cnt==0 )
-return; /* No anchor positioning, no ligature carets */
+ if ( !needsclass && lcnt==0 && sf->mark_class_cnt==0 && sf->mark_set_cnt==0 ){
+ free(glyphs);
+ return; /* No anchor positioning, no ligature carets */
+ }
at->gdef = tmpfile();
if ( sf->mark_set_cnt==0 ) {
diff -urN fontforge-20170731.old/fontforge/ttfinstrs.c fontforge-20170731/fontforge/ttfinstrs.c
--- fontforge-20170731.old/fontforge/ttfinstrs.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/ttfinstrs.c 2018-09-23 14:50:31.582567314 +0530
@@ -309,7 +309,8 @@
val = strtol(pt,&end,0);
if ( val>32767 || val<-32768 ) {
IVError(iv,_("A value must be between [-32768,32767]"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
pt = end;
@@ -317,14 +318,16 @@
if ( *pt == '@' ) { /* a delta control byte */
if ( val>8 || val<-8 || val==0 ) {
IVError(iv,_("A value must be between [-8,-1] or [1,8]"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
pt++;
if ( !isdigit( *pt ) ) {
IVError(iv,_("Number expected"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
temp = val;
@@ -332,7 +335,8 @@
if ( val>15 || val<0 ) {
IVError(iv,_("A value must be between [0,15]"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
val *= 16;
@@ -348,14 +352,16 @@
while ( *pt==' ' || *pt=='\t' ) ++pt;
if ( *pt!='(' ) {
IVError(iv,_("Missing left parenthesis in command to get a cvt index"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
temp = strtol(pt+1,&end,0);
pt = end;
while ( *pt==' ' || *pt=='\t' ) ++pt;
if ( *pt!=')' ) {
IVError(iv,_("Missing right paren in command to get a cvt index"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
numberstack[npos++] = TTF__getcvtval(sf,temp);
++pt;
@@ -396,7 +402,8 @@
}
if ( nread<npos && push_left==0 && (*pt=='\r' || *pt=='\n' || *pt=='\0')) {
IVError(iv,_("Unexpected number"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
if ( *pt=='\r' || *pt=='\n' || *pt=='\0' )
continue;
@@ -446,11 +453,13 @@
while ( *bend==' ' || *bend=='\t' ) ++bend;
if ( *bend!=']' ) {
IVError(iv,_("Missing right bracket in command (or bad binary value in bracket)"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
if ( val>=32 ) {
IVError(iv,_("Bracketted value is too large"),pt-text);
-return( NULL );
+ free(instrs);
+ return( NULL );
}
i += val;
}
diff -urN fontforge-20170731.old/fontforge/ttfspecial.c fontforge-20170731/fontforge/ttfspecial.c
--- fontforge-20170731.old/fontforge/ttfspecial.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/ttfspecial.c 2018-09-23 14:54:21.159033774 +0530
@@ -1356,6 +1356,7 @@
}
}
free(contours);
+ free(cur);
}
static void pfed_readguidelines(FILE *ttf,struct ttfinfo *info,uint32 base) {
diff -urN fontforge-20170731.old/fontforge/ufo.c fontforge-20170731/fontforge/ufo.c
--- fontforge-20170731.old/fontforge/ufo.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforge/ufo.c 2018-09-23 14:50:31.583567299 +0530
@@ -2513,7 +2513,18 @@
}
lastref = r;
}
- if (xs) free(xs); if (ys) free(ys); if (xys) free(xys); if (yxs) free(yxs); if (xo) free(xo); if (yo) free(yo);
+ if (xs)
+ free(xs);
+ if (ys)
+ free(ys);
+ if (xys)
+ free(xys);
+ if (yxs)
+ free(yxs);
+ if (xo)
+ free(xo);
+ if (yo)
+ free(yo);
} else if ( xmlStrcmp(contour->name,(const xmlChar *) "contour")==0 ) {
xmlNodePtr npoints;
@@ -4065,12 +4076,12 @@
strncmp(sf->names->names[ttf_version],"Version ",8)==0 )
sf->version = copy(sf->names->names[ttf_version]+8);
- char * layercontentsname = buildname(basedir,"layercontents.plist");
- char ** layernames = NULL;
- if (layercontentsname == NULL) {
+ char * layercontentsname = buildname(basedir,"layercontents.plist");
+ char ** layernames = NULL;
+ if (layercontentsname == NULL) {
switch_to_old_locale(&tmplocale, &oldlocale); // Switch to the cached locale.
return( NULL );
- } else if ( GFileExists(layercontentsname)) {
+ } else if ( GFileExists(layercontentsname)) {
xmlDocPtr layercontentsdoc = NULL;
xmlNodePtr layercontentsplist = NULL;
xmlNodePtr layercontentsdict = NULL;
diff -urN fontforge-20170731.old/fontforgeexe/alignment.c fontforge-20170731/fontforgeexe/alignment.c
--- fontforge-20170731.old/fontforgeexe/alignment.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/alignment.c 2018-09-23 14:50:31.583567299 +0530
@@ -126,8 +126,10 @@
}
/* we need at least three regions to space things out */
- if ( rcnt<3 )
+ if ( rcnt<3 ){
+ free(regions);
return;
+ }
/* Now should I allow equal spaces between regions, or spaces between */
/* region mid-points? I think spaces between regions */
diff -urN fontforge-20170731.old/fontforgeexe/charinfo.c fontforge-20170731/fontforgeexe/charinfo.c
--- fontforge-20170731.old/fontforgeexe/charinfo.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/charinfo.c 2018-09-23 14:50:31.584567283 +0530
@@ -3008,6 +3008,7 @@
for ( j=0; j<i; ++j )
BDFCharFree(others[j]);
free(others);
+ free(me);
return( NULL );
}
others[i] = Rasterize(othersc,def_layer);
@@ -3263,6 +3264,7 @@
for ( i=0; i<extracnt; ++i )
BDFCharFree(extras[i]);
free(extras);
+ free(me);
return( NULL );
}
diff -urN fontforge-20170731.old/fontforgeexe/charview.c fontforge-20170731/fontforgeexe/charview.c
--- fontforge-20170731.old/fontforgeexe/charview.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/charview.c 2018-09-23 14:50:31.585567268 +0530
@@ -4437,7 +4437,10 @@
sign = dx*dy<0?-1:1;
fake->u.mouse = event->u.mouse;
- if ( dx<0 ) dx = -dx; if ( dy<0 ) dy = -dy;
+ if ( dx<0 )
+ dx = -dx;
+ if ( dy<0 )
+ dy = -dy;
if ( dy >= 2*dx ) {
cv->p.x = fake->u.mouse.x = basex;
cv->p.cx = basetruex ;
@@ -4726,6 +4729,7 @@
{
GGadgetSetTitle( cv->charselector, p );
}
+ free(srctxt);
}
@@ -5328,7 +5332,10 @@
int sign = dx*dy<0?-1:1;
double aspect = 1.0;
- if ( dx<0 ) dx = -dx; if ( dy<0 ) dy = -dy;
+ if ( dx<0 )
+ dx = -dx;
+ if ( dy<0 )
+ dy = -dy;
if ( cv->p.img!=NULL && cv->p.img->bb.minx!=cv->p.img->bb.maxx )
aspect = (cv->p.img->bb.maxy - cv->p.img->bb.miny) / (cv->p.img->bb.maxx - cv->p.img->bb.minx);
else if ( cv->p.ref!=NULL && cv->p.ref->bb.minx!=cv->p.ref->bb.maxx )
@@ -7487,8 +7494,10 @@
e.u.control.subtype = et_textchanged;
e.u.control.u.tf_changed.from_pulldown = 0;
CV_OnCharSelectorTextChanged( cv->charselector, &e );
+ free(txt);
return;
}
+ free(txt);
}
if ( mid == MID_Next ) {
@@ -12810,6 +12819,7 @@
}
}
}
+ free(wll);
}
free(txt);
diff -urN fontforge-20170731.old/fontforgeexe/contextchain.c fontforge-20170731/fontforgeexe/contextchain.c
--- fontforge-20170731.old/fontforgeexe/contextchain.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/contextchain.c 2018-09-23 14:50:31.586567252 +0530
@@ -641,6 +641,7 @@
(&r->u.class.nclasses)[which] = malloc(any*sizeof(uint16));
}
}
+ free(pt);
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/cvgetinfo.c fontforge-20170731/fontforgeexe/cvgetinfo.c
--- fontforge-20170731.old/fontforgeexe/cvgetinfo.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/cvgetinfo.c 2018-09-23 14:50:31.586567252 +0530
@@ -1059,6 +1059,7 @@
ti = AnchorClassesLList(sf),false);
for ( j=0; ti[j]->text!=NULL && ti[j]->userdata!=ac; ++j )
GGadgetSelectOneListItem(GWidgetGetControl(ci->gw,CID_NameList),j);
+ free(name);
return( true );
}
@@ -3903,8 +3904,11 @@
}
++k;
} while ( k<_sf->subfontcnt );
- if ( tot==0 )
+ if ( tot==0 ){
+ free(deps);
+ free(depsc);
return;
+ }
if ( j==0 ) {
deps = calloc(tot+1,sizeof(char *));
depsc = malloc(tot*sizeof(SplineChar *));
diff -urN fontforge-20170731.old/fontforgeexe/cvimportdlg.c fontforge-20170731/fontforgeexe/cvimportdlg.c
--- fontforge-20170731.old/fontforgeexe/cvimportdlg.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/cvimportdlg.c 2018-09-23 14:50:31.587567236 +0530
@@ -427,8 +427,10 @@
GGadget *tf;
GFileChooserGetChildren(d->gfc,NULL,NULL,&tf);
- if ( *_GGadgetGetTitle(tf)=='\0' )
+ if ( *_GGadgetGetTitle(tf)=='\0' ){
+ free(temp);
return( true );
+ }
GDrawSetCursor(GGadgetGetWindow(g),ct_watch);
if ( d->fv!=NULL )
flast_format = pos;
diff -urN fontforge-20170731.old/fontforgeexe/cvpalettes.c fontforge-20170731/fontforgeexe/cvpalettes.c
--- fontforge-20170731.old/fontforgeexe/cvpalettes.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/cvpalettes.c 2018-09-23 14:50:31.587567236 +0530
@@ -2432,6 +2432,7 @@
layerinfo.rename_active = 0;
CVInfoDrawText(cv,cv->gw);
+ free(str);
}
}
diff -urN fontforge-20170731.old/fontforgeexe/displayfonts.c fontforge-20170731/fontforgeexe/displayfonts.c
--- fontforge-20170731.old/fontforgeexe/displayfonts.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/displayfonts.c 2018-09-23 14:50:31.588567221 +0530
@@ -138,6 +138,7 @@
float pw,ph, scale;
if ( sscanf(cret,"%gx%g",&pw,&ph)!=2 ) {
IError("Bad Pagesize must be a known name or <num>x<num><units>\nWhere <units> is one of pt (points), mm, cm, in" );
+ free(cret);
return( true );
}
pt = cret+strlen(cret)-1;
@@ -152,6 +153,7 @@
scale = 1;
else {
IError("Bad Pagesize units are unknown\nMust be one of pt (points), mm, cm, in" );
+ free(pt);
return( true );
}
pgwidth = pw*scale; pgheight = ph*scale;
@@ -1431,6 +1433,7 @@
feats[cnt] = 0;
/* These will be ordered because the list widget will do that */
SFTFSetFeatures(GWidgetGetControl(di->gw,CID_SampleText),-1,-1,feats);
+ free(feats);
}
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/fontinfo.c fontforge-20170731/fontforgeexe/fontinfo.c
--- fontforge-20170731.old/fontforgeexe/fontinfo.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/fontinfo.c 2018-09-23 14:50:31.589567205 +0530
@@ -3752,6 +3752,7 @@
}
}
}
+ free(changed);
}
static void StoreTTFNames(struct gfi_data *d) {
diff -urN fontforge-20170731.old/fontforgeexe/fontview.c fontforge-20170731/fontforgeexe/fontview.c
--- fontforge-20170731.old/fontforgeexe/fontview.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/fontview.c 2018-09-23 14:50:31.590567190 +0530
@@ -582,6 +582,7 @@
GGadgetSetTitle8(fc,newname);
save_to_dir = *_s2d = s2d;
SavePrefs(true);
+ free(oldname);
}
return( true );
}
@@ -3741,12 +3742,15 @@
free(filename);
if ( new==NULL )
return;
- if ( new->fv == &fv->b ) /* Already part of us */
+ if ( new->fv == &fv->b ) { /* Already part of us */
+ free(new);
return;
+ }
if ( new->fv != NULL ) {
if ( ((FontView *) (new->fv))->gw!=NULL )
GDrawRaise( ((FontView *) (new->fv))->gw);
ff_post_error(_("Please close font"),_("Please close %s before inserting it into a CID font"),new->origname);
+ free(new);
return;
}
EncMapFree(new->map);
@@ -4978,9 +4982,11 @@
nl = NameListByName(namelists[ret]);
if ( nl==NULL ) {
IError("Couldn't find namelist");
+ free(namelists);
return;
} else if ( nl!=NULL && nl->uses_unicode && !allow_utf8_glyphnames) {
ff_post_error(_("Namelist contains non-ASCII names"),_("Glyph names should be limited to characters in the ASCII character set, but there are names in this namelist which use characters outside that range."));
+ free(namelists);
return;
}
SFRenameGlyphsToNamelist(fv->b.sf,nl);
@@ -6699,7 +6705,9 @@
msg = xasprintf ( "%u 0x%x U+%04X \"%.25s\" %.100s",
localenc, localenc, upos,
(sc->name == NULL) ? "" : sc->name, uniname );
- if ( uniname != NULL ) free( uniname ); uniname = NULL;
+ if ( uniname != NULL )
+ free( uniname );
+ uniname = NULL;
/* annotation */
char *uniannot = unicode_annot( upos );
@@ -8129,6 +8137,7 @@
free(repr);
*rpt++ = ' ';
}
+ free(repr);
}
}
if ( k==0 )
diff -urN fontforge-20170731.old/fontforgeexe/fvfontsdlg.c fontforge-20170731/fontforgeexe/fvfontsdlg.c
--- fontforge-20170731.old/fontforgeexe/fvfontsdlg.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/fvfontsdlg.c 2018-09-23 14:50:31.590567190 +0530
@@ -73,6 +73,7 @@
file = fpt+2;
} while ( fpt!=NULL );
free(filename);
+ free(sf);
}
GTextInfo *BuildFontList(FontView *except) {
@@ -285,6 +286,7 @@
if ( sf==NULL )
return;
FontViewCreate(InterpolateFont(fv->b.sf,sf,amount,fv->b.map->enc),false);
+ free(sf);
}
#define CID_Amount 1000
diff -urN fontforge-20170731.old/fontforgeexe/histograms.c fontforge-20170731/fontforgeexe/histograms.c
--- fontforge-20170731.old/fontforgeexe/histograms.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/histograms.c 2018-09-23 14:50:31.590567190 +0530
@@ -573,8 +573,11 @@
ret2 = GGadgetGetTitle(GWidgetGetControl(hist->gw,CID_SecondaryVal));
hist->done = true;
if ( (*ret1=='\0' || uc_strcmp(ret1,"[]")==0 ) &&
- (*ret2=='\0' || uc_strcmp(ret2,"[]")==0 ) && p==NULL )
+ (*ret2=='\0' || uc_strcmp(ret2,"[]")==0 ) && p==NULL ){
+ free(ret1);
+ free(ret2);
return;
+ }
if ( p==NULL ) {
hist->sf->private = p = calloc(1,sizeof(struct psdict));
p->cnt = 10;
@@ -583,6 +586,8 @@
}
PSDictChangeEntry(p,primary,temp=cu_copy(ret1)); free(temp);
PSDictChangeEntry(p,secondary,temp=cu_copy(ret2)); free(temp);
+ free(ret1);
+ free(ret2);
}
static int leftside_e_h(GWindow gw, GEvent *event) {
diff -urN fontforge-20170731.old/fontforgeexe/kernclass.c fontforge-20170731/fontforgeexe/kernclass.c
--- fontforge-20170731.old/fontforgeexe/kernclass.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/kernclass.c 2018-09-23 14:50:31.591567174 +0530
@@ -1345,6 +1345,7 @@
KCD_VShow(kcd,i);
else
KCD_HShow(kcd,i);
+ free(name);
return( true );
}
if ( *pt=='(' ) {
@@ -1358,7 +1359,9 @@
/* Otherwise deselect everything */
if ( nlen!=0 )
GMatrixEditActivateRowCol(list,-1,-1);
+ free(name);
}
+
return( true );
}
@@ -2235,6 +2238,7 @@
GDrawRequestExpose(kcd->subw,NULL,false);
+ free(other);
return( new );
}
diff -urN fontforge-20170731.old/fontforgeexe/lookupui.c fontforge-20170731/fontforgeexe/lookupui.c
--- fontforge-20170731.old/fontforgeexe/lookupui.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/lookupui.c 2018-09-23 14:54:04.668283976 +0530
@@ -2779,16 +2779,23 @@
char *suffix = GGadgetGetTitle8(GWidgetGetControl(pstkd->gw,CID_Suffix));
SplineChar *alt, *sc;
- if ( col!=0 || !wasnew || psts[row*cols+0].u.md_str==NULL )
+ if ( col!=0 || !wasnew || psts[row*cols+0].u.md_str==NULL ){
+ free(suffix);
return;
- if ( *suffix=='\0' || ( suffix[0]=='.' && suffix[1]=='\0' ))
+ }
+ if ( *suffix=='\0' || ( suffix[0]=='.' && suffix[1]=='\0' )){
+ free(suffix);
return;
+ }
sc = SFGetChar(pstkd->sf,-1,psts[row*cols+0].u.md_str);
- if ( sc==NULL )
+ if ( sc==NULL ){
+ free(suffix);
return;
+ }
alt = SuffixCheck(sc,suffix);
if ( alt!=NULL )
psts[row*cols+1].u.md_str = copy(alt->name);
+ free(suffix);
}
static void PSTKD_FinishBoundsEdit(GGadget *g, int row, int col, int wasnew) {
@@ -3403,8 +3410,8 @@
if ( *suffix!='\0' && ( suffix[0]!='.' || suffix[1]!='\0' )) {
free(pstkd->sub->suffix);
pstkd->sub->suffix = ( *suffix=='.' ) ? copy(suffix+1): copy(suffix);
- free(suffix);
}
+ free(suffix);
}
static int PSTKD_PopulateWithSuffix(GGadget *g, GEvent *e) {
@@ -3953,6 +3960,8 @@
error2SFDTrimUndoOldToNew: fclose(nf);
error1SFDTrimUndoOldToNew: fclose(retf);
error0SFDTrimUndoOldToNew:
+ free(nglyph);
+ free(oglyph);
return 0;
}
@@ -4229,9 +4238,11 @@
dlist_pushfront( (struct dlistnode **)&sf->undoes, (struct dlistnode *)undo );
}
// printf("we now have %d splinefont level undoes\n", dlist_size((struct dlistnode **)&sf->undoes));
+ free(str);
}
pstkd->done = true;
+ free(oldsfd);
}
return( true );
}
@@ -6069,6 +6080,7 @@
while ( (lang = StrNextLang(&pt))!=0 ) {
if ( lang==0xffffffff ) {
ff_post_error(_("Invalid language"), _("Please specify a comma separated list of 4 letter opentype language tags"));
+ free(langs);
return( true );
}
}
diff -urN fontforge-20170731.old/fontforgeexe/macencui.c fontforge-20170731/fontforgeexe/macencui.c
--- fontforge-20170731.old/fontforgeexe/macencui.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/macencui.c 2018-09-23 14:50:31.592567158 +0530
@@ -425,6 +425,7 @@
nd->done = true;
}
}
+ free(temp);
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/metricsview.c fontforge-20170731/fontforgeexe/metricsview.c
--- fontforge-20170731.old/fontforgeexe/metricsview.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/metricsview.c 2018-09-23 14:50:31.592567158 +0530
@@ -1970,8 +1970,11 @@
if ( *pt!=mv->chars[i]->unicodeenc &&
!MVOddMatch(mv,*pt,mv->chars[i]))
break;
- if ( i==mv->clen && *pt=='\0' )
-return; /* Nothing changed */
+ if ( i==mv->clen && *pt=='\0' ){
+ free(pt);
+return;
+ }
+ /* Nothing changed */
for ( ept=ret+u_strlen(ret)-1, ei=mv->clen-1; ; --ei, --ept )
if ( ei<0 || ept<ret || (*ept!=mv->chars[ei]->unicodeenc &&
!MVOddMatch(mv,*ept,mv->chars[ei]))) {
@@ -2046,6 +2049,7 @@
selectUserChosenWordListGlyphs( mv, wll );
GDrawRequestExpose(mv->v,NULL,false);
+ free(pt);
}
GTextInfo mv_text_init[] = {
diff -urN fontforge-20170731.old/fontforgeexe/mmdlg.c fontforge-20170731/fontforgeexe/mmdlg.c
--- fontforge-20170731.old/fontforgeexe/mmdlg.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/mmdlg.c 2018-09-23 14:50:31.593567143 +0530
@@ -1980,6 +1980,7 @@
enc = default_encoding;
FVReencode((FontViewBase *) fv,enc);
mmw->done = true;
+ free(setto);
}
static void MMW_DoNext(MMW *mmw) {
@@ -2374,17 +2375,19 @@
if ( ti!=NULL && ti->userdata == (void *) -1 ) {
temp = GetPostScriptFontName(NULL,false);
if ( temp==NULL )
-return(true);
+ return(true);
sf = LoadSplineFont(temp,0);
free(temp); temp = NULL;
if ( sf==NULL )
-return(true);
+ return(true);
if ( sf->cidmaster!=NULL || sf->subfonts!=0 ) {
ff_post_error(_("Bad Multiple Master Font"),_("CID keyed fonts may not be a master design of a multiple master font"));
-return(true);
+ free(sf);
+ return(true);
} else if ( sf->mm!=NULL ) {
ff_post_error(_("Bad Multiple Master Font"),_("CID keyed fonts may not be a master design of a multiple master font"));
-return(true);
+ free(sf);
+ return(true);
}
if ( sf->fv==NULL ) {
if ( mmw->lcnt>=mmw->lmax ) {
@@ -2406,6 +2409,7 @@
GGadgetSetTitle(g,ut = uc_copy(sf->fontname));
free(ut);
}
+ free(sf);
}
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/prefs.c fontforge-20170731/fontforgeexe/prefs.c
--- fontforge-20170731.old/fontforgeexe/prefs.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/prefs.c 2018-09-23 14:50:31.593567143 +0530
@@ -2380,6 +2380,7 @@
hvarray[si++] = &pgcd[gc-1];
hvarray[si++] = GCD_ColSpan; hvarray[si++] = GCD_ColSpan;
y += 28;
+ free(nlnames);
} break;
case pr_string: case pr_file:
if ( pl->set==SetAutoTraceArgs || ((char **) pl->val)==&mf_args )
@@ -2955,6 +2956,7 @@
hvarray[si++] = &pgcd[gc-1];
hvarray[si++] = GCD_ColSpan; hvarray[si++] = GCD_ColSpan;
y += 28;
+ free(nlnames);
} break;
case pr_string: case pr_file:
if ( pl->set==SetAutoTraceArgs || ((char **) pl->val)==&mf_args )
diff -urN fontforge-20170731.old/fontforgeexe/savefontdlg.c fontforge-20170731/fontforgeexe/savefontdlg.c
--- fontforge-20170731.old/fontforgeexe/savefontdlg.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/savefontdlg.c 2018-09-23 14:50:31.594567127 +0530
@@ -1426,8 +1426,10 @@
oldbitmapstate = GGadgetGetFirstListSelectedItem(d->bmptype);
if ( oldbitmapstate!=bf_none )
sizes = ParseBitmapSizes(d->bmpsizes,_("Pixel List"),&err);
- if ( err )
+ if ( err ){
+ free(temp);
return;
+ }
if ( oldbitmapstate==bf_nfntmacbin && oldformatstate!=ff_pfbmacbin && !nfnt_warned ) {
nfnt_warned = true;
ff_post_notice(_("The 'NFNT' bitmap format is obsolete"),_("The 'NFNT' bitmap format is not used under OS/X (though you still need to create a (useless) bitmap font if you are saving a type1 PostScript resource)"));
@@ -1484,7 +1486,9 @@
d->done = true;
d->ret = false;
SFValidationWindow(d->sf,layer,oldformatstate);
-return;
+ free(sizes);
+ free(temp);
+ return;
}
/* Ok... they want to proceed */
}
@@ -1581,6 +1585,7 @@
free(temp);
d->done = !err;
d->ret = !err;
+ free(cur);
}
static void GFD_doesnt(GIOControl *gio) {
@@ -2109,6 +2114,7 @@
}
if ( MacStyleCode(sf,NULL)!=0 || familycnt<=1 || sf->multilayer ) {
ff_post_error(_("Bad Mac Family"),_("To generate a Mac family file, the current font must have plain (Normal, Regular, etc.) style, and there must be other open fonts with the same family name."));
+ free(familysfs);
return( 0 );
} else if ( dup ) {
MacStyleCode(dup,&psstyle);
diff -urN fontforge-20170731.old/fontforgeexe/scstylesui.c fontforge-20170731/fontforgeexe/scstylesui.c
--- fontforge-20170731.old/fontforgeexe/scstylesui.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/scstylesui.c 2018-09-23 14:50:31.594567127 +0530
@@ -2323,6 +2323,7 @@
FVTrans((FontViewBase *) fv,sc,transform,NULL,fvt_dontmovewidth);
}
}
+ free(ret);
}
diff -urN fontforge-20170731.old/fontforgeexe/sftextfield.c fontforge-20170731/fontforgeexe/sftextfield.c
--- fontforge-20170731.old/fontforgeexe/sftextfield.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/sftextfield.c 2018-09-23 14:50:31.594567127 +0530
@@ -702,8 +702,10 @@
}
for ( cnt=0; scriptlangs[cnt]!=NULL; ++cnt );
i = ff_choose(_("Text from script"),(const char **) scriptlangs,cnt,0,_("Insert random text in the specified script"));
- if ( i==-1 )
+ if ( i==-1) {
+ free(freq);
return;
+ }
pos = strlen(scriptlangs[i])-10;
script = (scriptlangs[i][pos+0]<<24) |
(scriptlangs[i][pos+1]<<16) |
@@ -2303,6 +2305,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
@@ -2323,6 +2326,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
@@ -2345,6 +2349,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
@@ -2365,6 +2370,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
@@ -2385,6 +2391,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
@@ -2401,6 +2408,7 @@
}
SFTFMetaChangeCleanup(st,start,end);
+ free(fl);
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/statemachine.c fontforge-20170731/fontforgeexe/statemachine.c
--- fontforge-20170731.old/fontforgeexe/statemachine.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/statemachine.c 2018-09-23 14:50:31.595567112 +0530
@@ -438,6 +438,7 @@
/* Show changes in main window */
GDrawRequestExpose(smd->gw,NULL,false);
+ free(ret8);
return( true );
}
diff -urN fontforge-20170731.old/fontforgeexe/wordlistparser.c fontforge-20170731/fontforgeexe/wordlistparser.c
--- fontforge-20170731.old/fontforgeexe/wordlistparser.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/fontforgeexe/wordlistparser.c 2018-09-23 14:50:31.595567112 +0530
@@ -799,7 +799,8 @@
if( element_selected )
uc_strcat( ret, "]" );
}
-
+
+ free(wll);
return ret;
}
@@ -841,6 +842,7 @@
{
// we can't go back manually!
u_strcpy( ret, original_data );
+ free(wll);
return ret;
}
}
@@ -862,6 +864,7 @@
uc_strcat( ret, "]" );
}
+ free(wll);
return ret;
}
diff -urN fontforge-20170731.old/gdraw/gimageclut.c fontforge-20170731/gdraw/gimageclut.c
--- fontforge-20170731.old/gdraw/gimageclut.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gimageclut.c 2018-09-23 14:50:31.595567112 +0530
@@ -460,9 +460,10 @@
if ( clut==NULL )
clut = malloc(sizeof(GClut));
- if ( clutmax<2 || clut==NULL )
+ if ( clutmax<2 || clut==NULL ){
+ free(clut);
return( 0 );
-
+ }
clut->clut_len = 0; clut->is_grey = false;
if ( clutmax>256 ) clutmax = 256;
@@ -555,14 +556,24 @@
if ( test->next==NULL )
return( (GCol *) test );
- if (( best = (r-test->red))<0 ) best = -best;
- if (( t = (g-test->green))<0 ) t = -t; best += t;
- if (( t = (b-test->blue))<0 ) t = -t; best += t;
+ if (( best = (r-test->red))<0 )
+ best = -best;
+ if (( t = (g-test->green))<0 )
+ t = -t;
+ best += t;
+ if (( t = (b-test->blue))<0 )
+ t = -t;
+ best += t;
bestcol = test;
for ( test=test->next; test!=NULL; test = test->next ) {
- if (( val = (r-test->red))<0 ) val = -val;
- if (( t = (g-test->green))<0 ) t = -t; val += t;
- if (( t = (b-test->blue))<0 ) t = -t; val += t;
+ if (( val = (r-test->red))<0 )
+ val = -val;
+ if (( t = (g-test->green))<0 )
+ t = -t;
+ val += t;
+ if (( t = (b-test->blue))<0 )
+ t = -t;
+ val += t;
if ( val<best ) {
val = best;
bestcol = test;
@@ -612,14 +623,24 @@
if ( test->next==NULL && this->cols[1]==NULL )
return( (GCol *) test );
- if (( best = (r-test->red))<0 ) best = -best;
- if (( t = (g-test->green))<0 ) t = -t; best += t;
- if (( t = (b-test->blue))<0 ) t = -t; best += t;
+ if (( best = (r-test->red))<0 )
+ best = -best;
+ if (( t = (g-test->green))<0 )
+ t = -t;
+ best += t;
+ if (( t = (b-test->blue))<0 )
+ t = -t;
+ best += t;
bestcol = test;
for ( test=test->next; test!=NULL; test = test->next ) {
- if (( val = (r-test->red))<0 ) val = -val;
- if (( t = (g-test->green))<0 ) t = -t; val += t;
- if (( t = (b-test->blue))<0 ) t = -t; val += t;
+ if (( val = (r-test->red))<0 )
+ val = -val;
+ if (( t = (g-test->green))<0 )
+ t = -t;
+ val += t;
+ if (( t = (b-test->blue))<0 )
+ t = -t;
+ val += t;
if ( val<best ) {
val = best;
bestcol = test;
@@ -627,9 +648,14 @@
}
for ( test=this->cols[1]; test!=NULL; test = test->next ) {
- if (( val = (r-test->red))<0 ) val = -val;
- if (( t = (g-test->green))<0 ) t = -t; val += t;
- if (( t = (b-test->blue))<0 ) t = -t; val += t;
+ if (( val = (r-test->red))<0 )
+ val = -val;
+ if (( t = (g-test->green))<0 )
+ t = -t;
+ val += t;
+ if (( t = (b-test->blue))<0 )
+ t = -t;
+ val += t;
if ( val<best ) {
val = best;
bestcol = test;
@@ -656,18 +682,28 @@
return( old );
bestoff = 3*255;
for ( test=test; test!=NULL; test = test->next ) {
- if ( (off = (r-test->red))<0 ) off = -off;
- if ( (t = (g-test->green))<0 ) t = -t; off +=t;
- if ( (b = (g-test->blue))<0 ) t = -t; off +=t;
+ if ( (off = (r-test->red))<0 )
+ off = -off;
+ if ( (t = (g-test->green))<0 )
+ t = -t;
+ off +=t;
+ if ( (b = (g-test->blue))<0 )
+ t = -t;
+ off +=t;
if ( off<bestoff ) {
bestoff = off;
best = test;
}
}
if ( old!=NULL ) {
- if ( (off = (r-old->red))<0 ) off = -off;
- if ( (t = (g-old->green))<0 ) t = -t; off +=t;
- if ( (b = (g-old->blue))<0 ) t = -t; off +=t;
+ if ( (off = (r-old->red))<0 )
+ off = -off;
+ if ( (t = (g-old->green))<0 )
+ t = -t;
+ off +=t;
+ if ( (b = (g-old->blue))<0 )
+ t = -t;
+ off +=t;
if ( off<bestoff ) {
bestoff = off;
best = old;
diff -urN fontforge-20170731.old/gdraw/gimagecvt.c fontforge-20170731/gdraw/gimagecvt.c
--- fontforge-20170731.old/gdraw/gimagecvt.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gimagecvt.c 2018-09-23 14:50:31.596567096 +0530
@@ -978,6 +978,11 @@
}}
if ( oldrev!=rev )
GClut_RevCMapFree(rev);
+ free(blue_dith);
+ free(green_dith);
+ free(red_dith);
+ free(hor);
+ free(vert);
}
GImage *GImageResize32(GImage *from, GRect *src, int width, int height, Color trans) {
diff -urN fontforge-20170731.old/gdraw/gmatrixedit.c fontforge-20170731/gdraw/gmatrixedit.c
--- fontforge-20170731.old/gdraw/gmatrixedit.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gmatrixedit.c 2018-09-23 14:50:31.596567096 +0530
@@ -40,7 +40,7 @@
static GBox gmatrixedit_box = GBOX_EMPTY; /* Don't initialize here */
static GBox gmatrixedit_button_box = GBOX_EMPTY; /* Don't initialize here */
-static FontInstance *gmatrixedit_font = NULL, *gmatrixedit_titfont = NULL;
+static FontInstance *gmatrixedit_font = NULL, *gmatrixedit_titfont = NULL, *gtitfont = NULL;
static Color gmatrixedit_title_bg = 0x808080, gmatrixedit_title_fg = 0x000000, gmatrixedit_title_divider = 0xffffff;
static Color gmatrixedit_rules = 0x000000;
static Color gmatrixedit_frozencol = 0xff0000,
@@ -115,7 +115,8 @@
GDrawDecomposeFont(gmatrixedit_font,&rq);
rq.point_size = (rq.point_size>=12) ? rq.point_size-2 : rq.point_size>=10 ? rq.point_size-1 : rq.point_size;
rq.weight = 700;
- gmatrixedit_titfont = GResourceFindFont("GMatrixEdit.TitleFont",GDrawInstanciateFont(NULL,&rq));
+ gtitfont = GDrawInstanciateFont(NULL,&rq);
+ gmatrixedit_titfont = GResourceFindFont("GMatrixEdit.TitleFont",gtitfont);
gmatrixedit_title_bg = GResourceFindColor("GMatrixEdit.TitleBG",gmatrixedit_title_bg);
gmatrixedit_title_fg = GResourceFindColor("GMatrixEdit.TitleFG",gmatrixedit_title_fg);
gmatrixedit_title_divider = GResourceFindColor("GMatrixEdit.TitleDivider",gmatrixedit_title_divider);
diff -urN fontforge-20170731.old/gdraw/gmenu.c fontforge-20170731/gdraw/gmenu.c
--- fontforge-20170731.old/gdraw/gmenu.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gmenu.c 2018-09-23 14:50:31.596567096 +0530
@@ -145,6 +145,7 @@
menu_3d_look = GResourceFindBool("GMenu.3DLook", menu_3d_look);
gmenubar_inited = true;
_GGroup_Init();
+ free(keystr);
}
typedef struct gmenu {
@@ -388,8 +389,12 @@
strcat(buffer,".");
if( stack[i]->ti.text_untranslated )
{
-// TRACE("adding %s\n", HKTextInfoToUntranslatedTextFromTextInfo( &stack[i]->ti ));
- strcat( buffer, HKTextInfoToUntranslatedTextFromTextInfo( &stack[i]->ti ));
+ char *res;
+ res = calloc(PATH_MAX, sizeof(char));
+ res = HKTextInfoToUntranslatedTextFromTextInfo( &stack[i]->ti );
+// TRACE("adding %s\n", res);
+ strcat( buffer, res);
+ free(res);
}
else if( stack[i]->ti.text )
{
diff -urN fontforge-20170731.old/gdraw/gpstxtinit.c fontforge-20170731/gdraw/gpstxtinit.c
--- fontforge-20170731.old/gdraw/gpstxtinit.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gpstxtinit.c 2018-09-23 14:50:31.596567096 +0530
@@ -503,6 +503,7 @@
}
}
fonts->names_loaded = true;
+ free(path);
return( fonts->names_loaded );
}
diff -urN fontforge-20170731.old/gdraw/gtextinfo.c fontforge-20170731/gdraw/gtextinfo.c
--- fontforge-20170731.old/gdraw/gtextinfo.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gtextinfo.c 2018-09-23 14:50:31.597567080 +0530
@@ -582,6 +582,7 @@
free(fname);
ri->filename = fname = absname;
}
+ free(absname);
}
if ( ret==NULL ) {
ri->filename = NULL;
diff -urN fontforge-20170731.old/gdraw/gxdraw.c fontforge-20170731/gdraw/gxdraw.c
--- fontforge-20170731.old/gdraw/gxdraw.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gdraw/gxdraw.c 2018-09-23 14:50:31.597567080 +0530
@@ -910,7 +910,6 @@
}
if ( (wattrs->mask&wam_ititle) && wattrs->icon_title!=NULL ) {
XmbSetWMProperties(display,nw->w,NULL,(pt = u2def_copy(wattrs->icon_title)),NULL,0,NULL,NULL,NULL);
- free(pt);
}
if ( (wattrs->mask&wam_utf8_wtitle) && wattrs->utf8_window_title!=NULL ) {
#ifdef X_HAVE_UTF8_STRING
@@ -918,7 +917,7 @@
#else
unichar_t *tit = utf82u_copy(wattrs->utf8_window_title);
XmbSetWMProperties(display,nw->w,(pt = u2def_copy(tit)),NULL,NULL,0,NULL,NULL,NULL);
- free(pt); free(tit);
+ free(tit);
#endif
}
if ( (wattrs->mask&wam_utf8_ititle) && wattrs->utf8_icon_title!=NULL ) {
@@ -927,7 +926,7 @@
#else
unichar_t *tit = utf82u_copy(wattrs->utf8_icon_title);
XmbSetWMProperties(display,nw->w,NULL,(pt = u2def_copy(tit)),NULL,0,NULL,NULL,NULL);
- free(pt); free(tit);
+ free(tit);
#endif
}
s_h.x = pos->x; s_h.y = pos->y;
diff -urN fontforge-20170731.old/gutils/gimagereadrgb.c fontforge-20170731/gutils/gimagereadrgb.c
--- fontforge-20170731.old/gutils/gimagereadrgb.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gutils/gimagereadrgb.c 2018-09-23 14:50:31.597567080 +0530
@@ -313,6 +313,7 @@
free(r); free(g); free(b); free(a);
}
}
+ free(fp);
return( ret );
errorGImageReadRgbFile:
diff -urN fontforge-20170731.old/gutils/gioftp.c fontforge-20170731/gutils/gioftp.c
--- fontforge-20170731.old/gutils/gioftp.c 2017-07-31 03:19:01.000000000 +0530
+++ fontforge-20170731/gutils/gioftp.c 2018-09-23 14:50:31.598567065 +0530
@@ -697,6 +697,7 @@
if ( ctl!=-1 ) close(ctl);
free(path);
free(host); free(username); free(password);
+ free(had);
return( NULL );
}
diff -urN fontforge-20170731.old/pycontrib/FontCompare/fc/GlyphConsistency.py fontforge-20170731/pycontrib/FontCompare/fc/GlyphConsistency.py
--- fontforge-20170731.old/pycontrib/FontCompare/fc/GlyphConsistency.py 2018-09-21 13:53:18.533769952 +0530
+++ fontforge-20170731/pycontrib/FontCompare/fc/GlyphConsistency.py 2018-09-23 14:50:31.598567065 +0530
@@ -83,7 +83,7 @@
if not font[i].boundingBox()[3] in ymax_cords:
ymax_cords.append(font[i].boundingBox()[3])
if not font[i].left_side_bearing in lbearing:
- rbearing.append(font[i].left_side_bearing)
+ lbearing.append(font[i].left_side_bearing)
if not font[i].right_side_bearing in rbearing:
rbearing.append(font[i].right_side_bearing)
if not font[i].layer_cnt in layer_cnts: