diff options
| author | Adrian Bunk <bunk@stusta.de> | 2005-11-07 00:59:23 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-07 07:53:36 -0800 |
| commit | ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b (patch) | |
| tree | c7a412727aa338fdfbf9f8ae6c46205852eeb7f0 | |
| parent | 6f08b72cdd4b99a190c4e1fcddd63b401e5b572d (diff) | |
| download | linux-ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b.tar.bz2 | |
[PATCH] fs/smbfs/request.c: turn NULL dereference into BUG()
In a case documented as
We should never be called with any of these states
BUG() in a case that would later result in a NULL pointer dereference.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | fs/smbfs/request.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/smbfs/request.c b/fs/smbfs/request.c index 2d85dd7415bb..a0f296d9928a 100644 --- a/fs/smbfs/request.c +++ b/fs/smbfs/request.c @@ -786,8 +786,7 @@ int smb_request_recv(struct smb_sb_info *server) /* We should never be called with any of these states */ case SMB_RECV_END: case SMB_RECV_REQUEST: - server->rstate = SMB_RECV_END; - break; + BUG(); } if (result < 0) { |