[PATCH] gdbserver: fix memory leak when handling qsupported packet

Simon Marchi simon.marchi@polymtl.ca
Tue Jul 14 02:28:39 GMT 2020


On 2020-07-13 2:41 p.m., Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:
> 
> Simon> gdbserver/ChangeLog:
> 
> Simon> 	* server.cc (handle_query): Use std::vector of
> Simon> 	std::string for `qsupported` vector.  Use separate
> Simon> 	vector for unknowns.
> Simon> 	* target.h (class process_stratum_target) <process_qsupported>:
> Simon> 	Change parameters to array_view of const char *.
> Simon> 	(target_process_qsupported): Remove `count` parameter.
> Simon> 	* target.cc (process_stratum_target::process_qsupported): Change
> Simon> 	parameters to array_view of const char *.
> Simon> 	* linux-x86-low.cc (class x86_target) <process_qsupported>:
> Simon> 	Likewise.
> 
> FWIW this seems fine to me.  Thanks for cleaning this up.
> 
> Tom
> 

Thanks, I pushed it.

Simon


More information about the Gdb-patches mailing list