Skip to content

Commit 26b9398

Browse files
committed
smb: client: fix memory leak during error handling for POSIX mkdir
JIRA: https://issues.redhat.com/browse/RHEL-96872 commit 1fe4a44 Author: Jethro Donaldson <[email protected]> Date: Thu May 15 01:23:23 2025 +1200 smb: client: fix memory leak during error handling for POSIX mkdir The response buffer for the CREATE request handled by smb311_posix_mkdir() is leaked on the error path (goto err_free_rsp_buf) because the structure pointer *rsp passed to free_rsp_buf() is not assigned until *after* the error condition is checked. As *rsp is initialised to NULL, free_rsp_buf() becomes a no-op and the leak is instead reported by __kmem_cache_shutdown() upon subsequent rmmod of cifs.ko if (and only if) the error path has been hit. Pass rsp_iov.iov_base to free_rsp_buf() instead, similar to the code in other functions in smb2pdu.c for which *rsp is assigned late. Cc: [email protected] Signed-off-by: Jethro Donaldson <[email protected]> Signed-off-by: Steve French <[email protected]> Signed-off-by: Paulo Alcantara <[email protected]>
1 parent eb50bc4 commit 26b9398

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

fs/smb/client/smb2pdu.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2975,7 +2975,7 @@ int smb311_posix_mkdir(const unsigned int xid, struct inode *inode,
29752975
/* Eventually save off posix specific response info and timestamps */
29762976

29772977
err_free_rsp_buf:
2978-
free_rsp_buf(resp_buftype, rsp);
2978+
free_rsp_buf(resp_buftype, rsp_iov.iov_base);
29792979
kfree(pc_buf);
29802980
err_free_req:
29812981
cifs_small_buf_release(req);

0 commit comments

Comments
 (0)