Received: by 2002:ab2:3319:0:b0:1ef:7a0f:c32d with SMTP id i25csp641406lqc; Fri, 8 Mar 2024 07:45:23 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWqvKrJT9lKM0q5N15s5Ek134vzKyrFDR2YmC9alH6URQST4d7UdoS3u7AntKU37IK3ev5tMIT2WVRQXjzcOrsM0zoeCv8hbSNxKH6Sow== X-Google-Smtp-Source: AGHT+IEtfBkbt6MbMYVYVQdknYy4k4+6n2Kg6ewiDmT6Uc6cALR/1Pd35Fk+7r5yu01P17f60+QR X-Received: by 2002:a37:e117:0:b0:787:ec1f:d6a0 with SMTP id c23-20020a37e117000000b00787ec1fd6a0mr11698904qkm.65.1709912723188; Fri, 08 Mar 2024 07:45:23 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709912723; cv=pass; d=google.com; s=arc-20160816; b=q8dKBSvGbCvPyeGnnzr06zU5iwY3UkdzzkHEC6psGQhsWz2FwnBqFbi820q2Kp8CKQ BaclGWDApsxjHJfqqf+W9IcaQQHIj6UStqmXbszxU2WAZyRHbr/LbKmQN4V5ioLTQVo7 fEnaFuzKS7w0nCOL++Xu4vMqOUuEvLHajA3F1bYSqBAgOkbJjr4VMOzLanwvWI4i8Iza yc9/1RX52Jie0h0F+4MZblNV5XppIeDwCQSU2iY3wbri+4NbzNPUrzG+LXOeW6uEgNQj 5hY3dmir/Xe+MqiTKD+N3MBK++jfZkj9EgnF/5RnW4xsa6YoqaZEYW1Ps3h4BZZXzeRN at8A== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:in-reply-to:message-id:subject:cc:to:from:date; bh=XkJffO4Kahb1zkXZvM3TCnNZLMJJAGLAGtJUiIL1pic=; fh=oH+d2QVWQr1FoAj0PloBENbVWKM/0bmT5+TOiiVMC3M=; b=VDOrtdQnXI/aDFMBqx0Po3MRUzieFPWSx5/0KZgFeOmRQsU4R7SL30fRIDhNvRcvS2 rnYK5nNke9V++26OQIGibw2ckDN0qvMwokoFYPlUarnt6559WLBIIjZCKvjQpUT1lF7P HV8+WmaYyXmZBvMB74GIzSq264sELoaAEuNBgDIMBHXoCKkSfAywbxp7ZLZRj9IEi6vw pG5lG44jLVY8Gaa8jS+eO/hLdL4k/D+kJRiPZLnNSHrWmTKyWZSpPa9oonKv1l6fsRP/ i3HjfMrfSBzS7TT8fyCczlnrLQCpvoeFsfWAAz9LgyatpwoCQURnoQfjokFxDglN4ZeM cU1A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=bues.ch); spf=pass (google.com: domain of linux-kernel+bounces-97247-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97247-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id vw12-20020a05620a564c00b007881c3a30d1si13355067qkn.544.2024.03.08.07.45.23 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Mar 2024 07:45:23 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-97247-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=bues.ch); spf=pass (google.com: domain of linux-kernel+bounces-97247-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97247-linux.lists.archive=gmail.com@vger.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 D0F931C2175B for ; Fri, 8 Mar 2024 15:45:22 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id B82F021A14; Fri, 8 Mar 2024 15:45:13 +0000 (UTC) Received: from bues.ch (bues.ch [80.190.117.144]) (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 4EF4E1F934; Fri, 8 Mar 2024 15:45:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=80.190.117.144 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709912713; cv=none; b=r7ZgaVFPbmIVOxzL21mz1MuyWN0Drg42Jr+Bi8sqife118/sM3nEllrUMkoBzLahXp3D5YYA3KD6sa4L98asIKVnCS88ZHhwYfhtReoDsnr8JaK5JWk6g+UV4g9BmzR39Eu1RM4bJ2Cp72NKgE+8IroGuivEQh8md/1TUMLKMfQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709912713; c=relaxed/simple; bh=XkJffO4Kahb1zkXZvM3TCnNZLMJJAGLAGtJUiIL1pic=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=iIfUW7f9EXzufV76Evwt5CLB+3HiG66lYy3O7yh4HjZaIHcfdvyZ8Imb7CpmGfoOlhAfcOSGSVxy/jQw0yrMzrjQMCPti35eUV9QS6/lyFDhqp5v1zOqwM60oUYiH5upSupoOuxd4B0DId1hMeOGThcQtleLHmxRQAzgGnBgXn4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bues.ch; spf=pass smtp.mailfrom=bues.ch; arc=none smtp.client-ip=80.190.117.144 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bues.ch Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bues.ch Received: by bues.ch with esmtpsa (Exim 4.96) (envelope-from ) id 1ricP9-000nrO-2u; Fri, 08 Mar 2024 16:45:03 +0100 Date: Fri, 8 Mar 2024 16:44:29 +0100 From: Michael =?UTF-8?B?QsO8c2No?= To: Rand Deeb Cc: deeb.rand@confident.ru, jonas.gorski@gmail.com, khoroshilov@ispras.ru, kvalo@kernel.org, linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, lvc-project@linuxtesting.org, voskresenski.stanislav@confident.ru, james.dutton@gmail.com Subject: Re: [PATCH v3] ssb: Fix potential NULL pointer dereference in ssb_device_uevent Message-ID: <20240308164429.0a01fc41@barney> In-Reply-To: References: <20240307223849.13d5b58b@barney> <20240307232927.171197-1-rand.sec96@gmail.com> <20240308060943.2410ef2e@barney> X-Mailer: Claws Mail 4.2.0 (GTK 3.24.41; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/UBAVpEGvFW.8SZN79fr457G"; protocol="application/pgp-signature"; micalg=pgp-sha512 --Sig_/UBAVpEGvFW.8SZN79fr457G Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Fri, 8 Mar 2024 14:36:56 +0300 Rand Deeb wrote: > Adding a redundant null > check goes against code clarity instead of enhancing it. You keep on moving goal posts. The check is already there. Therefore, this is about removal of this NULL c= heck. Which is not acceptable. > I respect your opinion, but it seems we are stuck in a while(1) loop > without a break. Don't worry. I'll ignore this thread from now on. --=20 Michael B=C3=BCsch https://bues.ch/ --Sig_/UBAVpEGvFW.8SZN79fr457G Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEEihRzkKVZOnT2ipsS9TK+HZCNiw4FAmXrMl0ACgkQ9TK+HZCN iw6djg//cErRxyXC6LYNIJ79g3Zqn0XiRGBUoxnpKZvv+kUJ+2qe14ne/H9CIEqT m+KyOI++0XWwWkrMEPvO01psTr3ra6WHeGFwYmVngj+hSmJ3mODWl+w+WCLeL6hU hhXabKrwaXs+2ZGAYDA8dyE6ihMSLsPpNElsUY1x6Y1id9NkkrSMkPXh/jCpa4lO Vzy5qXISAyQ4NIZajqH6e4oUjwuixr9oghFOkSe/FW7UI8aZ+Cci9F/YI2sJL9SD ThWiVfhEK9iiUtGulfW7gsknP0uEO9bj9oZj5KSd5Wvg4zhep59es/TzBJCYFLyF QShqke8zkRpRX1FaZLqhrSj2m9bnTSi5OmuwENBsuOELjDwbIPOx8+DHU4q6VMgq F+FZkHdzCNiAWHwWfc9Fi8brFdguVFN84FrvRaHkze76y0zh10uBDAHVeeY5ADDB rp/Kr0fPQxq/YoWqbbqZggE6CdoxrWQG6JypC+OwFeEGkLL5rFAnMwmC4nAbJAov +UE1Qd5OcgGsH7p/7BbVDlf6IWBYhqxBsFAMKzjPF/MeqBzhYYi8rwVxP4VOpgjL kJmgb5n+MtuLD5lCvdY+P+Rbn7M2JeAIaoDnA/QdgGB6Fs52+4SeHuPIDf9WXFDW q3kjTBJNfmDk7SykJn9jAt6Tv8Bl+IuTIB2XigrFy4PETaX2CNE= =6clV -----END PGP SIGNATURE----- --Sig_/UBAVpEGvFW.8SZN79fr457G--