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

[Dave Chinner]

On Wed, Feb 26, 2014 at 07:15:52PM -0600, Eric Sandeen wrote:
> 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 */

*nod*. I'm pretty sure I introduced that bug - thanks for exposing
it and fixing it :)

Reviewed-by: Dave Chinner <dchinner at redhat.com>

-- 
Dave Chinner
david at fromorbit.com