X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=Usermode%2FApplications%2Fdhcpclient_src%2Fmain.c;h=d6f4cda0565ecaa2a1cbe04f30282162024bb990;hb=HEAD;hp=affe1fd171706cc54aa861b09d74d1b31222f9c2;hpb=de359870d96a2b6c35da68ba400870657fbf3615;p=tpg%2Facess2.git

diff --git a/Usermode/Applications/dhcpclient_src/main.c b/Usermode/Applications/dhcpclient_src/main.c
index affe1fd1..d6f4cda0 100644
--- a/Usermode/Applications/dhcpclient_src/main.c
+++ b/Usermode/Applications/dhcpclient_src/main.c
@@ -77,7 +77,8 @@ typedef struct sInterface
 
 	uint32_t	TransactionID;
 	char	HWAddr[6];
-	
+
+	int64_t	StartTime;	
 	int64_t	Timeout;
 	 int	nTimeouts;
 } tInterface;
@@ -118,6 +119,7 @@ int main(int argc, char *argv[])
 			return -1;
 		}
 		i->State = STATE_PREINIT;
+		i->StartTime = _SysTimestamp();
 		
 		// Send request
 		Send_DHCPDISCOVER(i);
@@ -299,6 +301,7 @@ void Send_DHCPDISCOVER(tInterface *Iface)
 	msg->siaddr = htonl(0);	// siaddr - Zero? maybe -1
 	msg->giaddr = htonl(0);	// giaddr - Zero?
 	memcpy(msg->chaddr, Iface->HWAddr, 6);
+	memset(msg->chaddr+6, 0, sizeof(msg->chaddr)-6);
 
 	memset(msg->sname, 0, sizeof(msg->sname));	// Nuke the rest
 	memset(msg->file, 0, sizeof(msg->file));	// Nuke the rest
@@ -330,14 +333,19 @@ void Send_DHCPREQUEST(tInterface *Iface, void *OfferPacket, int TypeOffset)
 	 int	i;
 	msg = (void*) ((char*)OfferPacket) + 8;
 
+	if( msg->xid != htonl(Iface->TransactionID) ) {
+		_SysDebug("DHCPREQUEST: Transaction ID mismatch");
+		return ;
+	}
+
 	// Reuses old data :)
-	msg->op    = 1;
-	msg->htype = 1;
-	msg->hlen  = 6;
-	msg->hops  = 0;
-	msg->xid   = msg->xid;
-	msg->secs  = htons(0);	// TODO: Maintain times
-	msg->flags = htons(0);
+	msg->op    = htonb(1);
+	msg->htype = htonb(1);
+	msg->hlen  = htonb(6);
+	msg->hops  = htonb(0);
+	msg->xid   = htonl(Iface->TransactionID);
+	msg->secs  = htons( (_SysTimestamp()-Iface->StartTime+499)/1000 );	// TODO: Maintain times
+	msg->flags = htons(0x0000);
 	memcpy(msg->chaddr, Iface->HWAddr, 6);
 	memset(msg->sname, 0, sizeof(msg->sname));	// Nuke the rest
 	memset(msg->file, 0, sizeof(msg->file));	// Nuke the rest
@@ -380,7 +388,7 @@ int Handle_Packet(tInterface *Iface)
 	void	*router = NULL;
 	void	*subnet_mask = NULL;
 	
-	_SysDebug("Doing read on %i", Iface->SocketFD);
+	_SysDebug("Doing read on %s %i", Iface->Adapter, Iface->SocketFD);
 	len = _SysRead(Iface->SocketFD, data, sizeof(data));
 	_SysDebug("len = %i", len);
 
@@ -472,11 +480,15 @@ int Handle_Timeout(tInterface *Iface)
 	switch(Iface->State)
 	{
 	case STATE_DISCOVER_SENT:
+		fprintf(stderr, "DHCPDISCOVER timeout on %s, trying again\n", Iface->Adapter);
 		Send_DHCPDISCOVER(Iface);
 		break;
+	case STATE_REQUEST_SENT:
+		fprintf(stderr, "DHCPREQUEST timeout on %s\n", Iface->Adapter);
+		return 1;
 	default:
-		_SysDebug("Timeout with state = %i", Iface->State);
-		break;
+		_SysDebug("Timeout with state = %i (%s)", Iface->State, Iface->Adapter);
+		return 1;
 	}
 	return 0;
 }
@@ -494,7 +506,9 @@ void Update_State(tInterface *Iface, int newState)
 		// TODO: Exponential backoff
 		Iface->Timeout = _SysTimestamp() + 3000;
 		Iface->nTimeouts ++;
-		_SysDebug("State %i repeated, timeout is 3000ms now", newState);
+		_SysDebug("%s: State %i repeated, timeout is 3000ms now",
+			Iface->Adapter,
+			newState);
 	}
 }