From: Alex Elder Date: Wed, 28 Sep 2011 11:15:37 +0000 (+0000) Subject: xfsprogs: xfs_quota: don't print invalid quota file inode number X-Git-Tag: v3.1.6~16 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=06065b12cc21b1c891c247b237d98fea591f01e9;p=thirdparty%2Fxfsprogs-dev.git xfsprogs: xfs_quota: don't print invalid quota file inode number When the state of quota files is dumped, xfs_quota blindly shows whatever inode number is returned by the kernel. If one of the quota types is not enabled or enforced, the inode number provided is an invalid value ((__u64) -1). Rather than print a meaningless large integer, print "N/A" in its place to make interpreting the result it a little more obvious. Signed-off-by: Alex Elder Reviewed-by: Christoph Hellwig --- diff --git a/quota/state.c b/quota/state.c index 3862e677f..42bffc017 100644 --- a/quota/state.c +++ b/quota/state.c @@ -122,10 +122,13 @@ state_qfilestat( mount->fs_dir, mount->fs_name); fprintf(fp, _(" Accounting: %s\n"), accounting ? _("ON") : _("OFF")); fprintf(fp, _(" Enforcement: %s\n"), enforcing ? _("ON") : _("OFF")); - fprintf(fp, _(" Inode: #%llu (%llu blocks, %lu extents)\n"), - (unsigned long long)qfs->qfs_ino, - (unsigned long long)qfs->qfs_nblks, - (unsigned long)qfs->qfs_nextents); + if (qfs->qfs_ino != (__u64) -1) + fprintf(fp, _(" Inode: #%llu (%llu blocks, %lu extents)\n"), + (unsigned long long)qfs->qfs_ino, + (unsigned long long)qfs->qfs_nblks, + (unsigned long)qfs->qfs_nextents); + else + fprintf(fp, _(" Inode: N/A\n")); } static void