Commit96ad72d

Michael Meskes

committed

Fixed some memory leaks in ECPG.

Patch by Michael Paquier

1 parent82be1bf commit96ad72dCopy full SHA for 96ad72d

File tree

+15

-5

lines changed

+15

-5

lines changed

Lines changed: 9 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -175,6 +175,7 @@ output_get_descr(char desc_name, char index)`
`175`	`175`	`for (results=assignments;results!=NULL;results=results->next)`
`176`	`176`	`{`
`177`	`177`	`conststructvariable*v=find_variable(results->variable);`
	`178`	`+char*str_zero=mm_strdup("0");`
`178`	`179`
`179`	`180`	`switch (results->value)`
`180`	`181`	`{`
`@@ -188,7 +189,8 @@ output_get_descr(char desc_name, char index)`
`188`	`189`	`break;`
`189`	`190`	`}`
`190`	`191`	`fprintf(yyout,"%s,",get_dtype(results->value));`
`191`		`-ECPGdump_a_type(yyout,v->name,v->type,v->brace_level,NULL,NULL,-1,NULL,NULL,mm_strdup("0"),NULL,NULL);`
	`192`	`+ECPGdump_a_type(yyout,v->name,v->type,v->brace_level,NULL,NULL,-1,NULL,NULL,str_zero,NULL,NULL);`
	`193`	`+free(str_zero);`
`192`	`194`	`}`
`193`	`195`	`drop_assignments();`
`194`	`196`	`fputs("ECPGd_EODT);\n",yyout);`
`@@ -292,8 +294,12 @@ output_set_descr(char desc_name, char index)`
`292`	`294`	`caseECPGd_indicator:`
`293`	`295`	`caseECPGd_length:`
`294`	`296`	`caseECPGd_type:`
`295`		`-fprintf(yyout,"%s,",get_dtype(results->value));`
`296`		`-ECPGdump_a_type(yyout,v->name,v->type,v->brace_level,NULL,NULL,-1,NULL,NULL,mm_strdup("0"),NULL,NULL);`
	`297`	`+{`
	`298`	`+char*str_zero=mm_strdup("0");`
	`299`	`+fprintf(yyout,"%s,",get_dtype(results->value));`
	`300`	`+ECPGdump_a_type(yyout,v->name,v->type,v->brace_level,NULL,NULL,-1,NULL,NULL,str_zero,NULL,NULL);`
	`301`	`+free(str_zero);`
	`302`	`+}`
`297`	`303`	`break;`
`298`	`304`
`299`	`305`	`default:`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@`
`27`	`27`	`extern YYSTYPE yylval;`
`28`	`28`
`29`	`29`	`staticintxcdepth =0;/* depth of nesting in slash-star comments*/`
`30`		`-staticchar dolqstart;/ current $foo$ quote start string*/`
	`30`	`+staticchar dolqstart =NULL;/ current $foo$ quote start string*/`
`31`	`31`	`static YY_BUFFER_STATE scanbufhandle;`
`32`	`32`	`staticchar *scanbuf;`
`33`	`33`
`@@ -550,6 +550,8 @@ cppline{space}#([^i][A-Za-z]\|{if}\|{ifdef}\|{ifndef}\|{import})(.\\{space}).`
`550`	`550`	`}`
`551`	`551`	`<SQL>{dolqdelim} {`
`552`	`552`	`token_start = yytext;`
	`553`	`+if (dolqstart)`
	`554`	`+free(dolqstart);`
`553`	`555`	`dolqstart =mm_strdup(yytext);`
`554`	`556`	`BEGIN(xdolq);`
`555`	`557`	`startlit();`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -437,11 +437,13 @@ remove_variable_from_list(struct arguments ** list, struct variable * var)`
`437`	`437`	`void`
`438`	`438`	`dump_variables(structarguments*list,intmode)`
`439`	`439`	`{`
`440`		`-char*str_zero=mm_strdup("0");`
	`440`	`+char*str_zero;`
`441`	`441`
`442`	`442`	`if (list==NULL)`
`443`	`443`	`return;`
`444`	`444`
	`445`	`+str_zero=mm_strdup("0");`
	`446`	`+`
`445`	`447`	`/*`
`446`	`448`	`* The list is build up from the beginning so lets first dump the end of`
`447`	`449`	`* the list:`

Comments

(0)