Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.40.0.2.32\))
Subject: Re: [PATCH] Fix initializing response id after clearing struct
From:   Marcel Holtmann <marcel@holtmann.org>
In-Reply-To: <f2e0ca7e-4ffa-4e39-bd46-e0f70d23567e@www.fastmail.com>
Date:   Sun, 27 Dec 2020 21:36:17 +0100
Cc:     linux-bluetooth@vger.kernel.org
Content-Transfer-Encoding: 8BIT
Message-Id: <F7899F70-6238-4DFA-A0D0-95A18E256A4D@holtmann.org>
References: <f2e0ca7e-4ffa-4e39-bd46-e0f70d23567e@www.fastmail.com>
To:     Christopher William Snowhill <chris@kode54.net>
Precedence: bulk

Hi Christopher,

> Looks like this was missed when patching the source to clear the structures throughout, causing this one instance to clear the struct after the response id is assigned.
> 
> This is in regard to Message ID <20200806181714.3216076-1-luiz.dentz@gmail.com>,
> subject: [PATCH 1/4] Bluetooth: A2MP: Fix not initializing all members
> 
> I must apologize for not noticing this sooner, as I remember reading the articles linking to this patch several months ago. Either it slipped my mind, or I failed to notice it then. Apparently, nobody else noticed it, either, so I guess it's just not that obvious.
> 
> ---
> net/bluetooth/a2mp.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel