X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=Modules%2FIPStack%2Fmain.c;h=5aa671520bf7809ef2f6c6f4449a1aff199a5fa9;hb=e7d0a33cc9b8bf0005799ab2fb5151f78a255772;hp=2b04a6a560f0024847e10650811c21d8591cc23c;hpb=4dcdedff0ff1494189773c2d724f76a0291ca155;p=tpg%2Facess2.git

diff --git a/Modules/IPStack/main.c b/Modules/IPStack/main.c
index 2b04a6a5..5aa67152 100644
--- a/Modules/IPStack/main.c
+++ b/Modules/IPStack/main.c
@@ -26,7 +26,7 @@ extern tRoute	*IPStack_AddRoute(const char *Interface, void *Network, int Subnet
 
 // === PROTOTYPES ===
  int	IPStack_Install(char **Arguments);
- int	IPStack_CompareAddress(int AddressType, void *Address1, void *Address2, int CheckBits);
+ int	IPStack_CompareAddress(int AddressType, const void *Address1, const void *Address2, int CheckBits);
 
 // === GLOBALS ===
 MODULE_DEFINE(0, VERSION, IPStack, IPStack_Install, NULL, NULL);
@@ -112,6 +112,10 @@ int IPStack_Install(char **Arguments)
 					UnHex(addrData, size, addr);
 					
 					tInterface	*iface = IPStack_AddInterface(dev, "");
+					if( !iface ) {
+						Log_Warning("IPStack", "Unable to add interface on '%s'", dev);
+						continue ;
+					}
 					iface->Type = iType;
 					memcpy(iface->Address, addrData, size);
 					iface->SubnetBits = iBits;
@@ -214,17 +218,17 @@ int IPStack_GetAddressSize(int AddressType)
 /**
  * \brief Compare two IP Addresses masked by CheckBits
  */
-int IPStack_CompareAddress(int AddressType, void *Address1, void *Address2, int CheckBits)
+int IPStack_CompareAddress(int AddressType, const void *Address1, const void *Address2, int CheckBits)
 {
 	 int	size = IPStack_GetAddressSize(AddressType);
 	Uint8	mask;
-	Uint8	*addr1 = Address1, *addr2 = Address2;
+	const Uint8	*addr1 = Address1, *addr2 = Address2;
 	
 	// Sanity check size
 	if( CheckBits < 0 )	CheckBits = 0;
 	if( CheckBits > size*8 )	CheckBits = size*8;
 	
-	if( CheckBits == 0 )	return 1;	// /0 matches anythin0
+	if( CheckBits == 0 )	return 1;	// /0 matches anything
 	
 	// Check first bits/8 bytes
 	if( memcmp(Address1, Address2, CheckBits/8) != 0 )	return 0;
@@ -240,20 +244,20 @@ int IPStack_CompareAddress(int AddressType, void *Address1, void *Address2, int
 	return 0;
 }
 
-const char *IPStack_PrintAddress(int AddressType, void *Address)
+const char *IPStack_PrintAddress(int AddressType, const void *Address)
 {
 	switch( AddressType )
 	{
 	case 4: {
 		static char	ret[4*3+3+1];
-		Uint8	*addr = Address;
+		const Uint8	*addr = Address;
 		sprintf(ret, "%i.%i.%i.%i", addr[0], addr[1], addr[2], addr[3]);
 		return ret;
 		}
 	
-	case 6: {
+	case 6: {	// TODO: address compression
 		static char	ret[8*4+7+1];
-		Uint16	*addr = Address;
+		const Uint16	*addr = Address;
 		sprintf(ret, "%x:%x:%x:%x:%x:%x:%x:%x",
 			addr[0], addr[1], addr[2], addr[3],
 			addr[4], addr[5], addr[6], addr[7]