diff options
| author | Adrian Bunk <bunk@stusta.de> | 2005-11-07 09:59:23 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-07 16:53:36 +0100 |
| commit | ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b (patch) | |
| tree | c7a412727aa338fdfbf9f8ae6c46205852eeb7f0 /fs/smbfs | |
| parent | [PATCH] serial moxa: fix wrong BUG (diff) | |
| download | linux-ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b.tar.xz linux-ccb6e363a68144cdfdaa6d964d63d620c8ac9a9b.zip | |
[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>
Diffstat (limited to 'fs/smbfs')
| -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) { |