linux-kernel - [PATCH] quota: don't let mark_dquot

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date: Sun,  7 Apr 2024 15:31:28 +0800
From: Chao Yu <chao@...nel.org>
To: jack@...e.com
Cc: linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Chao Yu <chao@...nel.org>
Subject: [PATCH] quota: don't let mark_dquot_dirty() fail silently

mark_dquot_dirty() will callback to specified filesystem function,
it may fail due to any reasons, however, no caller will check return
value of mark_dquot_dirty(), so, it may fail silently, let's print
one line message for such case.

Signed-off-by: Chao Yu <chao@...nel.org>
---
 fs/quota/dquot.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c
index dacbee455c03..c5df7863942a 100644
--- a/fs/quota/dquot.c
+++ b/fs/quota/dquot.c
@@ -399,21 +399,20 @@ int dquot_mark_dquot_dirty(struct dquot *dquot)
 EXPORT_SYMBOL(dquot_mark_dquot_dirty);
 
 /* Dirtify all the dquots - this can block when journalling */
-static inline int mark_all_dquot_dirty(struct dquot __rcu * const *dquots)
+static inline void mark_all_dquot_dirty(struct dquot __rcu * const *dquots)
 {
-	int ret, err, cnt;
+	int ret, cnt;
 	struct dquot *dquot;
 
-	ret = err = 0;
 	for (cnt = 0; cnt < MAXQUOTAS; cnt++) {
 		dquot = srcu_dereference(dquots[cnt], &dquot_srcu);
-		if (dquot)
-			/* Even in case of error we have to continue */
-			ret = mark_dquot_dirty(dquot);
-		if (!err)
-			err = ret;
+		if (!dquot)
+			continue;
+		ret = mark_dquot_dirty(dquot);
+		if (ret < 0)
+			quota_error(dquot->dq_sb,
+				"mark_all_dquot_dirty fails, ret: %d", ret);
 	}
-	return err;
 }
 
 static inline void dqput_all(struct dquot **dquot)
@@ -2725,6 +2724,7 @@ static int do_set_dqblk(struct dquot *dquot, struct qc_dqblk *di)
 {
 	struct mem_dqblk *dm = &dquot->dq_dqb;
 	int check_blim = 0, check_ilim = 0;
+	int ret;
 	struct mem_dqinfo *dqi = &sb_dqopt(dquot->dq_sb)->info[dquot->dq_id.type];
 
 	if (di->d_fieldmask & ~VFS_QC_MASK)
@@ -2807,7 +2807,10 @@ static int do_set_dqblk(struct dquot *dquot, struct qc_dqblk *di)
 	else
 		set_bit(DQ_FAKE_B, &dquot->dq_flags);
 	spin_unlock(&dquot->dq_dqb_lock);
-	mark_dquot_dirty(dquot);
+	ret = mark_dquot_dirty(dquot);
+	if (ret < 0)
+		quota_error(dquot->dq_sb,
+			"mark_dquot_dirty fails, ret: %d", ret);
 
 	return 0;
 }
-- 
2.40.1