diff options
| author |   J. Bruce Fields <bfields@redhat.com> | 2011-03-06 19:11:03 -0500 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2011-03-07 11:44:53 -0500 |
| commit | 32b007b4e19b50ff4d27ea8b69cd6d744cfec86b (patch) | |
| tree | ada73d6c52a6e9b37cbdc95d086485fdf338561a /fs/block_dev.c | |
| parent | NFSD: fix decode_cb_sequence4resok (diff) | |
| download | linux-dev-32b007b4e19b50ff4d27ea8b69cd6d744cfec86b.tar.xz linux-dev-32b007b4e19b50ff4d27ea8b69cd6d744cfec86b.zip | |
nfsd4: fix bad pointer on failure to find delegation
In case of a nonempty list, the return on error here is obviously bogus;
it ends up being a pointer to the list head instead of to any valid
delegation on the list.
In particular, if nfsd4_delegreturn() hits this case, and you're quite unlucky,
then renew_client may oops, and it may take an embarassingly long time to
figure out why. Facepalm.
BUG: unable to handle kernel NULL pointer dereference at 0000000000000090
IP: [<ffffffff81292965>] nfsd4_delegreturn+0x125/0x200
...
Cc: stable@kernel.org
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/block_dev.c')
0 files changed, 0 insertions, 0 deletions