diff options
| author | Darrick J. Wong <darrick.wong@oracle.com> | 2017-11-06 23:01:48 +0300 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2017-11-09 20:10:45 +0300 |
| commit | 72f76f73642fa8528cab098b5f66abb299f1a018 (patch) | |
| tree | dfc92172edf7700f2b7d889933d32aee326c1956 /fs/xfs/scrub | |
| parent | 0a1e1567b386b96c710d385181330c13ca03ffe3 (diff) | |
| download | linux-72f76f73642fa8528cab098b5f66abb299f1a018.tar.xz | |
xfs: fix uninitialized return values in scrub code
Fix smatch complaints about uninitialized return codes.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Diffstat (limited to 'fs/xfs/scrub')
| -rw-r--r-- | fs/xfs/scrub/dir.c | 2 | ||||
| -rw-r--r-- | fs/xfs/scrub/parent.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/fs/xfs/scrub/dir.c b/fs/xfs/scrub/dir.c index c8ca3fd67445..69e1efdd4019 100644 --- a/fs/xfs/scrub/dir.c +++ b/fs/xfs/scrub/dir.c @@ -748,7 +748,7 @@ xfs_scrub_directory( }; size_t bufsize; loff_t oldpos; - int error; + int error = 0; if (!S_ISDIR(VFS_I(sc->ip)->i_mode)) return -ENOENT; diff --git a/fs/xfs/scrub/parent.c b/fs/xfs/scrub/parent.c index cc2b8f665416..63a25334fc83 100644 --- a/fs/xfs/scrub/parent.c +++ b/fs/xfs/scrub/parent.c @@ -143,7 +143,7 @@ xfs_scrub_parent_validate( struct xfs_inode *dp = NULL; xfs_nlink_t expected_nlink; xfs_nlink_t nlink; - int error; + int error = 0; *try_again = false; @@ -258,7 +258,7 @@ xfs_scrub_parent( xfs_ino_t dnum; bool try_again; int tries = 0; - int error; + int error = 0; /* * If we're a directory, check that the '..' link points up to |