diff options
| author | Steve French <sfrench@us.ibm.com> | 2007-09-15 03:01:17 +0000 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2007-09-15 03:01:17 +0000 |
| commit | 88f370a688e765de9755a343702ca04e6817e5f5 (patch) | |
| tree | 82ad179c2310cf60c8aa83c50e221f363b51d8fe /fs/cifs/connect.c | |
| parent | 638b250766272fcaaa0f7ed2776f58f4ac701914 (diff) | |
| download | op-kernel-dev-88f370a688e765de9755a343702ca04e6817e5f5.zip op-kernel-dev-88f370a688e765de9755a343702ca04e6817e5f5.tar.gz | |
[CIFS] Fix potential NULL pointer usage if kzalloc fails
Potential problem was noticed by Cyrill Gorcunov
CC: Cyrill Gorcunov <gorcunov@gmail.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/connect.c')
| -rw-r--r-- | fs/cifs/connect.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index a83684d..5f2ec19 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -3386,9 +3386,11 @@ CIFSTCon(unsigned int xid, struct cifsSesInfo *ses, kfree(tcon->nativeFileSystem); tcon->nativeFileSystem = kzalloc(length + 2, GFP_KERNEL); - cifs_strfromUCS_le(tcon->nativeFileSystem, - (__le16 *) bcc_ptr, - length, nls_codepage); + if (tcon->nativeFileSystem) + cifs_strfromUCS_le( + tcon->nativeFileSystem, + (__le16 *) bcc_ptr, + length, nls_codepage); bcc_ptr += 2 * length; bcc_ptr[0] = 0; /* null terminate the string */ bcc_ptr[1] = 0; @@ -3403,8 +3405,9 @@ CIFSTCon(unsigned int xid, struct cifsSesInfo *ses, kfree(tcon->nativeFileSystem); tcon->nativeFileSystem = kzalloc(length + 1, GFP_KERNEL); - strncpy(tcon->nativeFileSystem, bcc_ptr, - length); + if (tcon->nativeFileSystem) + strncpy(tcon->nativeFileSystem, bcc_ptr, + length); } /* else do not bother copying these information fields*/ } |
