Subject: Re: CVS commit: src/lib/libpcap
To: None <M.Drochner@fz-juelich.de>
From: Christos Zoulas <christos@zoulas.com>
List: source-changes
Date: 03/20/2006 13:17:31
On Mar 20,  7:07pm, M.Drochner@fz-juelich.de (Matthias Drochner) wrote:
-- Subject: Re: CVS commit: src/lib/libpcap

| 
| christos@astron.com said:
| > 	masklen == 128 = (16 * 8)
| > 	mask.s6_addr[masklen / 8] = mask.s6_addr[16] = oops, since in6_addr: 
| 
| A masklen of 128 is certainly legal, but in that singular case
| (masklen % 8) is zero so that the code in question is not executed.

Yes, I see that now. A comment in the code would help.

christos