diff options
| author | Adrian Bunk <bunk@stusta.de> | 2005-11-07 03:59:23 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-07 10:53:36 -0500 |
| commit | ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b (patch) | |
| tree | c7a412727aa338fdfbf9f8ae6c46205852eeb7f0 | |
| parent | 6f08b72cdd4b99a190c4e1fcddd63b401e5b572d (diff) | |
[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) | |||
| 786 | /* We should never be called with any of these states */ | 786 | /* We should never be called with any of these states */ |
| 787 | case SMB_RECV_END: | 787 | case SMB_RECV_END: |
| 788 | case SMB_RECV_REQUEST: | 788 | case SMB_RECV_REQUEST: |
| 789 | server->rstate = SMB_RECV_END; | 789 | BUG(); |
| 790 | break; | ||
| 791 | } | 790 | } |
| 792 | 791 | ||
| 793 | if (result < 0) { | 792 | if (result < 0) { |