diff options
author | Rob Landley <rob@landley.net> | 2006-03-01 16:39:45 +0000 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2006-03-01 16:39:45 +0000 |
commit | e7c43b66d74ee9902a6732122788a7a16bcfbf18 (patch) | |
tree | ebb8583e9e1265588592614c0b4878daded44125 /modutils/insmod.c | |
parent | 93f2286e6e59dab5eed14b5912a79254031c5a62 (diff) | |
download | busybox-w32-e7c43b66d74ee9902a6732122788a7a16bcfbf18.tar.gz busybox-w32-e7c43b66d74ee9902a6732122788a7a16bcfbf18.tar.bz2 busybox-w32-e7c43b66d74ee9902a6732122788a7a16bcfbf18.zip |
Cleanup patch from Denis Vlasenko. Mostly variants of removing the if(x)
from before "if(x) free(x)".
Diffstat (limited to 'modutils/insmod.c')
-rw-r--r-- | modutils/insmod.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/modutils/insmod.c b/modutils/insmod.c index d5d58a191..d40012d70 100644 --- a/modutils/insmod.c +++ b/modutils/insmod.c | |||
@@ -4231,9 +4231,8 @@ out: | |||
4231 | #ifdef CONFIG_FEATURE_CLEAN_UP | 4231 | #ifdef CONFIG_FEATURE_CLEAN_UP |
4232 | if(fp) | 4232 | if(fp) |
4233 | fclose(fp); | 4233 | fclose(fp); |
4234 | if(tmp1) { | 4234 | free(tmp1); |
4235 | free(tmp1); | 4235 | if(!tmp1) { |
4236 | } else { | ||
4237 | free(m_name); | 4236 | free(m_name); |
4238 | } | 4237 | } |
4239 | free(m_filename); | 4238 | free(m_filename); |