summary refs log tree commit diff
path: root/src/funcs/newgrp
diff options
context:
space:
mode:
authorHerbert Xu <herbert@gondor.apana.org.au>2010-09-08 20:07:26 +0800
committerHerbert Xu <herbert@gondor.apana.org.au>2010-09-08 20:07:26 +0800
commitf42e443bb511ed3224f09b4fcf0772438ebdbbfa (patch)
treefc2523d1f399d4d4a8a6432e3855de649ddf7346 /src/funcs/newgrp
parent[JOBS] Debug compile fix (diff)
downloaddash-f42e443bb511ed3224f09b4fcf0772438ebdbbfa.tar.gz
dash-f42e443bb511ed3224f09b4fcf0772438ebdbbfa.zip
[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 <herbert@gondor.apana.org.au>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions