From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f43.google.com (mail-pj2-f43.google.com [74.125.227.171]) (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 1B82F3FF8A5 for ; Thu, 24 Sep 2026 06:22:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790230968; cv=none; b=O+2xERNKxqDU/Cft1FIyjhf+qYMXkpwmEcMbM6QhI30+nKRihhLF5fX0X3gJE81MMSXFATZW/0hsVSOdjTkEBUSwni9CUjtpCsEjSoVh2EGEqvXMoZIrdadnafKigvnJoHytjK76qqWnr5KqulgnKHi9X/OtjMvBnqvmTeAU3Bg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790230968; c=relaxed/simple; bh=PA4WHdZL97fHMVeMYVBzQqDhDDVdPEHyHKSQZGzkDpc=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=nEn549q0lgknk0lSzlBfN8lv2GxRQnQwQvpnoQ9mOZcHzk++59Y02LqtBOTYzzU8CvDFFdDKNd0qjMozbqo7QWehK85iy9VAnke/+QG11plW30rLUxIy+HDQI4I+o63T3njvDWQXDwdRyHFFaAuL8zyDDxATNc/k/pOzicziTAI= 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=J/lMUaCK; arc=none smtp.client-ip=74.125.227.171 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="J/lMUaCK" Received: by mail-pj2-f43.google.com with SMTP id d9443c01a7336-2d91ede8035so15535965ad.3 for ; Wed, 23 Sep 2026 23:22:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1790230966; x=1790835766; 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=lD/BJV6m0Rx8++xUr7O227okZULoxqZpRvzaX7SrdP4=; b=J/lMUaCKLRVqgLTwn7tCWfXPKoVqIwXhjTL3NImowkZ9JhHlyw844ac6RfSfh8HLP3 Mt3bOrZ6uZEgDpqBAMpRRoT+GwSGddOCg8wXBPWG+9ln47Ir3XPceFK9iWgYnI4sM/3O +H2jKuKGMxhzWUh8P7SF++ZPFFWYP9RUuqbxITCIZQ5W/5EzGGToADbwY+66YHqeB4Ab EfnVWHqQ1K1Ps+zYx9wxfdq9VJZYHnJDgvqJ04ehiz0g8FBUUoaBoP3AnH6AaWsBzRuH 18Pomi4cSznbemu1YMdknR3ycJz1RXxlDggZ1KziOjFiNyDo10DQepCerIFOAYF0q4sq hmxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790230966; x=1790835766; 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=lD/BJV6m0Rx8++xUr7O227okZULoxqZpRvzaX7SrdP4=; b=vhyGublZRrm+/ti2DsPFRgXp43eOY+0XAKgFSdnXofF9EesCwkVZP7ehZjvmo+MxzH tISTvUwQvKshId+DqkyZ0Z87mWwSwkZ/WEzijageTrHLy1tNeN/dGRD67KLZ5xJ8Zl/Q ygP5vEbqqZv5Nul6OpapK+ukvhyoT+ftkuTli5fwHtw1uwq0fhuewQpYLEWdhMF00VaL 1AhTxXFNoSeLIBCXDzoKFAdV/GkOq/YqCAsjDucRBsxrm8+5H8fhM1vQ2cvqIpQH5pPx rg4nmwDePZSNaizrb8s3Dq7oaVC8M9r+XAdNLnoVUiY5HB6+xCN3jGvsOrOMULNdeFbl QVyw== X-Forwarded-Encrypted: i=1; AKwUvBw1+iNWz0THiBi/1V0PYJ0Gc1xbHxcYgUkDrcJyEqSJ5peu+wVCymjMY3W+zxVODmmi+Jz2YQzpsiF4@vger.kernel.org X-Gm-Message-State: AFuF++nKNuZvruNrGc3SqDzdm8wG0QMIC7a1Q4eYU1qVpTtaY7oObiuz 63LQKjNjQTXqGGFb8aS8pLTsTw4zfdL6fM4G+0iUwtbZSQj9fuoSNzMf X-Gm-Gg: AYBFou3HxHtQjEGZE72PiNHg3i1+8jyOdPxWn/QGeMfB2ORBHLtmfX72dy9ndHq73Zf rngL9SVnPiqwcQ6yn1BuvALkAplBWCrXIm3upSP0VJjMiArfhqS38fcnmLWvXj9TJ4/o+n17AY3 uv0bv0qzJnVDQeLvJtuzw8k8GLrjJ4i8pFu+1Y3nHohrPmVQEXuN+W1G8sVfsLcBbm0BxR2M4IK RYK7Clor3NsCyrzAIrKxkriD2V4Efw3e71KU6lDdmfs4fq+NES+sFa+ltsOFU/GXQFVRrVrZuU/ cfTu2+b/w6kyIrf7GRHAbCJ+QquvCKpQTD5NGmCu77yOlrDT+kuqeyQ0ZQk6TbcHxOEMSfiXJ1z Pme3jjbACS7jL+tqiYvMUgNaCCkCznJ0HV1QxU5mZCfb0xTvCpRPvbhcg2K7thsNhmscxBvmF/Q dl7w5+pvj8XlRrGUQGkKWrhQys2z2fEZSbow/7eWDpfs6e25vQiMhZ1Mm57r+hfk6ZwVTnELr84 rBdIKhomlR/S7lx9HWP3kexM3144Z3v7kxBUs0imthHCk06DhFQxzRy58hn1YeoJ0OxB3IxDNt9 bSuoZi6lvg== X-Received: by 2002:a17:902:fc48:b0:2d9:484:861d with SMTP id d9443c01a7336-2df7e24f14cmr11867725ad.10.1790230966408; Wed, 23 Sep 2026 23:22:46 -0700 (PDT) Received: from phui-2.c.googlers.com.com (67.51.127.34.bc.googleusercontent.com. [34.127.51.67]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2df6a5d7a48sm21220615ad.48.2026.09.23.23.22.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 23 Sep 2026 23:22:45 -0700 (PDT) From: Hui Peng To: h.morris@cascoda.com, alex.aring@gmail.com, miquel.raynal@bootlin.com, stefan@datenfreihafen.org Cc: david.laight.linux@gmail.com, linux-wpan@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Hui Peng Subject: [PATCH net v3 0/3] ieee802154: ca8210: fix stack and slab out-of-bounds accesses Date: Thu, 24 Sep 2026 06:22:41 +0000 Message-ID: <20260924062244.2444225-1-benquike@gmail.com> X-Mailer: git-send-email 2.56.0.rc1.310.g51773c2048-goog Precedence: bulk X-Mailing-List: linux-wpan@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This series fixes three out-of-bounds access bugs in the Cascoda CA8210 IEEE 802.15.4 driver: 1. Reject received SPI packets with len > sizeof(struct mac_message) in ca8210_rx_done() instead of checking len > CA8210_SPI_BUF_SIZE (256), preventing a stack buffer overflow when copying a synchronous response into priv->sync_command_response (a struct mac_message on the caller's stack) and matching the actual SPI transfer length (cas_ctl->transfer.len = sizeof(struct mac_message)). 2. Initialize lenvar = 1 in ca8210_get_ed() and validate hw_attribute_length against *hw_attribute_length in hwme_get_request_sync() before memcpy() to prevent overflowing the caller's stack buffer. 3. Validate the received SPI frame length len upfront at the start of ca8210_skb_rx() before reading data_ind or allocating the skb. Changes in v3: - Patch 1/3: Check len > sizeof(struct mac_message) in ca8210_rx_done() where dev_crit() logs "Received packet len (%u) erroneously long" and drops the packet, instead of silently truncating the memcpy() with min_t(), addressing David Laight's feedback. Changes in v2: - Split the ca8210 fixes into three single-issue patches (1/3..3/3) and addressed Miquel Raynal's review comments on patches 2/3 and 3/3. Hui Peng (3): ieee802154: ca8210: prevent stack buffer overflow in ca8210_rx_done() ieee802154: ca8210: prevent stack buffer overflow in hwme_get_request_sync() ieee802154: ca8210: validate data_ind length upfront in ca8210_skb_rx() drivers/net/ieee802154/ca8210.c | 39 +++++++++++++++++++++++---------- 1 file changed, 27 insertions(+), 12 deletions(-) -- 2.49.0