From 06065b12cc21b1c891c247b237d98fea591f01e9 Mon Sep 17 00:00:00 2001 From: Alex Elder Date: Wed, 28 Sep 2011 11:15:37 +0000 Subject: [PATCH] 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 --- quota/state.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) 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 -- 2.47.2