[PATCH] mux: fix memory leak of control path if bind() fails

Bert Wesarg bert.wesarg at googlemail.com
Fri Apr 27 21:47:07 EST 2012


2012/4/27 Dag-Erling Smørgrav <des at des.no>:
> Bert Wesarg <bert.wesarg at googlemail.com> writes:
>> +                     xfree(orig_control_path);
>
> so you want to free the buffer
>
>>               error("ControlSocket %s already exists, disabling multiplexing",
>>                   orig_control_path);
>
> about 20 lines before passing it to a printf() equivalent?

Luckily there is a return only 4 lines after the free.

Bert

>
> DES
> --
> Dag-Erling Smørgrav - des at des.no


More information about the openssh-unix-dev mailing list