HTCondorWiki: Ipv Six Work Note

Page History

What should be changed?

All the deprecated BSD socket functions. These are complete list.

struct sockaddr_in
gethostbyname
gethostbyname2
getservbyname
gethostbyaddr
getservbyport
inet_addr
inet_aton
inet_nsap_addr
inet_ntoa
inet_nsap_ntoa
inet_makeaddr
inet_netof
inet_network
inet_neta
inet_net_netop
inet_net_pton
rcmd
rexec
rrsevport

IP address is no longer fixed 4 bytes nor fixed 16 bytes. It can be both of them. So, every storage class should be changed. For some of source codes that use 'int' as storage for IP address, this is most troublesome because it could be hidden to text search.

Current Condor Code Base

sockaddr_in is used in many areas and also unsigned long is used to hold IP address.

There are many places to change. For example, in sock.h,

unsigned int Sock::get_ip_int()
unsigned int Sock::peer_ip_int()

It uses 'unsigned int' to pass IP address.

The method of Attack

Having some network abstraction classes, single IP address class that deals both IPv4 and IPv6. So, nobody uses sockaddr*, sockaddr_in, or just int to represent IP address.

class Sock already has abstraction of BSD socket interface. But, IP address escapes from this classes by returning it as an 4 byte int and sockaddr_in.

Todo

Manageable daily work-list.

investigate every source code where calls obsolete BSD socket interface.
remove every obsolete interface.
investigate ipv6 address handling in C++ Boost library
make unified address class (is it really required? since ipv6 can contain ipv4 addr)

Note

internet.h seems to have a lot of IP-v6 incompatible functions.
why do you call <a.b.c.d:pppp> as sinful string? what does it mean 'sinful'?