Jan Christoph Nordholz <he...@pool.math.tu-berlin.de> writes:

> found while using aklog, so reporting against -krb5...

This code doesn't exist in the current version of openafs in unstable or
testing, so I assume you're using the version from experimental?

>     /* p[] is a byte pointer into the DNS answer to a type SRV query
>        and points into the contents of the SRV RR right at the priority,
>        so the dest port is at (p+4) */
> ]   ports[server_num] = (p[4] << 8) | p[5];

> So ports[] is manually extracted from the DNS answer and thus
> implicitely converted from network to host byte order, but not converted
> back when assigning the (struct sin_addr).sin_port member.

> Suggestion: Wrap either assignment with htons().

Thanks!  I'm submitting a patch upstream to fix this.

-- 
Russ Allbery (r...@debian.org)               <http://www.eyrie.org/~eagle/>



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to