From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ua2-f12.google.com (mail-ua2-f12.google.com [74.125.226.204]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 124994C7550 for ; Tue, 15 Sep 2026 16:02:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.226.204 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789488175; cv=none; b=ig5vJXHOS7SMr4zFO7cRreFxDC1ilbzDhoVt/RdUbwMZu4heq7WVk9S4T1KI2kl5xcLYE52zLSKS4SfJDEjKQ8ml0hHNBzdeWq6fxZrr9KwRgS+11AT4PNZHtmzZUmG8rZqLa2FDgYXtSbRGTS0lordCvp19+xi00M5mYFkMPUU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789488175; c=relaxed/simple; bh=9UwYC4oAOD0QyUOBHTahfXva8dPMW5L+Y0G4I2GbhK8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=XyerHQYADqZyBtuxwe7balSiE0cqKSGHECncbhKAOw8c7MaiQuOpakux533taT/yzkHzlY9O9iR/bRdgJiAXhAOYh4oyv8/5w3dGxvF2CrmpOx3qChm10k4tNJiwtFuG7coNA3rjKD7xenXihmzo9yyDRnXbZbiyqAMRh30c8y0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=lunoOwBz; arc=none smtp.client-ip=74.125.226.204 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="lunoOwBz" Received: by mail-ua2-f12.google.com with SMTP id a1e0cc1a2514c-982db8d3ab8so156798241.1 for ; Tue, 15 Sep 2026 09:02:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789488172; x=1790092972; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=G2OyX4TBZ7Y1HQ2aZjbOcGyMSBVmU4uZJiE97DYV0Rw=; b=lunoOwBzgY2LcIWA3JpVQLZ49XKm3odL0hFDJ454Iqf3vnGbCOgklpIeTl4gJvPjel OJzx0Kk6CfE5Z6SplGUXnrDngkDgHjgpeJw+CPTIcdNzZCmYxdPM+HKp+LmRbG02M8C8 zXJVMtRHed50mwAZjOL4qWbk/CImNvCqZcf4QBVapV8uXdi44JOtS+reZyOXu4BS3i+L CdhAOgQ5JNEr7xFc2u5AwVhjC4rfPfUU01sVCUr5qBGrotTX6SvxijmxOiQdKUkRytTw qlWXq+z2S2Jb+P7ajktbOCUTnWG2Z3RMuL+q5rkz70o3HZUExw7t6tEpuTVILALjG0UD 2Aog== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1789488172; x=1790092972; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=G2OyX4TBZ7Y1HQ2aZjbOcGyMSBVmU4uZJiE97DYV0Rw=; b=2G8QMItClxyZROWb2P4GC5jaebwCHJCouACv1dVkZOAy6xcMu7NwXHGjQmDSAZu6/8 5YdLbvWkaq9dRm9JoRADvC68wUhksPha6mYr8F2STcyNX9LBnfWoVQJqNeZc2Ew9jnJa xwqnMmUOV5KT7XeZG4xGjPmO5MlEo9gRrfuHWc9eDWhzLDBPq6i7yzdPWD1kCy/D3Tfr LbJ5et62OkjXNlzaw50kcrfBOv6uAxn6vHcqCG3gBEVvcIDJMxVyMvvLPaQ+s5uat6GJ uBKIEGhxM55CP+SH+yjsckAsohAffnTcetuwXBXE72lM7YqRZlAj3pCKhEkzbDVkqKR7 L5tA== X-Forwarded-Encrypted: i=1; AKwUvByBQ+hGJd0bDcsk7gaOpbAFPOK5PjxTZY+UQPT9sgsbYiNk3u02tTK7hFy5QaMgyOalvnjwXVk=@vger.kernel.org X-Gm-Message-State: AFuF++lfAGspBBM2n9DA6fyhmzxp4xsmJThiZnVgHYHQyGXa7JaCyCJR /To/vnI16/afm9DfTmqD/wwc7WTGkenGnpJ7tgNJ1pm+Nnbhbn9p59C2 X-Gm-Gg: AYBFou1kf0/FHfMOgVHWM7j8k6TYVy7Ak9ByQcgleegvDcxJCy/VmgVhLMtEkmFk6bz 1+uUNp91rK55gD6a4PKMA6xzK6SbnaGhZjxgAgCUPoaqm5iqUZ+W/UN0t3mHc6lRORFIVPInvKe 1ba+/3di8hMp7TgB76nuHfDNqd6U3UKInBRNf0hOB2AiTFZN6DFsNcnrFvVigg2M+LQW42/aIDX fty+V956nDNFqq6dpn6mhNhG6D02KNAnwiuAUz5x2rHPKaO7/SAlhea6uH/hamkRb+6baRIWMS8 D7q7HN8ChtIwcBv16Wm+KLVqbLTkqfsqKJ9s0NBsd8xk/Y+wCfX2wA1Xz+5srxu+5a/SepgLPyE 5pEavwu8q0H+TwyPjT2MsDjn2vPVkXZ6jaMj/yg3e0oXFwoXZRiy+3NgJMuTWroIF7YFjrGmUCo aXsxbRJv3Vv3Y+rS2ffob5kGbRxKEwgLiuQR4nbtSyxWspSU98Vfwb+0G8rD+vvCrm X-Received: by 2002:a05:6102:38c6:b0:79e:373e:8216 with SMTP id ada2fe7eead31-79ecdbfe5f3mr3192373137.0.1789488171648; Tue, 15 Sep 2026 09:02:51 -0700 (PDT) Received: from beelink.. ([187.13.206.89]) by smtp.gmail.com with ESMTPSA id ada2fe7eead31-79fb193e849sm78576137.8.2026.09.15.09.02.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 15 Sep 2026 09:02:51 -0700 (PDT) From: Aldo Ariel Panzardo To: Luiz Augusto von Dentz Cc: linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Aldo Ariel Panzardo Subject: [PATCH] Bluetooth: L2CAP: validate frame length before control and FCS access Date: Tue, 15 Sep 2026 13:02:39 -0300 Message-ID: <20260915160239.3106697-1-qwe.aldo@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit l2cap_data_rcv() unpacks either a two-byte or four-byte control field without first ensuring that it is present. A short ERTM or streaming-mode frame can therefore cause an out-of-bounds read. There is a second short-frame case when CRC16 is enabled. After the control field is pulled, l2cap_check_fcs() subtracts two from skb->len without checking it. If fewer than two bytes remain, the subtraction wraps; skb_trim() leaves the buffer unchanged and the subsequent FCS load reads past the logical end of the frame. Validate that the frame contains both its control field and, when enabled, its FCS before either field is accessed. Fixes: 1c2acffb76d4 ("Bluetooth: Add initial support for ERTM packets transfers") Fixes: fcc203c30d72 ("Bluetooth: Add support for FCS option to L2CAP") Cc: stable@vger.kernel.org Signed-off-by: Aldo Ariel Panzardo --- net/bluetooth/l2cap_core.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 1156aba4e..40ebabe34 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c @@ -6683,9 +6683,17 @@ static int l2cap_stream_rx(struct l2cap_chan *chan, struct l2cap_ctrl *control, static int l2cap_data_rcv(struct l2cap_chan *chan, struct sk_buff *skb) { struct l2cap_ctrl *control = &bt_cb(skb)->l2cap; - u16 len; + u16 len, min_len; u8 event; + min_len = test_bit(FLAG_EXT_CTRL, &chan->flags) ? + L2CAP_EXT_CTRL_SIZE : L2CAP_ENH_CTRL_SIZE; + if (chan->fcs == L2CAP_FCS_CRC16) + min_len += L2CAP_FCS_SIZE; + + if (skb->len < min_len) + goto drop; + __unpack_control(chan, skb); len = skb->len; -- 2.43.0