Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp8162315pxu; Sat, 26 Dec 2020 19:18:47 -0800 (PST) X-Google-Smtp-Source: ABdhPJySfkSARBqLWb0V5H4h9sE18aqS/jv6vlbE9jPuoqckQc92a+fqWM5i5z7g3uFw1WfRvEdr X-Received: by 2002:a17:906:fb1a:: with SMTP id lz26mr36635518ejb.194.1609039127728; Sat, 26 Dec 2020 19:18:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1609039127; cv=none; d=google.com; s=arc-20160816; b=yKHI0uDAVxEOuvmw3QmsWVYYceoGmcu1GbI/yqlxTERBUoOCPd4mhAAmDa/gmKZq/+ G+vnrpJ4A94lT6RiWeFihQULnoNnXPqV+rwkDTtBvAzj1F9D5H1FFpzGuyVmQOwfCOnJ 9C8LOQIwtY73b6xEbJsapVTrdqMTc0/4EjxLxV91QP/guY3xiL2Qbdw3KAKYf+KXkxO+ JezvHHrBnLeLiQ5mgsGxveNgdL4pZxZhcJ2Qc7oQve3H724if/A2v2s9xNVW+ndOnDVw Ndi4/EjIKXL9JTLOXE72KbhpDskx8BOQ72O00YN+KJJZix4MUHYXIV6r0RHeljThHqIJ 1Osg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:subject:to:from:date:message-id:mime-version :user-agent:dkim-signature:dkim-signature; bh=Sqg6xhuI3FJYBvYXktRNdcBtMhsmYgzm99qWgxD7h9c=; b=i+eOrzSB6HscqIPniRzQ0ebWUTX+UVTOnZkkPMOHelXxpva0FfwNF0tPLMFlPWgwDG NSumWvS7b0v6lKyp3fStjJSNIkLYgjRv4LJ+AZq2kf8yHPn+76/uMIu6rksIPqCNFgea Dfc8JkTKg/DKJWDa7CYW/IVme4hQpPSSeaBjya0S3IysGUd5TwmFm88Jvq/CJAibspxD g6uRTOZo3+cukowNgeczvhCIP6/ritJK4Bdavls3DYhhG81Me8n9AWzJC4SyKatwHDGl 7en4HPYxz08raTO+zIX83WGw9LljekrTFHI/0NFhoIa4QIu5wVypMAR6uMqTi7tCv5qK K9TQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kode54.net header.s=fm2 header.b=JwfApTyP; dkim=pass header.i=@messagingengine.com header.s=fm1 header.b=SHtHYeb5; spf=pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-bluetooth-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id lt15si17108850ejb.499.2020.12.26.19.18.10; Sat, 26 Dec 2020 19:18:47 -0800 (PST) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@kode54.net header.s=fm2 header.b=JwfApTyP; dkim=pass header.i=@messagingengine.com header.s=fm1 header.b=SHtHYeb5; spf=pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-bluetooth-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726253AbgL0DOC (ORCPT + 99 others); Sat, 26 Dec 2020 22:14:02 -0500 Received: from wout1-smtp.messagingengine.com ([64.147.123.24]:44531 "EHLO wout1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726024AbgL0DOC (ORCPT ); Sat, 26 Dec 2020 22:14:02 -0500 Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.west.internal (Postfix) with ESMTP id 6052F864 for ; Sat, 26 Dec 2020 22:12:56 -0500 (EST) Received: from imap6 ([10.202.2.56]) by compute2.internal (MEProxy); Sat, 26 Dec 2020 22:12:56 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kode54.net; h= mime-version:message-id:date:from:to:subject:content-type; s= fm2; bh=Sqg6xhuI3FJYBvYXktRNdcBtMhsmYgzm99qWgxD7h9c=; b=JwfApTyP 3x8GJ8BIwLRxJVZJ0XZuSR4b4Of+u8+y48VQw4CMQUnY7m/Dj8V59XNZuiWg7gPL 1khoyWRJ8IVfVBONCK/AzPz9Djov1i0fJ+pX5T9z/WuihNvs4iGmWyY+84LZf0sP 1UGVXzM4BQxR02V0CFJ46vmk/hgHor1esXQQ4bQS1ocegnjmg7OrJKUiJDfzGpj5 kSw6Btpo4UHcOHTjBPOXgMd3UJBh7qUvNs3HLlUnc8iYLm3IUsPeGEe91nGFRsKg 2zjfexfJy6mUAg7VFjdejfwaeh/oHUXGyBbBJIBDCk9ZVmGoXr5zYMitPFeV+UPV sNuxwzCoIwPpkA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=content-type:date:from:message-id :mime-version:subject:to:x-me-proxy:x-me-proxy:x-me-sender :x-me-sender:x-sasl-enc; s=fm1; bh=Sqg6xhuI3FJYBvYXktRNdcBtMhsmY gzm99qWgxD7h9c=; b=SHtHYeb5EnpYBkgbu3kpq1fOgVy1UAomAK9yp585P1EA1 +2mtnuteyWy3a/OkYzmffuLm39ZwbXCNKcpBxhDtr/7ds3p0HPO7U7cX1g3035x8 4BMZvd/IF1LBNXJcDZa+lOipbGnsoDEt7ay4XxqLLE4ytHT2lAncgawROcDMgecj igVil/dh4FzWfWG3xpYUDw3fsX3tG712ZTMyIX1B4j4eNUwFLP6d5cROQGnkDLZE U78qmdeTpEqTarQkJXfziIJRIhCsrRGLHR/dj0xJf2ouys9gW6fLqA3qi2MSXYXW m1Kq0Zf/NG3MdfJJQKyC0pymKOgI7LQFkay2jk0vA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrvdduiedgheejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucenucfjughrpefofgggkfffhffvufgtsehttdertd erredtnecuhfhrohhmpedfvehhrhhishhtohhphhgvrhcuhghilhhlihgrmhcuufhnohif hhhilhhlfdcuoegthhhrihhssehkohguvgehgedrnhgvtheqnecuggftrfgrthhtvghrnh epleduvddtteduhedvuefhhfehkeetlefhjeefvddtffdugfeikedtueetgeffgfehnecu vehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheptghhrhhish eskhhouggvheegrdhnvght X-ME-Proxy: Received: by mailuser.nyi.internal (Postfix, from userid 501) id 0D1731400A1; Sat, 26 Dec 2020 22:13:01 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.3.1-61-gb52c239-fm-20201210.001-gb52c2396 Mime-Version: 1.0 Message-Id: Date: Sat, 26 Dec 2020 19:12:32 -0800 From: "Christopher William Snowhill" To: linux-bluetooth@vger.kernel.org Subject: [PATCH] Fix initializing response id after clearing struct Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org 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(-) diff --git a/net/bluetooth/a2mp.c b/net/bluetooth/a2mp.c index da7fd7c8c2dc0..7a1e0b785f459 100644 --- a/net/bluetooth/a2mp.c +++ b/net/bluetooth/a2mp.c @@ -381,10 +381,11 @@ static int a2mp_getampassoc_req(struct amp_mgr *mgr, struct sk_buff *skb, hdev = hci_dev_get(req->id); if (!hdev || hdev->amp_type == AMP_TYPE_BREDR || tmp) { struct a2mp_amp_assoc_rsp rsp; - rsp.id = req->id; memset(&rsp, 0, sizeof(rsp)); + rsp.id = req->id; + if (tmp) { rsp.status = A2MP_STATUS_COLLISION_OCCURED; amp_mgr_put(tmp);