* [RFA] readelf.c (process_version_sections): Free symbols.
@ 2011-03-08 0:54 Michael Snyder
2011-03-14 12:09 ` Richard Sandiford
0 siblings, 1 reply; 2+ messages in thread
From: Michael Snyder @ 2011-03-08 0:54 UTC (permalink / raw)
To: binutils
[-- Attachment #1: Type: text/plain, Size: 5 bytes --]
OK?
[-- Attachment #2: readelf.txt --]
[-- Type: text/plain, Size: 915 bytes --]
2011-03-07 Michael Snyder <msnyder@vmware.com>
* readelf.c (process_version_sections): Free symbols.
Index: readelf.c
===================================================================
RCS file: /cvs/src/src/binutils/readelf.c,v
retrieving revision 1.534
diff -u -p -r1.534 readelf.c
--- readelf.c 1 Mar 2011 00:02:32 -0000 1.534
+++ readelf.c 8 Mar 2011 00:51:29 -0000
@@ -8053,7 +8053,10 @@ process_version_sections (FILE * file)
string_sec->sh_size,
_("version string table"));
if (!strtab)
- break;
+ {
+ free (symbols);
+ break;
+ }
printf (_("\nVersion symbols section '%s' contains %d entries:\n"),
SECTION_NAME (section), total);
@@ -8073,6 +8076,7 @@ process_version_sections (FILE * file)
if (!edata)
{
free (strtab);
+ free (symbols);
break;
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [RFA] readelf.c (process_version_sections): Free symbols.
2011-03-08 0:54 [RFA] readelf.c (process_version_sections): Free symbols Michael Snyder
@ 2011-03-14 12:09 ` Richard Sandiford
0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2011-03-14 12:09 UTC (permalink / raw)
To: Michael Snyder; +Cc: binutils
Michael Snyder <msnyder@vmware.com> writes:
> OK?
Yes, thanks.
> 2011-03-07 Michael Snyder <msnyder@vmware.com>
>
> * readelf.c (process_version_sections): Free symbols.
>
> Index: readelf.c
> ===================================================================
> RCS file: /cvs/src/src/binutils/readelf.c,v
> retrieving revision 1.534
> diff -u -p -r1.534 readelf.c
> --- readelf.c 1 Mar 2011 00:02:32 -0000 1.534
> +++ readelf.c 8 Mar 2011 00:51:29 -0000
> @@ -8053,7 +8053,10 @@ process_version_sections (FILE * file)
> string_sec->sh_size,
> _("version string table"));
> if (!strtab)
> - break;
> + {
> + free (symbols);
> + break;
> + }
>
> printf (_("\nVersion symbols section '%s' contains %d entries:\n"),
> SECTION_NAME (section), total);
> @@ -8073,6 +8076,7 @@ process_version_sections (FILE * file)
> if (!edata)
> {
> free (strtab);
> + free (symbols);
> break;
> }
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-03-14 12:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-08 0:54 [RFA] readelf.c (process_version_sections): Free symbols Michael Snyder
2011-03-14 12:09 ` Richard Sandiford
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).