Re: git: Wrong parsing of ssh urls with IPv6 literals ignores port

On Wed, Jun 13, 2012 at 10:21:04AM -0700, Junio C Hamano wrote:

René Scharfe [off-list ref] writes:

quoted

Am 12.06.2012 23:00, schrieb Jonathan Nieder:

quoted

quoted

Oh, hold on a second.  Won't this get confused by

	ssh://[::1]/foo/bar/baz:80/qux

[...]

quoted

It shouldn't, because the host part is NUL-terminated before
get_port() is called.  Let's see (with the patch):

[...]

Yeah, I was wondering how that would get confused myself.  Jonathan,
ack again?

Yeah.  I had missed that when proto == PROTO_SSH that means the proto
!= PROTO_LOCAL branch has been taken and the port is NUL-terminated.

So

Reviewed-by: Jonathan Nieder <redacted>

It seems like a good fix given the current code structure.  Sorry for
the false alarm.