xfs_repair: add printf format checking and fix the fallout

Alex Elder aelder at sgi.com
Mon Sep 19 17:39:36 CDT 2011


On Sun, 2011-08-14 at 16:12 -0400, Christoph Hellwig wrote:
> Add the gcc printf like attribute to the xfs_repair-internal logging helpers,
> and fix the massive fallout.  A large part of it is dealing with the correct
> format for fixed size 64-bit types, but there were a lot of real bug in there,
> including some that lead to crashed when repairing certain corrupted
> filesystems on ARM based systems.
> 
> Signed-off-by: Christoph Hellwig <hch at lst.de>
> Reported-by: Anisse Astier <anisse at astier.eu>

OK, I compiled in these environments:
  x86_64: clean build
  ia64:   some new warnings, eliminated none of the old ones
  i686:   still a few (non-printf) warnings, but none new

If it weren't for the new warnings I would just say
let's commit this and I can commit fixes for the remaining
warnings separately.  But instead I'll send you a patch
privately for you to merge with what you've got.

					-Alex





More information about the xfs mailing list