aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Snyder <msnyder@vmware.com>2007-07-03 01:01:13 +0000
committerMichael Snyder <msnyder@vmware.com>2007-07-03 01:01:13 +0000
commit7877e97730123929da00a6fd91c7b8257f26d027 (patch)
tree5b6927908ab6cccac4c502a7ef843d57b9bbc2d9
parent3d07a0f02eeb9abaf9eef3f2b8c2782571ad338b (diff)
downloadgdb-7877e97730123929da00a6fd91c7b8257f26d027.zip
gdb-7877e97730123929da00a6fd91c7b8257f26d027.tar.gz
gdb-7877e97730123929da00a6fd91c7b8257f26d027.tar.bz2
2007-07-02 Michael Snyder <msnyder@access-company.com>
* p-exp.y (yylex): Memory leak, 'uptokstart' must be freed (Coverity).
-rw-r--r--gdb/ChangeLog4
-rw-r--r--gdb/p-exp.y38
2 files changed, 36 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e1f7e3e..30a1652 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2007-07-02 Michael Snyder <msnyder@access-company.com>
+
+ * p-exp.y (yylex): Memory leak, 'uptokstart' must be freed (Coverity).
+
2007-07-02 Daniel Jacobowitz <dan@codesourcery.com>
* Makefile.in (XMLFILES): Add library-list.dtd.
diff --git a/gdb/p-exp.y b/gdb/p-exp.y
index 6834441..92f93ff 100644
--- a/gdb/p-exp.y
+++ b/gdb/p-exp.y
@@ -1342,6 +1342,7 @@ yylex ()
removed from the input stream. */
if (namelen == 2 && uptokstart[0] == 'I' && uptokstart[1] == 'F')
{
+ free (uptokstart);
return 0;
}
@@ -1354,18 +1355,31 @@ yylex ()
{
case 6:
if (DEPRECATED_STREQ (uptokstart, "OBJECT"))
- return CLASS;
+ {
+ free (uptokstart);
+ return CLASS;
+ }
if (DEPRECATED_STREQ (uptokstart, "RECORD"))
- return STRUCT;
+ {
+ free (uptokstart);
+ return STRUCT;
+ }
if (DEPRECATED_STREQ (uptokstart, "SIZEOF"))
- return SIZEOF;
+ {
+ free (uptokstart);
+ return SIZEOF;
+ }
break;
case 5:
if (DEPRECATED_STREQ (uptokstart, "CLASS"))
- return CLASS;
+ {
+ free (uptokstart);
+ return CLASS;
+ }
if (DEPRECATED_STREQ (uptokstart, "FALSE"))
{
yylval.lval = 0;
+ free (uptokstart);
return FALSEKEYWORD;
}
break;
@@ -1373,6 +1387,7 @@ yylex ()
if (DEPRECATED_STREQ (uptokstart, "TRUE"))
{
yylval.lval = 1;
+ free (uptokstart);
return TRUEKEYWORD;
}
if (DEPRECATED_STREQ (uptokstart, "SELF"))
@@ -1384,7 +1399,10 @@ yylex ()
if (lookup_symbol (this_name, expression_context_block,
VAR_DOMAIN, (int *) NULL,
(struct symtab **) NULL))
- return THIS;
+ {
+ free (uptokstart);
+ return THIS;
+ }
}
break;
default:
@@ -1401,6 +1419,7 @@ yylex ()
so in expression to enter hexadecimal values
we still need to use C syntax with 0xff */
write_dollar_variable (yylval.sval);
+ free (uptokstart);
return VARIABLE;
}
@@ -1493,6 +1512,7 @@ yylex ()
strncpy (tempbuf, tokstart, namelen); tempbuf [namelen] = 0;
yylval.sval.ptr = tempbuf;
yylval.sval.length = namelen;
+ free (uptokstart);
return FIELDNAME;
}
/* Call lookup_symtab, not lookup_partial_symtab, in case there are
@@ -1503,6 +1523,7 @@ yylex ()
{
yylval.ssym.sym = sym;
yylval.ssym.is_a_field_of_this = is_a_field_of_this;
+ free (uptokstart);
return BLOCKNAME;
}
if (sym && SYMBOL_CLASS (sym) == LOC_TYPEDEF)
@@ -1593,13 +1614,17 @@ yylex ()
#else /* not 0 */
yylval.tsym.type = SYMBOL_TYPE (sym);
#endif /* not 0 */
+ free (uptokstart);
return TYPENAME;
}
yylval.tsym.type
= language_lookup_primitive_type_by_name (current_language,
current_gdbarch, tmp);
if (yylval.tsym.type != NULL)
- return TYPENAME;
+ {
+ free (uptokstart);
+ return TYPENAME;
+ }
/* Input names that aren't symbols but ARE valid hex numbers,
when the input radix permits them, can be names or numbers
@@ -1614,6 +1639,7 @@ yylex ()
{
yylval.ssym.sym = sym;
yylval.ssym.is_a_field_of_this = is_a_field_of_this;
+ free (uptokstart);
return NAME_OR_INT;
}
}