lkml.org 
[lkml]   [2019]   [Sep]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: build_path_from_dentry_optional_prefix() may schedule from invalid context
ср, 28 авг. 2019 г. в 22:02, Sergey Senozhatsky
<sergey.senozhatsky.work@gmail.com>:
>
> Hello,
>
> Looking at commit "cifs: create a helper to find a writeable handle
> by path name":
>
> ->open_file_lock scope is atomic context, while build_path_from_dentry()
> can schedule - kmalloc(GFP_KERNEL)
>
> spin_lock(&tcon->open_file_lock);
> list_for_each(tmp, &tcon->openFileList) {
> cfile = list_entry(tmp, struct cifsFileInfo,
> tlist);
> full_path = build_path_from_dentry(cfile->dentry);
> if (full_path == NULL) {
> spin_unlock(&tcon->open_file_lock);
> return -ENOMEM;
> }
> if (strcmp(full_path, name)) {
> kfree(full_path);
> continue;
> }
> kfree(full_path);
>
> cinode = CIFS_I(d_inode(cfile->dentry));
> spin_unlock(&tcon->open_file_lock);
> return cifs_get_writable_file(cinode, 0, ret_file);
> }
>
> spin_unlock(&tcon->open_file_lock);
>
> Additionally, kfree() can (and should) be done outside of
> ->open_file_lock scope.
>
> -ss

Good catch. I think we should have another version of
build_path_from_dentry() which takes pre-allocated (probably on stack)
full_path as an argument. This would allow us to avoid allocations
under the spin lock.
--
Best regards,
Pavel Shilovsky

\
 
 \ /
  Last update: 2019-09-20 02:12    [W:0.062 / U:1.536 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site