Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp1733819pxb; Thu, 16 Sep 2021 14:17:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwepCZiPgMHy022bJlUsx02f3/CeUghttIgrRaY8WZuNtv09Qz6BDru683BcmpHHihtEBkh X-Received: by 2002:a92:c8c7:: with SMTP id c7mr5610829ilq.62.1631827059546; Thu, 16 Sep 2021 14:17:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631827059; cv=none; d=google.com; s=arc-20160816; b=ATfXVJbOKuzmgwnWt4r5EUeZMUywrZtlcVUUYjgpz6vqf1XAnEc3fkEoY6JxlZ9dL0 0LOpUxjQiV310Uotum+G9yF0BbN3b+5RnDkI2NHCgksXcESAaME820udMiik3SSZBj71 ZQVlux/B1MckuOoMnPLYhmI1aAb4islZ2t8pxM6IV5oTuwsNLjHbwYWl7/988jDSmPTi IxsH3x2NvTTncc/jRk4dhvLfEBei2Pda+Y6ioyOoWy3FEKMXl15ZsrU0w1wzn6F71uX5 viALZ3++DYn49tWoiseZbZgL3mi1pd0wCZHTAXqicGVa0NMK0bKeEl/caADdgbABZfA3 vMSQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=iBmkBPf5sHKfZJmErXuuEkd1lcVtgqLc1MHj8ltC8yk=; b=y0SZwpRymoLctyaAOypvfSqVe0WMR6kqj9p8u+gR9Fd4saB1BXnPpIrnKNG/a4mbJG VTqC0wk9kst1JkQyrE2YpuaXISXQDkCRT6YyM5+oQ80JKZUvgIr1iw8Mf1uCyGUoTVwe fjn8sZ9HuAE2xxXT6PHopKBJTNZ6AhMjI3OJLN9szQWwOGx0NYT8A9zEUnv0XJgS7Pe7 Xbi8WkoTYEPXy2xbWHMcgplEA2eJ1K8l/4oOZoVbzNb8GGt47FGzfwpZ5sE86cDHNwyF NU+vPrbSBL0mvWy2SWNLfrblh3OkdZ/4y6KEgfusSniY9NsKmbGenadi7hqxrqqsT9HX KWjw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=Mu2XxU7w; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q11si3601222ilg.114.2021.09.16.14.17.27; Thu, 16 Sep 2021 14:17:39 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=Mu2XxU7w; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244364AbhIPQfI (ORCPT + 99 others); Thu, 16 Sep 2021 12:35:08 -0400 Received: from mail.kernel.org ([198.145.29.99]:37900 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236611AbhIPQ1a (ORCPT ); Thu, 16 Sep 2021 12:27:30 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 8B78F61555; Thu, 16 Sep 2021 16:17:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1631809056; bh=aQ3ss94Ysz8Z2rgJ7EBicK34Av2AyGubTaj1Y/1YGx4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Mu2XxU7wsY4p+bWmos8vzp0jwQRb5iTkfiHdtS9Lezb/xQ9KQfDvGqExB+1XuWlEg ZLzxEshcqENCHXdrob/WFFIlpFcMJkyC388k9s+/e6Jt0sCgDW8XahnycvdYeJuh8M Gge5aM3nF4EyiNq6wSEUmLaaVO6naizhywM2zf6c= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Harshvardhan Jha , Stefano Stabellini , Dominique Martinet Subject: [PATCH 5.13 015/380] 9p/xen: Fix end of loop tests for list_for_each_entry Date: Thu, 16 Sep 2021 17:56:12 +0200 Message-Id: <20210916155804.487482613@linuxfoundation.org> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210916155803.966362085@linuxfoundation.org> References: <20210916155803.966362085@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Harshvardhan Jha commit 732b33d0dbf17e9483f0b50385bf606f724f50a2 upstream. This patch addresses the following problems: - priv can never be NULL, so this part of the check is useless - if the loop ran through the whole list, priv->client is invalid and it is more appropriate and sufficient to check for the end of list_for_each_entry loop condition. Link: http://lkml.kernel.org/r/20210727000709.225032-1-harshvardhan.jha@oracle.com Signed-off-by: Harshvardhan Jha Reviewed-by: Stefano Stabellini Tested-by: Stefano Stabellini Cc: Signed-off-by: Dominique Martinet Signed-off-by: Greg Kroah-Hartman --- net/9p/trans_xen.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/net/9p/trans_xen.c +++ b/net/9p/trans_xen.c @@ -138,7 +138,7 @@ static bool p9_xen_write_todo(struct xen static int p9_xen_request(struct p9_client *client, struct p9_req_t *p9_req) { - struct xen_9pfs_front_priv *priv = NULL; + struct xen_9pfs_front_priv *priv; RING_IDX cons, prod, masked_cons, masked_prod; unsigned long flags; u32 size = p9_req->tc.size; @@ -151,7 +151,7 @@ static int p9_xen_request(struct p9_clie break; } read_unlock(&xen_9pfs_lock); - if (!priv || priv->client != client) + if (list_entry_is_head(priv, &xen_9pfs_devs, list)) return -EINVAL; num = p9_req->tc.tag % priv->num_rings;