diff options
author | Tom Rix <trix@redhat.com> | 2021-05-15 20:09:41 +0200 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2021-05-26 16:00:35 +0200 |
commit | 4f55dc2a988b304d3595887f1161151d1c3b1f33 (patch) | |
tree | 467887f973b525c913b85f6bd986bbb5a6306c82 /fs/f2fs/inode.c | |
parent | f2fs: compress: clean up parameter of __f2fs_cluster_blocks() (diff) | |
download | linux-4f55dc2a988b304d3595887f1161151d1c3b1f33.tar.xz linux-4f55dc2a988b304d3595887f1161151d1c3b1f33.zip |
f2fs: return success if there is no work to do
Static analysis reports this problem
file.c:3206:2: warning: Undefined or garbage value returned to caller
return err;
^~~~~~~~~~
err is only set if there is some work to do. Because the loop returns
immediately on an error, if all the work was done, a 0 would be returned.
Instead of checking the unlikely case that there was no work to do,
change the return of err to 0.
Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/inode.c')
0 files changed, 0 insertions, 0 deletions