Skip to content

Commit

Permalink
---
Browse files Browse the repository at this point in the history
yaml
---
r: 214777
b: refs/heads/master
c: f91ff5b
h: refs/heads/master
i:
  214775: f5481b7
v: v3
  • Loading branch information
Eric Dumazet authored and David S. Miller committed Sep 28, 2010
1 parent 7c7b061 commit 6b7cafc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
2 changes: 1 addition & 1 deletion [refs]
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
refs/heads/master: 7fa7cb7109d07c29ab28bb877bc7049a0150dbe5
refs/heads/master: f91ff5b9ff529be8aac2039af63b2c8ea6cd6ebe
10 changes: 4 additions & 6 deletions trunk/net/core/filter.c
Original file line number Diff line number Diff line change
Expand Up @@ -638,10 +638,9 @@ int sk_attach_filter(struct sock_fprog *fprog, struct sock *sk)
return err;
}

rcu_read_lock_bh();
old_fp = rcu_dereference_bh(sk->sk_filter);
old_fp = rcu_dereference_protected(sk->sk_filter,
sock_owned_by_user(sk));
rcu_assign_pointer(sk->sk_filter, fp);
rcu_read_unlock_bh();

if (old_fp)
sk_filter_delayed_uncharge(sk, old_fp);
Expand All @@ -654,14 +653,13 @@ int sk_detach_filter(struct sock *sk)
int ret = -ENOENT;
struct sk_filter *filter;

rcu_read_lock_bh();
filter = rcu_dereference_bh(sk->sk_filter);
filter = rcu_dereference_protected(sk->sk_filter,
sock_owned_by_user(sk));
if (filter) {
rcu_assign_pointer(sk->sk_filter, NULL);
sk_filter_delayed_uncharge(sk, filter);
ret = 0;
}
rcu_read_unlock_bh();
return ret;
}
EXPORT_SYMBOL_GPL(sk_detach_filter);

0 comments on commit 6b7cafc

Please sign in to comment.