diff options
| author | Linus Torvalds | 2020-03-12 15:51:26 -0700 |
|---|---|---|
| committer | Linus Torvalds | 2020-03-12 15:51:26 -0700 |
| commit | 807f030b44ccbb26a346df6f6438628315d9ad98 (patch) | |
| tree | 5c5bf61456bf8f6649fab5e60e8ef0e8985a713a /Documentation/filesystems | |
| parent | 3cc6e2c599cdca573a8f347aea5da4c855ff5a78 (diff) | |
| parent | d9a9f4849fe0c9d560851ab22a85a666cddfdd24 (diff) | |
Merge branch 'fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs
Pull vfs fixes from Al Viro:
"A couple of fixes for old crap in ->atomic_open() instances"
* 'fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs:
cifs_atomic_open(): fix double-put on late allocation failure
gfs2_atomic_open(): fix O_EXCL|O_CREAT handling on cold dcache
Diffstat (limited to 'Documentation/filesystems')
| -rw-r--r-- | Documentation/filesystems/porting.rst | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/Documentation/filesystems/porting.rst b/Documentation/filesystems/porting.rst index f18506083ced..26c093969573 100644 --- a/Documentation/filesystems/porting.rst +++ b/Documentation/filesystems/porting.rst @@ -850,3 +850,11 @@ business doing so. d_alloc_pseudo() is internal-only; uses outside of alloc_file_pseudo() are very suspect (and won't work in modules). Such uses are very likely to be misspelled d_alloc_anon(). + +--- + +**mandatory** + +[should've been added in 2016] stale comment in finish_open() nonwithstanding, +failure exits in ->atomic_open() instances should *NOT* fput() the file, +no matter what. Everything is handled by the caller. |