Received: by 2002:a05:6358:16cc:b0:ea:6187:17c9 with SMTP id r12csp13157820rwl; Wed, 4 Jan 2023 04:28:26 -0800 (PST) X-Google-Smtp-Source: AMrXdXu5tjEatE/0J2oKj+oCi0IA9+qqekTeE8py/UVuOY6qhuoJ7nbPXFsdFMp8YfXPyn1WMxtA X-Received: by 2002:a05:6402:124a:b0:47f:b4f3:6db9 with SMTP id l10-20020a056402124a00b0047fb4f36db9mr50525089edw.13.1672835306191; Wed, 04 Jan 2023 04:28:26 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1672835306; cv=none; d=google.com; s=arc-20160816; b=dM4G4zfI5XDO3ZTNqPwOCdJsS+Q8w5Kr3wSfX5LUe+1WGn0GjvgQjHiuPw4QqC2jvt btV3iF1J4i4axJln1O3lYzYKw2QmAmgluRHI4zEQNYoV/xMisKzLKV5NVmytSmaNYZr/ aKfhr1QmGraEK2sxExhq0WYjNj21MiDnzn9O/Ov0HeD4mtVj4KGV8tSAFn40FurJYta7 3xt1Hd78tryjXGl2bN3N04HjDhotF3M2Gs/uhTp+MwGasZb94QAI5Zy7nSqpsaNViTM4 4VLCAE+bHLCP/dZOZJyDlBwT+hTXcdCJJyeUAw3XvrlDaz50vhQa4dPjtfX2By6T8UiT orXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:references :in-reply-to:subject:cc:to:dkim-signature:from; bh=/+pRFYBFedAsDHtqPmYRk6+gARXS3x0OS30HV0pfR2o=; b=SIFGgy5XhCdkbv/+TUOg0dDhg+tuB8IuoGhV6+XoBbu0rdwIwMsJuw2Yvs2df8JhDr oJNV5Kn5vrsNdzPEy0JXD9OuhKiMX8qvQzTHUAqDeE5vwJuJf5vs166PNdt6k0PsUj2Z ojrBRH/JWxv0dqaqRV4wUWUadabo3RYCMTR5amuw4nRn59nc92Q0mnbFfdbHJgnZxUcL /oa6Bo/lxELX+ZbLYybsdlYZVflxNVuIiW3OaUnFLi2dj9fDD/S97MVnf63bchwPpyzQ g71Of1j8b2JIteBwOWhkMGAejmoQ1uhHTewujpCznRR3kBABV0qhNiz0ndyAGcMaKDQP hQAg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@toke.dk header.s=20161023 header.b=PO6juiQS; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=toke.dk Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id en9-20020a056402528900b00484e1329439si17349582edb.156.2023.01.04.04.28.08; Wed, 04 Jan 2023 04:28:26 -0800 (PST) Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@toke.dk header.s=20161023 header.b=PO6juiQS; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=toke.dk Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234667AbjADMZQ (ORCPT + 67 others); Wed, 4 Jan 2023 07:25:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50314 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230200AbjADMZP (ORCPT ); Wed, 4 Jan 2023 07:25:15 -0500 Received: from mail.toke.dk (mail.toke.dk [45.145.95.4]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 229F227A; Wed, 4 Jan 2023 04:25:15 -0800 (PST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=toke.dk; s=20161023; t=1672835113; bh=/+pRFYBFedAsDHtqPmYRk6+gARXS3x0OS30HV0pfR2o=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=PO6juiQSEWQ1PvLyaLt0qEeuSEh5AXrX2wGV4plGsdqSOw3RdtozBvvggYtXUZbvW WmTVS+NiKhPUGlQEpoRy/557epfU56AU/H7QxUAisd8q0vFoIl/ssTuVIERPvmMPOJ QEju92uqpzntj2r/qKU8feDGrDdi+kkw+PLWJPR/yBCDl+PWDGUjAmNXx0EiEd+ydr 3EeRJzZeJmVtzSawzBjEtnQ78nfOmEslEUfW2AtON1Y4zZeKKLK3IgH4KAKAkfPXSM zzADMXSg7XiiROdIi9T14UfHfJl/H7Z2kx2xCtBY7gbcej5sT17mqvMkOsPIlQkJAF IMwMXnTfUnpiA== To: Fedor Pchelkin , Kalle Valo Cc: Fedor Pchelkin , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Sujith , "John W. Linville" , Vasanthakumar Thiagarajan , Senthil Balasubramanian , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Alexey Khoroshilov , lvc-project@linuxtesting.org, syzbot+e008dccab31bd3647609@syzkaller.appspotmail.com, syzbot+6692c72009680f7c4eb2@syzkaller.appspotmail.com Subject: Re: [PATCH v3] wifi: ath9k: htc_hst: free skb in ath9k_htc_rx_msg() if there is no callback function In-Reply-To: <20230104121558.38969-1-pchelkin@ispras.ru> References: <20230104121558.38969-1-pchelkin@ispras.ru> Date: Wed, 04 Jan 2023 13:25:13 +0100 X-Clacks-Overhead: GNU Terry Pratchett Message-ID: <87edsa32s6.fsf@toke.dk> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS 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-wireless@vger.kernel.org Fedor Pchelkin writes: > It is stated that ath9k_htc_rx_msg() either frees the provided skb or > passes its management to another callback function. However, the skb is > not freed in case there is no another callback function, and Syzkaller was > able to cause a memory leak. Also minor comment fix. The comment fix seems to be missing from this version? So either it should be reinstated, or the commit message updated to not mention it... -Toke