Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp849319ybe; Fri, 6 Sep 2019 08:12:51 -0700 (PDT) X-Google-Smtp-Source: APXvYqwK1BdnY9RMQ7ibEhKY7W/Xcp5BIKb/bqQ11rjUGCZ4z2vhpUARooAyZ7RuQ3gxoMZJr2mL X-Received: by 2002:a17:902:9a8d:: with SMTP id w13mr9558879plp.157.1567782771096; Fri, 06 Sep 2019 08:12:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1567782771; cv=none; d=google.com; s=arc-20160816; b=hRGsIV+WAQ+w2B5LArGsstoBUDnPsb1f2zK8B0VrxxN8ze+J+oZ4Dn2lfqiVJlBuZ6 rzpYKxf6WoercYZKsTmr0Uc0ql21a4oPEypvSDCW85OPZYPMkzVlhQBH0wCQZkLuG2lT G7phAD5kL/RNon/SJI2VneUlOOWqmdk0CXHxy8uvzzfzrz+MUkpKOcsECnTA0wU993Nz mLXxWbioNgNqGbGwf/RiVa3odDcXE+aAFX6GYXbpbm3SqI3s72yh2kgbMtzFgBXu/d93 8JVRdCwz6knsobH3vmzNmHj4pqdylV4Hi/BUrDbg9bh4pJFZgTkFJcYGmLmi8lNChgNm cO6A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=Q3wnUri8A3MSngjww1WJQHvJg6Dsi5neuJrd5Rfx5VI=; b=kU/W8uQf/YJBBrzeY5KxLy0dWjpY1FaDe4DsEwsniawds90Rcw5yaZ5kGDvc31bpsr REeYDJ9NFBgCHaKxO3aj0NuWVCcEYRlniiTLBUSHlhonzd8CZr+VOmXX1iFX3+umvm2R KSkxw2XEwMCxjcd9Fmqbd/sDMaummkke90kL9ZlISEM+NRl7YqqP1HosIbAl8OIPfu31 YVZcC41goLGw0Lyci+UKDaVVtfgIrJrK4+Mtm4VxqFseXi+2/T36CchNxhfdR13mb9NA HfYbSbY3Dum82eG/ZTBaHG0DRClb5MVZasRZWcJNebsyW3H1zl/8gKtR/MtRCC85FSe8 eHcg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q26si4719742pgt.233.2019.09.06.08.12.33; Fri, 06 Sep 2019 08:12:51 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2391112AbfIFLws (ORCPT + 99 others); Fri, 6 Sep 2019 07:52:48 -0400 Received: from mx1.redhat.com ([209.132.183.28]:52204 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727381AbfIFLws (ORCPT ); Fri, 6 Sep 2019 07:52:48 -0400 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 16FFF305D637; Fri, 6 Sep 2019 11:52:48 +0000 (UTC) Received: from localhost (ovpn-117-208.ams2.redhat.com [10.36.117.208]) by smtp.corp.redhat.com (Postfix) with ESMTP id 51C755D6A9; Fri, 6 Sep 2019 11:52:42 +0000 (UTC) Date: Fri, 6 Sep 2019 12:52:41 +0100 From: Stefan Hajnoczi To: Vivek Goyal Cc: linux-fsdevel@vger.kernel.org, virtualization@lists.linux-foundation.org, miklos@szeredi.hu, linux-kernel@vger.kernel.org, virtio-fs@redhat.com, dgilbert@redhat.com, mst@redhat.com Subject: Re: [PATCH 13/18] virtiofs: Do not access virtqueue in request submission path Message-ID: <20190906115241.GU5900@stefanha-x1.localdomain> References: <20190905194859.16219-1-vgoyal@redhat.com> <20190905194859.16219-14-vgoyal@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="iBHcHRCIarfY7C0j" Content-Disposition: inline In-Reply-To: <20190905194859.16219-14-vgoyal@redhat.com> User-Agent: Mutt/1.12.1 (2019-06-15) X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Fri, 06 Sep 2019 11:52:48 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --iBHcHRCIarfY7C0j Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Sep 05, 2019 at 03:48:54PM -0400, Vivek Goyal wrote: > In request submission path it is possible that virtqueue is already gone > due to driver->remove(). So do not access it in dev_dbg(). Use pr_debug() > instead. >=20 > If virtuqueue is gone, this will result in NULL pointer deference. >=20 > Signed-off-by: Vivek Goyal > --- > fs/fuse/virtio_fs.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Stefan Hajnoczi --iBHcHRCIarfY7C0j Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEhpWov9P5fNqsNXdanKSrs4Grc8gFAl1ySIkACgkQnKSrs4Gr c8gXMQf/QUTibXMbQdKidGB1Vqv7pi+fendYycKROG3QMPsuTAtmXYzqvaPyQ+V7 IQKRcXkPStzxee3RJePDleTtTNsjDlsuLgfpvgh0Bik7ZTOfN7ddHzXvcBx2VDJf OtV+D2N9Cm2YkI1r2OEVWylqmv9aH83xFJHj5hsc7vNO+yu9V+7TA+RZKasdT1GX zd/W2E4xlYS/vz9fKRX7QReLWIla2yJVy7Qb6MECtMGELGmq02OdzsO5SJWX52cG gSh58Gw1wIyidQXqTbEkl+gNMORVs3xtNvlIXNyEcXjvaVZzRCAdq3UGVAFL2C7x LeqWw3/P3WTKxDZwTHLyAwoo2fpnpg== =cLla -----END PGP SIGNATURE----- --iBHcHRCIarfY7C0j--