[PATCH 2/2] xfs_logprint: don't advance op counter in xlog_print_trans_icreate

Eric Sandeen sandeen at sandeen.net
Wed Feb 26 19:15:52 CST 2014


xlog_print_trans_icreate is advancing the op counter
"(*i)++" incorrectly; it only contains one region, and
the loop which called it will properly advance the op
once we're done.

Found-by: Dave Chinner <dchinner at redhat.com>
Signed-off-by: Eric Sandeen <sandeen at redhat.com>
---

diff --git a/logprint/log_misc.c b/logprint/log_misc.c
index 7070fa3..52f1e85 100644
--- a/logprint/log_misc.c
+++ b/logprint/log_misc.c
@@ -810,7 +810,6 @@ xlog_print_trans_icreate(
 
 	memmove(&icl_buf, *ptr, MIN(sizeof(struct xfs_icreate_log), len));
 	icl = &icl_buf;
-	(*i)++;
 	*ptr += len;
 
 	/* handle complete header only */




More information about the xfs mailing list