diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2005-03-31 04:29:09 -0800 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2005-03-31 04:29:09 -0800 |
| commit | 4492174a34f3118342587bedef10117817179375 (patch) | |
| tree | bd8b986ccfdd79f77ba8abd03bf14e3f326e9bd5 /net/unix/af_unix.c | |
| parent | 3b4155300fc0cdc66ac07846899dc84dc3a2dfb4 (diff) | |
[NETLINK]: More complete fix for race.
Unfortunately my patch only closed half the race. There is still
a chunk of code between netlink_dump_start and netlink_dump that runs
outside the cb lock which isn't protected by an sk reference.
Here is a better patch which protects the entire netlink_dump function
with a sk reference.
The other call to netlink_dump by recvmsg is safe as the open file
descriptor already holds a reference. As such the final sock_put
in netlink_dump can be turned into a __sock_put since there is at
least one reference held by the caller.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions
