[PATCH] XDMCP: For IPv6 add IPv6 link local addresses to the end of the list
Egbert Eich
eich at freedesktop.org
Sat Feb 8 10:45:54 PST 2014
On Sat, Feb 08, 2014 at 08:58:34PM +0600, Alexander E. Patrakov wrote:
>
> I have looked, and I have some remarks.
Thanks for doing so!
>
> First, there is an almost identical copy-paste between the two places where
> entries are added to linklocal_list. Maybe it's worth making that a
> function? And why the difference between IN6_IS_ADDR_LINKLOCAL in one place
> and +IN6_IS_ADDR_LINKLOCAL in the other?
>
> Second, that piece doesn't deal with realloc failures.
Right.
>
> Third, the commit message does not reassure the readers that XDMCP over
> link-local addresses works at all (I have not tried this) and is a
> supportable configuration. If any of these conditions are false, then let's
> just drop these addresses, like we do with IN6_IS_ADDR_LOOPBACK.
AFAIR they don't. This was why this issue had popped up.
>
> ... OK, actually I have tested IPv6 UDP communication using link-local
> addresses using netcat6. It doesn't work (-EINVAL on bind() that is used to
> set the source address and port), so I will be very much surprised if XDMCP
> works. If that requirement did not violate the list rules, I would even
> require a tcpdump capture of a successful XDMCP session and some X11 traffic
> along with the patch.
Ok, it's been a while and I didn't write the patch but I believe the whole
reason for this patch was to move the link-local addresses out of the way
ie. at the end of the list where they don't bother anybody.
Therefore dropping them would also be an option - codewise it would be simpler
and address most of the other concerns voiced.
Reinhard, what do you thing?
Thanks for reviewing!
Cheers,
Egbert.
More information about the xorg-devel
mailing list