On Fri, Aug 12, 2005 at 05:07:51PM +0200, Benoit Panizzon wrote:
> sub my_inet_aton { unpack("N", pack("C4", split(/\./, $_[0]))) }
> 
> What is that done for? Why would somebody want to pack an IPv4 address into a 
> binary notation as Net::DNS and Net::IP don't need this packing stuff?

Probably because we need it packed for other things.

Grepping for "my_inet_aton" in the code leads me to the NetSet module:

# Mail::SpamAssassin::NetSet - object to manipulate CIDR net IP addrs

Also, the URIDNSBL plugin and Dns module seem to use it when they're
matching results.

> This breaks the whole IPv6 support.

I don't think SA currently supports IPv6 for RBLs, so it's not actually
breaking anything.

-- 
Randomly Generated Tagline:
"... until this thing blows up, or yells 'stop', or says Uncle or 
 something..."                  - Prof. Demetry

Attachment: pgpsClXPxMpuz.pgp
Description: PGP signature

Reply via email to