diff options
author | Theodore Ts'o <tytso@mit.edu> | 2022-08-11 23:01:42 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2022-08-11 23:01:42 -0400 |
commit | 45dc484a25f234722f6b0fe4f8fc12080429a1dd (patch) | |
tree | cf21eff82053e312982190337e633da0058866c0 | |
parent | 1bd16e790308f92e89a5dfbd40ab9e164fe88aa9 (diff) | |
download | e2fsprogs-45dc484a25f234722f6b0fe4f8fc12080429a1dd.tar.gz |
e2fsck: streamline problem latch handling
No functional changes, but streamline the logic, and avoid a coverity
warning.
Addresses-Coverity-Bug: 1507763
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r-- | e2fsck/problem.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/e2fsck/problem.c b/e2fsck/problem.c index 95f0ace88..e2572f595 100644 --- a/e2fsck/problem.c +++ b/e2fsck/problem.c @@ -2494,8 +2494,7 @@ int fix_problem(e2fsck_t ctx, problem_t code, struct problem_context *pctx) if ((ctx->options & E2F_OPT_PREEN) && (ptr->flags & PR_PREEN_OK)) suppress++; - if ((ptr->flags & PR_LATCH_MASK) && - (ldesc->flags & (PRL_YES | PRL_NO))) + if (ldesc && (ldesc->flags & (PRL_YES | PRL_NO))) suppress++; if (ptr->count == ptr->max_count + 1) { if (ctx->problem_logf) @@ -2540,8 +2539,7 @@ int fix_problem(e2fsck_t ctx, problem_t code, struct problem_context *pctx) answer = def_yn; if (!(ptr->flags & PR_PREEN_NOMSG)) print_answer = 1; - } else if ((ptr->flags & PR_LATCH_MASK) && - (ldesc->flags & (PRL_YES | PRL_NO))) { + } else if (ldesc && (ldesc->flags & (PRL_YES | PRL_NO))) { print_answer = 1; if (ldesc->flags & PRL_YES) answer = 1; |