From f42e443bb511ed3224f09b4fcf0772438ebdbbfa Mon Sep 17 00:00:00 2001 From: Herbert Xu Date: Wed, 8 Sep 2010 20:07:26 +0800 Subject: [EXPAND] Fix ifsfirst/ifslastp leak As it stands expandarg may return with a non-NULL ifslastp which then confuses any subsequent ifsbreakup user that doesn't clear it directly. What's worse, if we get interrupted before we hit ifsfree in expandarg we will leak memory. This patch fixes this by always calling ifsfree in expandarg thus ensuring that ifslastp is always NULL on the normal path. It also adds an ifsfree call to the RESET path to ensure that memory isn't leaked. Signed-off-by: Herbert Xu --- ChangeLog | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'ChangeLog') diff --git a/ChangeLog b/ChangeLog index 3c26149..a51975c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2010-09-08 Herbert Xu + + * Fix ifsfirst/ifslastp leak. + 2010-09-08 maximilian attems * Debug compile fix. -- cgit 1.4.1