diff options
author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2022-05-08 14:40:32 -0400 |
---|---|---|
committer | Matthew Wilcox (Oracle) <willy@infradead.org> | 2022-05-08 14:40:32 -0400 |
commit | 652118b8c979e7f21358016b5568aac4b25c64d9 (patch) | |
tree | dc939c9a086152b319a016075fa94244e36a40bc /fs/hfs | |
parent | 44ab23b9b3c7a73c738b37a8f5f84adb7d66c267 (diff) | |
download | linux-stable-652118b8c979e7f21358016b5568aac4b25c64d9.tar.gz linux-stable-652118b8c979e7f21358016b5568aac4b25c64d9.tar.bz2 linux-stable-652118b8c979e7f21358016b5568aac4b25c64d9.zip |
ntfs3: Remove fsdata parameter from ntfs_extend_initialized_size()
After the last patch, Smatch reports:
fs/ntfs3/file.c:168 ntfs_extend_initialized_size()
error: uninitialized symbol 'fsdata'.
fsdata is indeed unused. This is not new, but Smatch couldn't see it
before because calls through pagecache_write_begin()/pagecache_write_end()
could theoretically call any implemention of ->write_begin/write_end,
some of which do use fsdata. Now that the calls are direct, Smatch can
see they're never used.
Fix this by simply passing NULL. While ntfs3 does pass this parameter
on to generic functions, those generic functions also never dereference
the fsdata parameter, so it's unnecessary to pass the address of a real
pointer.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Diffstat (limited to 'fs/hfs')
0 files changed, 0 insertions, 0 deletions