On Mon, Jun 22, 2015 at 08:36:32AM -0500, Eric Sandeen wrote:
> The earlier commit:
>
> ec693e1 metadump: zero out clean log
>
> ignored the "obfuscate" state, but there's no reason to
> zero out the log if we're not obfuscating; all the other
> metadata is in the clear, so we may as well keep it around
> in the log as well.
>
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> ---
Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
>
> (this is for the progs-misc-fixes-1 branch)
>
>
> diff --git a/db/metadump.c b/db/metadump.c
> index eb5e9da..bdc48a0 100644
> --- a/db/metadump.c
> +++ b/db/metadump.c
> @@ -2184,6 +2184,10 @@ copy_log(void)
> return !stop_on_read_error;
> }
>
> + /* If not obfuscating, just copy the log as it is */
> + if (!obfuscate)
> + goto done;
> +
> dirty = xlog_is_dirty(mp, &x, 0);
>
> switch (dirty) {
> @@ -2196,18 +2200,17 @@ copy_log(void)
> break;
> case 1:
> /* keep the dirty log */
> - if (obfuscate)
> - print_warning(
> + print_warning(
> _("Filesystem log is dirty; image will contain unobfuscated metadata in
> log."));
> break;
> case -1:
> /* log detection error */
> - if (obfuscate)
> - print_warning(
> + print_warning(
> _("Could not discern log; image will contain unobfuscated metadata in
> log."));
> break;
> }
>
> +done:
> return !write_buf(iocur_top);
> }
>
>
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs
|