Bruce Momjian <bruce@momjian.us> writes:
> Uh, you are saying pg_ctl leaks memory or the database server leaks
> memory? I don't see any memory allocation in the code you posted.
I've not looked at the code yet, but I think the OP is worried that
we need to free(newval) before these "return"s.
regards, tom lane