Re: [patch] target_read_string, potential memory leak.

Jim Blandy
2007-08-06T22:30:49Z

msnyder@sonic.net writes:
> OK -- it would be pretty stupid to call target_read_string without
> passing it a buffer, but if we're going to check it for null...

I don't think the contract for target_read_string permits STRING to be
null. I think either the test should go, or we should add a
gdb_assert (string).