[binutils-gdb] Remove two cleanups using std::string

[sergiodj+buildbot at sergiodj dot net]

*** TEST RESULTS FOR COMMIT b577b6af8e7f4cc53e73ee01188d1700fb8d3b82 ***

Author: Tom Tromey <tom@tromey.com>
Branch: master
Commit: b577b6af8e7f4cc53e73ee01188d1700fb8d3b82

Remove two cleanups using std::string

This patches removes cleanups from a couple of spots by using
std::string rather than manual memory management.

Regression tested by the buildbot, though note that I don't believe
the buildbot actually exercises the machoread code.

gdb/ChangeLog
2018-03-13  Tom Tromey  <tom@tromey.com>

	* machoread.c (macho_check_dsym): Change filenamep to a
	std::string*.
	(macho_symfile_read): Update.
	* symfile.c (load_command): Use std::string.