Skip to content

W
wine-winehq
Commit 31c5652d authored by Mike McCormack's avatar Mike McCormack Committed by Alexandre Julliard
Browse files
Options

msi: Fix some memory leaks in the condition parser.

parent f3dae17c
Hide whitespace changes
Inline Side-by-side
Showing with 5 additions and 0 deletions
dlls/msi/cond.y
View file @ 31c5652d
...@@ -186,6 +186,7 @@ boolean_factor: ...@@ -186,6 +186,7 @@ boolean_factor:
| value_s | value_s
{ {
$$ = ($1 && $1[0]) ? 1 : 0; $$ = ($1 && $1[0]) ? 1 : 0;
msi_free($1);
} }
| value_i operator value_i | value_i operator value_i
{ {
...@@ -198,6 +199,7 @@ boolean_factor: ...@@ -198,6 +199,7 @@ boolean_factor:
$$ = compare_int( num, $2, $3 ); $$ = compare_int( num, $2, $3 );
else else
$$ = ($2 == COND_NE || $2 == COND_INE ); $$ = ($2 == COND_NE || $2 == COND_INE );
msi_free($1);
} }
| value_i operator symbol_s | value_i operator symbol_s
{ {
...@@ -206,6 +208,7 @@ boolean_factor: ...@@ -206,6 +208,7 @@ boolean_factor:
$$ = compare_int( $1, $2, num ); $$ = compare_int( $1, $2, num );
else else
$$ = ($2 == COND_NE || $2 == COND_INE ); $$ = ($2 == COND_NE || $2 == COND_INE );
msi_free($3);
} }
| symbol_s operator symbol_s | symbol_s operator symbol_s
{ {
...@@ -226,10 +229,12 @@ boolean_factor: ...@@ -226,10 +229,12 @@ boolean_factor:
| literal operator value_i | literal operator value_i
{ {
$$ = 0; $$ = 0;
msi_free($1);
} }
| value_i operator literal | value_i operator literal
{ {
$$ = 0; $$ = 0;
msi_free($3);
} }
| COND_LPAR expression COND_RPAR | COND_LPAR expression COND_RPAR
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment