From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH] quota: paranoia: check quota tree root Date: Thu, 12 Feb 2015 15:04:06 +0100 Message-ID: <20150212140406.GB12905@quack.suse.cz> References: <20150212093649.6117.62111.stgit@buzz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org, Jan Kara , linux-kernel@vger.kernel.org To: Konstantin Khlebnikov Return-path: Content-Disposition: inline In-Reply-To: <20150212093649.6117.62111.stgit@buzz> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Thu 12-02-15 12:36:49, Konstantin Khlebnikov wrote: > Root level in quota tree must be already allocated otherwise this block > could be used for something else. Thanks. I've added the patch to my tree. Honza > > Signed-off-by: Konstantin Khlebnikov > --- > fs/quota/quota_tree.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/fs/quota/quota_tree.c b/fs/quota/quota_tree.c > index d65877f..58efb83 100644 > --- a/fs/quota/quota_tree.c > +++ b/fs/quota/quota_tree.c > @@ -349,6 +349,13 @@ static inline int dq_insert_tree(struct qtree_mem_dqinfo *info, > struct dquot *dquot) > { > int tmp = QT_TREEOFF; > + > +#ifdef __QUOTA_QT_PARANOIA > + if (info->dqi_blocks <= QT_TREEOFF) { > + quota_error(dquot->dq_sb, "Quota tree root isn't allocated!"); > + return -EIO; > + } > +#endif > return do_insert_tree(info, dquot, &tmp, 0); > } > > -- Jan Kara SUSE Labs, CR