Received: by 2002:ab2:620c:0:b0:1ef:ffd0:ce49 with SMTP id o12csp553039lqt; Mon, 18 Mar 2024 16:39:31 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCXcNBPARcoEZofPZCkzbgeiTJzWu1C/uuHPb5dFr+eHtNBiWqdU7cAZSZU52/NIV1b3PawahB/Sz8jFZs5KJrvDG0duZeEUk+8aB7LVMg== X-Google-Smtp-Source: AGHT+IEaL1NlLLXrMgXVB5z/ugGF4VU/mz+OS1jPguwGGBRnV/0Vf0tw/7uzzOghsT5q3kXimBbW X-Received: by 2002:ac8:5a82:0:b0:430:c4a6:1108 with SMTP id c2-20020ac85a82000000b00430c4a61108mr1964803qtc.19.1710805171115; Mon, 18 Mar 2024 16:39:31 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1710805171; cv=pass; d=google.com; s=arc-20160816; b=zZnurKiRsVD5QAtWxC1JLDhd3uI9XSeLJilELcL2G0Vh7LSJxPERqqp7W7bG8VFvCj PbVdXVBvxft3haQiwD1s3Yw6vhdEssZpSnFfFvsUVrZvugdSr7CpTFVC0y0neyxBQMiF vjtN7ZJ0Hz1oTZ6OgjapFW3EGq6nRJT2Mi3mfO5XcfcIupWfaCWPsV8Wsuursa7Vs2CX tCCMwl+/zGLFpNYPYvxeXzyuYPtvdIiZoJC/5iJcc8q5shQ3a1/j/ovycnCTy1taPvj0 GopvHOc1u5LoK4NLFcFO7BWEf9Oo7C55i740GI3jIea700x7igV/E3JweDafQqxYroVc rvYA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:dkim-signature; bh=3a9h5J2Sh8PUt4bHEGgwNPRDWFa/RlmPLDNfy47RBYk=; fh=g8GAOG2z8AjAJpWcxBbrGe08DRPpRNDHzJ6xAott6Pg=; b=c2YZ9fyIh1mOwmJa4iGvv0zyOcqBnLUJHjehM53dQ9yFAILQI4Rwj6SIs8gmS2pYlT mazOlawefpSwweMzQiFahZkZ4dYILXQlZzmHNv7+O3uAcZIgkqK0vM8KUDA6b0FaHY6/ Rn+Ni4jv7Ern3ivrFTwcowyR+A48JCkb7UEGkxt9nciyAuAacKXpDwA0OjFHpIINbKPs QqNdOrR4a0Axp3hELBGV+lGsqq57/DcSi/En+eR4+O8Bj1F+bKm+18bGJZLdljDuVGt+ owqTbuUQDnxUjh4lC8AekpLKNXFJpgWd5xEEsRjSLeGYDIIfi/ijrJPnT7DM19r2EGnr lZfg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=fRxQxe1v; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-crypto+bounces-2736-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-crypto+bounces-2736-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id x16-20020a05622a001000b0042ef806d090si5203388qtw.111.2024.03.18.16.39.30 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 18 Mar 2024 16:39:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-crypto+bounces-2736-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=fRxQxe1v; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-crypto+bounces-2736-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-crypto+bounces-2736-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 79E391C21A69 for ; Mon, 18 Mar 2024 23:39:30 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id BBBF55F862; Mon, 18 Mar 2024 23:39:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="fRxQxe1v" X-Original-To: linux-crypto@vger.kernel.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 712655F859; Mon, 18 Mar 2024 23:39:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710805145; cv=none; b=AuQp2vawRqTbW1j3sdmUNtrryyUVomPepuhgnZ3S0Rsv/jlMpm8nKhDbveBp2EMCdK6frS0y6oQywY2GC5Zc9gbM5rov35N6WClS2WdOOYFoLvI+S66kl+XdLkxTaDw84aokJW3FC0lpdq17TzHzMSMG2BQ31acZvClALvQ33qY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710805145; c=relaxed/simple; bh=vNPV+Zm1jo6GQke9UBMxQprrLogWyW6mK1dZOZf+eZ4=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=AASBViRejV7jwIgjj+TW5gZgGwB0qUAp1iLUrDulUylYjECAxz/P/xpSgTfAUl/s/DVSzj0cigz0oLs0XeTXN6N3BBd4sm2vTBkmJ5Q52ofxcE2psyFY/eZLXg2fcYhj22p24gXlNMaK7ZmO2ngODUV+TXWPlMqnAeml/QHT4uA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=fRxQxe1v; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id E5A9FC433F1; Mon, 18 Mar 2024 23:39:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710805144; bh=vNPV+Zm1jo6GQke9UBMxQprrLogWyW6mK1dZOZf+eZ4=; h=Date:Cc:Subject:From:To:References:In-Reply-To:From; b=fRxQxe1vX8gFA4H4/tqhTxyx1PTwtyNbL63yiwezOezdjGrGu2yUxdzdrO6YfwvDJ Gy2MYnigaGTllxZyONLD/zOrf14lQ9QxcUqxZc4DwlWfec16z2A2zYb3Ja7d2ggEAj SDGtivMXm0puj/iglEDdFF7AJBHiljiYpaCvuaopqHnubCGqrkbs+ijVyhNaALqQpb jFacRouvnfhdEc9PNNd1FFZquD0pWYM6LXBh7iot2GreTyHec1jci00jEH0FwPmum/ C8QpZe/h539LeIGNqtwghhAZzX2XYyqzRk3f2Hd6ddNmiBkELNZuQjVwWXXvGPs1JW 6Z436URGh79cA== Precedence: bulk X-Mailing-List: linux-crypto@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 19 Mar 2024 01:39:00 +0200 Message-Id: Cc: , , , , "Sergey Shtylyov" Subject: Re: [PATCH] KEYS: prevent NULL pointer dereference in find_asymmetric_key() From: "Jarkko Sakkinen" To: "Roman Smirnov" , "David Howells" , "Herbert Xu" , "David S. Miller" , "Andrew Zaborowski" X-Mailer: aerc 0.15.2 References: <20240315103320.18754-1-r.smirnov@omp.ru> In-Reply-To: <20240315103320.18754-1-r.smirnov@omp.ru> On Fri Mar 15, 2024 at 12:33 PM EET, Roman Smirnov wrote: > With the current code, in case all NULLs are passed in id_{0,1,2}, "current code" is not unambigious reference of any part of the kernel tree. Please just write down the function name instead. > the kernel will first print out a WARNING and then have an oops > because id_2 gets dereferenced anyway. Would be more exact": s/print out a WARNING/emit WARN/ > Note that WARN_ON() is also considered harmful by Greg Kroah- > Hartman since it causes the Android kernels to panic as they > get booted with the panic_on_warn option. Despite full respect to Greg, and agreeing what he had said about the topic (which you are lacking lore link meaning that in all cases the current description is incomplete), the only thing that should be documented should be that since WARN_ON() can emit panic when panic_on_warn is set in the *kernel command-line* (not "option") this condition should be relaxed. > > Found by Linux Verification Center (linuxtesting.org) with Svace. I'm not sure if this should be part of the commit message. > > Fixes: 7d30198ee24f ("keys: X.509 public key issuer lookup without AKID") > Suggested-by: Sergey Shtylyov Should be reported-by. > Signed-off-by: Roman Smirnov > Reviewed-by: Sergey Shtylyov > --- > crypto/asymmetric_keys/asymmetric_type.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/crypto/asymmetric_keys/asymmetric_type.c b/crypto/asymmetric= _keys/asymmetric_type.c > index a5da8ccd353e..f5cbd6ff14e2 100644 > --- a/crypto/asymmetric_keys/asymmetric_type.c > +++ b/crypto/asymmetric_keys/asymmetric_type.c > @@ -60,17 +60,17 @@ struct key *find_asymmetric_key(struct key *keyring, > char *req, *p; > int len; > =20 > - WARN_ON(!id_0 && !id_1 && !id_2); > - Weird, I recall discussing about this issue in the past. Unfortunately could not find the thread from lore. Anyway I agree with the code change. > if (id_0) { > lookup =3D id_0->data; > len =3D id_0->len; > } else if (id_1) { > lookup =3D id_1->data; > len =3D id_1->len; > - } else { > + } else if (id_2) { > lookup =3D id_2->data; > len =3D id_2->len; > + } else { > + return ERR_PTR(-EINVAL); > } > =20 > /* Construct an identifier "id:". */ BR, Jarkko