Skip to content

Commit

Permalink
[NETFILTER]: nfctnetlink: Don't allow to change helper
Browse files Browse the repository at this point in the history
There is no realistic situation to change helper (Who wants IRC helper to
track FTP traffic ?). Moreover, if we want to do that, we need to fix race
issue by nfctnetlink and running helper. That will add overhead to packet
processing. It wouldn't pay. So this rejects the request to change
helper. The requests to add or remove helper are accepted as ever.

Signed-off-by: Yasuyuki Kozakai <[email protected]>
Signed-off-by: Patrick McHardy <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
yasuyuki5 authored and davem330 committed Jun 22, 2007
1 parent d258131 commit e2d8e31
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions net/netfilter/nf_conntrack_netlink.c
Original file line number Diff line number Diff line change
Expand Up @@ -869,8 +869,7 @@ ctnetlink_change_helper(struct nf_conn *ct, struct nfattr *cda[])
return 0;

if (help->helper)
/* we had a helper before ... */
nf_ct_remove_expectations(ct);
return -EBUSY;

/* need to zero data of old helper */
memset(&help->help, 0, sizeof(help->help));
Expand Down

0 comments on commit e2d8e31

Please sign in to comment.