From: Tom Rix Date: Sat, 15 May 2021 18:09:41 +0000 (-0700) Subject: f2fs: return success if there is no work to do X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=4f55dc2a988b304d3595887f1161151d1c3b1f33;p=linux.git 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 Signed-off-by: Jaegeuk Kim --- diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 895dfcf45fe19..a6be76289452f 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -3202,7 +3202,7 @@ int f2fs_precache_extents(struct inode *inode) map.m_lblk = m_next_extent; } - return err; + return 0; } static int f2fs_ioc_precache_extents(struct file *filp, unsigned long arg)