On Wed, Nov 22, 2023 at 03:07:22 PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > Coverity complained about the "is fd a file?" flags being uninitialized. > Clean this up. > Looks good to me. Reviewed-by: Chandan Babu R <chandanbabu@xxxxxxxxxx> > Coverity-id: 1554270 > Signed-off-by: Darrick J. Wong <djwong@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 2de177c6e3f..5dfc423493e 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); -- Chandan
