From 54b62fe49314e012d6b5770c1b6a3d52fa6f38d3 Mon Sep 17 00:00:00 2001
From: cinap_lenrek <cinap_lenrek@gmx.de>
Date: Tue, 22 Jan 2013 15:26:34 +0100
Subject: arp: fix memory leaks for "flush" and "del" arp ctl messages

---
 sys/src/9/ip/arp.c | 36 +++++++++++++-----------------------
 1 file changed, 13 insertions(+), 23 deletions(-)

diff --git a/sys/src/9/ip/arp.c b/sys/src/9/ip/arp.c
index 7ea27bf23..04d1b0b99 100644
--- a/sys/src/9/ip/arp.c
+++ b/sys/src/9/ip/arp.c
@@ -405,7 +405,7 @@ arpwrite(Fs *fs, char *s, int len)
 	Route *r;
 	Arp *arp;
 	Block *bp;
-	Arpent *a, *fl, **l;
+	Arpent *a;
 	Medium *m;
 	char *f[4], buf[256];
 	uchar ip[IPaddrlen], mac[MAClen];
@@ -439,8 +439,12 @@ arpwrite(Fs *fs, char *s, int len)
 		memset(arp->hash, 0, sizeof(arp->hash));
 		/* clear all pkts on these lists (rxmt, dropf/l) */
 		arp->rxmt = nil;
-		arp->dropf = nil;
 		arp->dropl = nil;
+		while(arp->dropf != nil){
+			bp = arp->dropf->list;
+			freeblist(arp->dropf);
+			arp->dropf = bp;
+		}
 		qunlock(arp);
 	} else if(strcmp(f[0], "add") == 0){
 		switch(n){
@@ -480,31 +484,17 @@ arpwrite(Fs *fs, char *s, int len)
 			error(Ebadip);
 		qlock(arp);
 
-		l = &arp->hash[haship(ip)];
-		for(a = *l; a; a = a->hash){
-			if(memcmp(ip, a->ip, sizeof(a->ip)) == 0){
-				*l = a->hash;
+		for(a = arp->hash[haship(ip)]; a; a = a->hash)
+			if(memcmp(ip, a->ip, sizeof(a->ip)) == 0)
 				break;
-			}
-			l = &a->hash;
-		}
 	
 		if(a){
-			/* take out of re-transmit chain */
-			l = &arp->rxmt;
-			for(fl = *l; fl; fl = fl->nextrxt){
-				if(fl == a){
-					*l = a->nextrxt;
-					break;
-				}
-				l = &fl->nextrxt;
+			while(a->hold != nil){
+				bp = a->hold->list;
+				freeblist(a->hold);
+				a->hold = bp;
 			}
-
-			a->nextrxt = nil;
-			a->hash = nil;
-			a->hold = nil;
-			a->last = nil;
-			a->ifc = nil;
+			cleanarpent(arp, a);
 			memset(a->ip, 0, sizeof(a->ip));
 			memset(a->mac, 0, sizeof(a->mac));
 		}
-- 
cgit v1.2.3