From: Darrick J. Wong <djwong@xxxxxxxxxx> Coverity complained about the "is fd a file?" flags being uninitialized. Clean this up. Coverity-id: 1554270 Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> Reviewed-by: Christoph Hellwig <hch@xxxxxx> Reviewed-by: Chandan Babu R <chandanbabu@xxxxxxxxxx> --- mdrestore/xfs_mdrestore.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/mdrestore/xfs_mdrestore.c b/mdrestore/xfs_mdrestore.c index 2de177c6..5dfc4234 100644 --- a/mdrestore/xfs_mdrestore.c +++ b/mdrestore/xfs_mdrestore.c @@ -472,11 +472,11 @@ main( union mdrestore_headers headers; FILE *src_f; char *logdev = NULL; - int data_dev_fd; - int log_dev_fd; + int data_dev_fd = -1; + int log_dev_fd = -1; int c; - bool is_data_dev_file; - bool is_log_dev_file; + bool is_data_dev_file = false; + bool is_log_dev_file = false; mdrestore.show_progress = false; mdrestore.show_info = false; @@ -561,7 +561,6 @@ main( /* check and open data device */ data_dev_fd = open_device(argv[optind], &is_data_dev_file); - log_dev_fd = -1; if (mdrestore.external_log) /* check and open log device */ log_dev_fd = open_device(logdev, &is_log_dev_file);