Received: by 2002:a05:7412:da14:b0:e2:908c:2ebd with SMTP id fe20csp2318271rdb; Mon, 9 Oct 2023 23:20:05 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHc5hn7LwEXkTzJxuyvXxxulaIYS46kyltrxA0gS+I7cuH6KFyRA0MC1Nx4qyLfaOXy1KN1 X-Received: by 2002:a05:6870:a98b:b0:1d6:55a4:d97 with SMTP id ep11-20020a056870a98b00b001d655a40d97mr19045502oab.32.1696918805110; Mon, 09 Oct 2023 23:20:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1696918805; cv=none; d=google.com; s=arc-20160816; b=SFTKsctPYdKg9+oqEHxxikRtYBd3aX2EPf66g3QwEe2e5oVNZRZiGc/PZBPa9h3pMm 3Hl4AcX5poykN+3j8gPYvMfzll7IgcYunTfBEsfg88HKHamMopkHs8G/KnTT93kUUWDp OctPAwMeE6GNN9+HrnmrKOjAH9P6jrVO5eehcYslqdwiw0sXMEUsb4VqhTbZNqqkLBH8 C0+gTjO6FLF97sPcw0hTtXpBtLjvlZ4DYIXYpK+KrVc7ipPA52NuXUJ1ol8fZxvAzoEg ZZpNvxG/CKugYwxwAYR3TOkx5kkz9D0hQRHMyokNAkJV3mndRs2WxKYim/ge0WlAIYZZ 3Hrg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent:message-id:in-reply-to :date:references:subject:cc:to:from; bh=RLR4o1b9VvT1SqpS3h/1/0hEauFb8a45Qsvowcxr6Fs=; fh=ZFLsITcEvg+NuYoPxye5twGvM7AR0AV82GRo2u9+Arw=; b=C7Lr04HPJAOsezirg0mJ+GjuwHOCKFIYF+2mbLvN0F16++HW5tLpqiWqECBnGqJbUh JP5OKhmf9+/OP2+C6DdtbKvSAaAg1WYOM/Gf+7nVB9G/CpU6H8S0jTA82XJPQm9UKzWX ChFMZPAl6ZtczsJI6aL9VmOcv84mGYpmgQg73lkk4TxrI7SoM2uG3ExYBxglFHsHKCzx +/QX5I9GPTfpkcL3BuiSr0kNSP/M9f/0HBS4BlhDdFLOfnw8t6rgLIvnNiBV9DH5YhFx tSgimeP5jaoXKBMAL7kqVR2Eh8OJ7wHhXhoFbMtTT2WYX8YUgwGBvfMwRcEedBJdhVLA hguA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id z1-20020a63c041000000b00573fd9be4bdsi11144536pgi.493.2023.10.09.23.20.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 09 Oct 2023 23:20:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id CFE8F811F91F; Mon, 9 Oct 2023 23:20:03 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1442235AbjJJGUB (ORCPT + 99 others); Tue, 10 Oct 2023 02:20:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33294 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1442156AbjJJGUA (ORCPT ); Tue, 10 Oct 2023 02:20:00 -0400 Received: from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net [217.70.183.198]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1CBCD9D; Mon, 9 Oct 2023 23:19:57 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id 92222C0003; Tue, 10 Oct 2023 06:19:54 +0000 (UTC) Received: from peko by dell.be.48ers.dk with local (Exim 4.94.2) (envelope-from ) id 1qq65s-00GDHo-Io; Tue, 10 Oct 2023 08:19:48 +0200 From: Peter Korsgaard To: Greg KH Cc: Javier Carrasco , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , syzbot+1f53a30781af65d2c955@syzkaller.appspotmail.com, netdev@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] net: usb: dm9601: fix uninitialized variable use in dm9601_mdio_read References: <20231009-topic-dm9601_uninit_mdio_read-v2-1-f2fe39739b6c@gmail.com> <2023101036-fleshy-dude-aec0@gregkh> Date: Tue, 10 Oct 2023 08:19:48 +0200 In-Reply-To: <2023101036-fleshy-dude-aec0@gregkh> (Greg KH's message of "Tue, 10 Oct 2023 08:00:14 +0200") Message-ID: <87v8bfc83f.fsf@48ers.dk> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-GND-Sasl: peter@korsgaard.com X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL, SPF_HELO_PASS,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Mon, 09 Oct 2023 23:20:04 -0700 (PDT) >>>>> "Greg" == Greg KH writes: > On Tue, Oct 10, 2023 at 12:26:14AM +0200, Javier Carrasco wrote: >> syzbot has found an uninit-value bug triggered by the dm9601 driver [1]. >> >> This error happens because the variable res is not updated if the call >> to dm_read_shared_word returns an error. In this particular case -EPROTO >> was returned and res stayed uninitialized. >> >> This can be avoided by checking the return value of dm_read_shared_word >> and propagating the error if the read operation failed. >> >> [1] https://syzkaller.appspot.com/bug?extid=1f53a30781af65d2c955 >> >> Signed-off-by: Javier Carrasco >> Reported-and-tested-by: syzbot+1f53a30781af65d2c955@syzkaller.appspotmail.com >> --- >> Changes in v2: >> - Remove unnecessary 'err == 0' case >> - Link to v1: https://lore.kernel.org/r/20231009-topic-dm9601_uninit_mdio_read-v1-1-d4d775e24e3b@gmail.com >> --- >> drivers/net/usb/dm9601.c | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) > What commit id does this fix? It has been there since the beginning, so: Fixes: d0374f4f9c35cdfbee0 ("USB: Davicom DM9601 usbnet driver") Acked-by: Peter Korsgaard -- Bye, Peter Korsgaard