Fix memory leak in add_symbol_file_command
authorTom Tromey <tom@tromey.com>
Thu, 3 Aug 2017 23:07:06 +0000 (17:07 -0600)
committerTom Tromey <tom@tromey.com>
Mon, 14 Aug 2017 14:31:07 +0000 (08:31 -0600)
commitf978cb06dbfbd93dbd52bd39d992f8644b0c639e
tree094f0fe74f05bd69cc6622369c3c40c8a1526ca7
parent2f5404b3581d125a96f14d92c4a7985e8156d1d6
Fix memory leak in add_symbol_file_command

I happened to notice that add_symbol_file_command leaks "sect_opts".
This patch fixes the leak by changing sect_opts to be a std::vector.

I had to change the logic in the loop a little bit.  Previously, it
was incrementing section_index after completing an entry; but this
changes it to push a new entry when the name is seen.

I believe the argument parsing here is mildly incorrect, in that
nothing checks whether the -s option actually had any arguments.
Maybe gdb can crash if "-s NAME" is given without an argument.  I
didn't try to fix this in this patch, but I do have another patch I
can send later that fixes it up.

Regression tested on the buildbot.

ChangeLog
2017-08-11  Tom Tromey  <tom@tromey.com>

* symfile.c (add_symbol_file_command): Use std::vector.
gdb/ChangeLog
gdb/symfile.c