Subject: Re: racoon broken in -current: grab_myaddrs and SIOCGIFCONF
To: Greg Troxel <gdt@ir.bbn.com>
From: Quentin Garnier <cube@cubidou.net>
List: tech-net
Date: 08/30/2007 17:59:02
--eOZ5LUtYRk41oVSR
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Thu, Aug 30, 2007 at 11:48:53AM -0400, Greg Troxel wrote:
> racoon doesn't work in current because SIOCGIFCONF is broken. (I know
> racoon should probably not use SIOCGIFCONF, but instead getifaddrs. I
> won't argue, but SIOCGIFCONF should work in any case.)
>=20
> As far as I can tell there are two things wrong:
>=20
> * commit to sys/net/if.h to include sockaddr_storage in struct ifreq.
> This breaks ABI compatability and things don't seem versioned. I
> don't understand the rationale for this change.
Yes they are. Granted, the code is hard to understand.
Besides, SIOCGIFCONF is explicitely versioned.
[...]
> Christos: can you explain which ioctls were trouble? I don't follow
> from your commit message below.
The issue is that ifreq was used to store stuff larger that struct
sockaddr. Christos's change makes ifreq suitable for any address
family, which is a good thing IMO.
--=20
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"You could have made it, spitting out benchmarks
Owe it to yourself not to fail"
Amplifico, Spitting Out Benchmarks, Hometakes Vol. 2, 2005.
--eOZ5LUtYRk41oVSR
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (NetBSD)
iQEVAwUBRtbpRdgoQloHrPnoAQKI8Af/fgdw/M+KZdnGLL4prb5j/tu+Clwgks7T
PsAc2TK/fpNlBOv8kzPYl1xsIikzMPRVLYtlfJs38SLNvVdkhOx5CueAksfkQPUx
p+eCli4xRcYWxk4GoBplXKZi5K0keAfdIlWizUGMGs3O1DOnkqK6GdvpyKzVgQj5
Jtg8c+Ckq98feSMyMkfltYHb86yTSZUOP3Mr/wLeFEcXMWSz8CoSHKie3BOeuhfR
COI2ufaLoZdk6xZB1VApdq3+tb0lZdQ22icO5P7xUz2TxJAh5TilzqYLomU3jrr1
iuxF9gyOfliB+X2KSNwtYUtpu/SNb5SVcCR38YYT4eojgvTnfIdLQQ==
=j8Z+
-----END PGP SIGNATURE-----
--eOZ5LUtYRk41oVSR--