diff options
-rw-r--r-- | net/netfilter/nf_conntrack_amanda.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/net/netfilter/nf_conntrack_amanda.c b/net/netfilter/nf_conntrack_amanda.c index b8869eab7650..0568f2e86b59 100644 --- a/net/netfilter/nf_conntrack_amanda.c +++ b/net/netfilter/nf_conntrack_amanda.c | |||
@@ -208,13 +208,14 @@ static int __init nf_conntrack_amanda_init(void) | |||
208 | { | 208 | { |
209 | int ret, i; | 209 | int ret, i; |
210 | 210 | ||
211 | ret = -ENOMEM; | ||
212 | for (i = 0; i < ARRAY_SIZE(search); i++) { | 211 | for (i = 0; i < ARRAY_SIZE(search); i++) { |
213 | search[i].ts = textsearch_prepare(ts_algo, search[i].string, | 212 | search[i].ts = textsearch_prepare(ts_algo, search[i].string, |
214 | search[i].len, | 213 | search[i].len, |
215 | GFP_KERNEL, TS_AUTOLOAD); | 214 | GFP_KERNEL, TS_AUTOLOAD); |
216 | if (search[i].ts == NULL) | 215 | if (IS_ERR(search[i].ts)) { |
216 | ret = PTR_ERR(search[i].ts); | ||
217 | goto err1; | 217 | goto err1; |
218 | } | ||
218 | } | 219 | } |
219 | ret = nf_conntrack_helper_register(&amanda_helper[0]); | 220 | ret = nf_conntrack_helper_register(&amanda_helper[0]); |
220 | if (ret < 0) | 221 | if (ret < 0) |
@@ -227,10 +228,9 @@ static int __init nf_conntrack_amanda_init(void) | |||
227 | err2: | 228 | err2: |
228 | nf_conntrack_helper_unregister(&amanda_helper[0]); | 229 | nf_conntrack_helper_unregister(&amanda_helper[0]); |
229 | err1: | 230 | err1: |
230 | for (; i >= 0; i--) { | 231 | while (--i >= 0) |
231 | if (search[i].ts) | 232 | textsearch_destroy(search[i].ts); |
232 | textsearch_destroy(search[i].ts); | 233 | |
233 | } | ||
234 | return ret; | 234 | return ret; |
235 | } | 235 | } |
236 | 236 | ||