diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2008-11-20 03:57:01 -0500 |
---|---|---|
committer | Patrick McHardy <kaber@trash.net> | 2008-11-20 03:57:01 -0500 |
commit | b0ceb560a4119f187dc50da655be389cb54ae4f9 (patch) | |
tree | fc348316fb6a8a644708a4a03c34b85701c0d5bd /net | |
parent | e17b666a468285409ab9f6caff9df16936d27d71 (diff) |
netfilter: xt_recent: don't save proc dirs
Not needed, since creation and removal are done by name.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/netfilter/xt_recent.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/net/netfilter/xt_recent.c b/net/netfilter/xt_recent.c index 3c3dd22b1d06..fe80b614a400 100644 --- a/net/netfilter/xt_recent.c +++ b/net/netfilter/xt_recent.c | |||
@@ -72,9 +72,6 @@ struct recent_entry { | |||
72 | struct recent_table { | 72 | struct recent_table { |
73 | struct list_head list; | 73 | struct list_head list; |
74 | char name[XT_RECENT_NAME_LEN]; | 74 | char name[XT_RECENT_NAME_LEN]; |
75 | #ifdef CONFIG_PROC_FS | ||
76 | struct proc_dir_entry *proc_old, *proc; | ||
77 | #endif | ||
78 | unsigned int refcnt; | 75 | unsigned int refcnt; |
79 | unsigned int entries; | 76 | unsigned int entries; |
80 | struct list_head lru_list; | 77 | struct list_head lru_list; |
@@ -284,6 +281,9 @@ static bool recent_mt_check(const struct xt_mtchk_param *par) | |||
284 | { | 281 | { |
285 | const struct xt_recent_mtinfo *info = par->matchinfo; | 282 | const struct xt_recent_mtinfo *info = par->matchinfo; |
286 | struct recent_table *t; | 283 | struct recent_table *t; |
284 | #ifdef CONFIG_PROC_FS | ||
285 | struct proc_dir_entry *pde; | ||
286 | #endif | ||
287 | unsigned i; | 287 | unsigned i; |
288 | bool ret = false; | 288 | bool ret = false; |
289 | 289 | ||
@@ -318,25 +318,25 @@ static bool recent_mt_check(const struct xt_mtchk_param *par) | |||
318 | for (i = 0; i < ip_list_hash_size; i++) | 318 | for (i = 0; i < ip_list_hash_size; i++) |
319 | INIT_LIST_HEAD(&t->iphash[i]); | 319 | INIT_LIST_HEAD(&t->iphash[i]); |
320 | #ifdef CONFIG_PROC_FS | 320 | #ifdef CONFIG_PROC_FS |
321 | t->proc = proc_create_data(t->name, ip_list_perms, recent_proc_dir, | 321 | pde = proc_create_data(t->name, ip_list_perms, recent_proc_dir, |
322 | &recent_mt_fops, t); | 322 | &recent_mt_fops, t); |
323 | if (t->proc == NULL) { | 323 | if (pde == NULL) { |
324 | kfree(t); | 324 | kfree(t); |
325 | goto out; | 325 | goto out; |
326 | } | 326 | } |
327 | pde->uid = ip_list_uid; | ||
328 | pde->gid = ip_list_gid; | ||
327 | #ifdef CONFIG_NETFILTER_XT_MATCH_RECENT_PROC_COMPAT | 329 | #ifdef CONFIG_NETFILTER_XT_MATCH_RECENT_PROC_COMPAT |
328 | t->proc_old = proc_create_data(t->name, ip_list_perms, proc_old_dir, | 330 | pde = proc_create_data(t->name, ip_list_perms, proc_old_dir, |
329 | &recent_old_fops, t); | 331 | &recent_old_fops, t); |
330 | if (t->proc_old == NULL) { | 332 | if (pde == NULL) { |
331 | remove_proc_entry(t->name, proc_old_dir); | 333 | remove_proc_entry(t->name, proc_old_dir); |
332 | kfree(t); | 334 | kfree(t); |
333 | goto out; | 335 | goto out; |
334 | } | 336 | } |
335 | t->proc_old->uid = ip_list_uid; | 337 | pde->uid = ip_list_uid; |
336 | t->proc_old->gid = ip_list_gid; | 338 | pde->gid = ip_list_gid; |
337 | #endif | 339 | #endif |
338 | t->proc->uid = ip_list_uid; | ||
339 | t->proc->gid = ip_list_gid; | ||
340 | #endif | 340 | #endif |
341 | spin_lock_bh(&recent_lock); | 341 | spin_lock_bh(&recent_lock); |
342 | list_add_tail(&t->list, &tables); | 342 | list_add_tail(&t->list, &tables); |