https://bugzilla.mindrot.org/show_bug.cgi?id=2373

--- Comment #1 from Darren Tucker <[email protected]> ---
Comment on attachment 2577
  --> https://bugzilla.mindrot.org/attachment.cgi?id=2577
patch

>-          get_local_ipaddr(sock_in), get_local_port());
>+          (addr = get_local_ipaddr(sock_in)), get_local_port());

I think it'd be cleaner to do that assignment on a separate line rather
than buried in the format string args.

>+      xfree(addr);

What version did you test this against?  xfree() is long gone.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
_______________________________________________
openssh-bugs mailing list
[email protected]
https://lists.mindrot.org/mailman/listinfo/openssh-bugs

Reply via email to