From: Darrick J. Wong <djwong@xxxxxxxxxx>
Commit de5d20ec converted xfrog_scrub_metadata to return negative error
codes directly, but forgot to fix up the str_errno calls to use
str_liberror. This doesn't result in incorrect error reporting
currently, but (a) the calls in the switch statement are inconsistent,
and (b) this will matter in future patches where we can call library
functions in between xfrog_scrub_metadata and str_liberror.
Fixes: de5d20ec ("libfrog: convert scrub.c functions to negative error codes")
Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
---
scrub/scrub.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/scrub/scrub.c b/scrub/scrub.c
index a4b7084e..07ae0673 100644
--- a/scrub/scrub.c
+++ b/scrub/scrub.c
@@ -153,7 +153,7 @@ _("Filesystem is shut down, aborting."));
case EIO:
case ENOMEM:
/* Abort on I/O errors or insufficient memory. */
- str_errno(ctx, descr_render(&dsc));
+ str_liberror(ctx, error, descr_render(&dsc));
return CHECK_ABORT;
case EDEADLOCK:
case EBUSY:
@@ -164,10 +164,10 @@ _("Filesystem is shut down, aborting."));
* and the other two should be reported via sm_flags.
*/
str_liberror(ctx, error, _("Kernel bug"));
- fallthrough;
+ return CHECK_DONE;
default:
/* Operational error. */
- str_errno(ctx, descr_render(&dsc));
+ str_liberror(ctx, error, descr_render(&dsc));
return CHECK_DONE;
}
