From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dy1-f179.google.com (mail-dy1-f179.google.com [74.125.82.179]) (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 C635F2F0C74 for ; Sun, 24 May 2026 04:15:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779596109; cv=none; b=lmz8xVaq0YPFsNpC/hUZ9X2KhObAIawH5ZkU4nRnyvIav0Mk+YkAqzkUio1e3MYsAxe9Tk1KTSWPJ+tFwDNM1rcrbWxPDdgeLGP9v35NP76RQofsPtdTh6fPoS3eyvHwKj/mtulTGtIPQ/Cnhtn816izj0W3Mb0UJ1OGxih/SjM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779596109; c=relaxed/simple; bh=zlROHpspN4svB9VHuRgW/RNOHKabtQok9h5ytT2jjOk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=jdRc213CRWCxEHJ/P9Skph/Kvuo/om23l0huy3zfvP65oCcC+JOJpWBQcLZ9c6X1D85Jsf91XklzHZciyBWSOOvN+tcEP3UNr3cKxoTwg+1csdJSoBbYUfb5MkBO0oU60RTCBrcIKoF0cUsepcKJWfU7CdohBeg345DW9Miu8jA= 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=DXklRN6E; arc=none smtp.client-ip=74.125.82.179 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="DXklRN6E" Received: by mail-dy1-f179.google.com with SMTP id 5a478bee46e88-30246cfd41aso3872582eec.1 for ; Sat, 23 May 2026 21:15:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779596107; x=1780200907; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=Y7xIC92IytMwCoIpp3F6gfBN1+ahv97EwLManb72nZc=; b=DXklRN6Ero41d1ZWvZEgMh+V2QyLzI0NY6IjduXNOqGEDbVDKqx1p48d1GBRvnNhGk JRoPkGexU4TxAjJgtQDCXf7m2zKKaCydXLdNiItRGpYnBCrzmJyJSZt/JFOI8OQlKH2p JNwHzu+9+iJXGpdZQGn0mA10LWLCU6k4FvCw+kMHLKR4P4HJSBmsOqyE17HgiROdei/m WnP7t6EJdd1194jGABhu4YWoyIeggXuQUrLtmrnF7gRXxvpGujWD4sAm1Uzts9V8c+5L LFLaIQ0xrxCgKjF9FszNz4MReVjoEVqmvXrCDnG2HooW5JAL8zX4+hXtl5X7YNNwSVda +g6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779596107; x=1780200907; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=Y7xIC92IytMwCoIpp3F6gfBN1+ahv97EwLManb72nZc=; b=fEjzZ4gqgNNWnINaRXBpSPPVgJa5Xl5CWVmBKMicb0J1ttXcl5fdm8ermPGKW2u//X 3DbkFpwEbUk+gZruq9x4kRYXGuRNy6ZuuKkByI5+O0tmlyauPKdUElLi5Dww4SnDrfJd H8d6pFBZoVZD1XNqrVYy0DeGsGNWm02q7QYr3qh7587awf3nrZ4lF4dTRfcoQP/wIgbN g71zIxc8MpMpCa8vePj56STJmnDUXZfm9ap9GL3eHMWZxDO/bQWq8/BrsPQqND/s1yn8 H0r6XYJ9oDcI+9IkQeOoLxmPwXYu77ZMYXl1Ufkw5y3JzheRjlVU1+9BGRGSDZbt/IYV Y5Jg== X-Forwarded-Encrypted: i=1; AFNElJ/TLqbw1bFfj77tkbD1Vc/tlVTnekOQeVd0K10CScWzhn3VvKyqr6PugYZFw2bbhAWdzHkjjGM=@vger.kernel.org X-Gm-Message-State: AOJu0YzUPqRLWmklFvsz6rn7ewj0nxZqTyzfxGO3eO5AUr90z3UmYQlu k6wGvJKkyar21Pwdkey2r1JbWNX6M+FeyGmhQbkNJtyxTggYFW8zoXdE X-Gm-Gg: Acq92OFEAgSmWuyGV9bcOEaSOw2gSg0yT4r2aluHfcOEKP0vRzGeNqZ5HU0Q/Z3CLLY 1qLvXNGSKikq0jbFjEOukSO3rl45TseG5BggxRJtuX8Og33Tqk+QKVWILaYneDaTL/oq3iECE15 Tpswuz5wKrR1REnovlMtb0tl8DWf2VRmom7kRHUk5NrVzyOVwnmI1CLhKYmdDcMF4HzSJv6Carh gvb0ahgTjPFx67vecvRJ1WrQRyZPDptiAK62Tewj2pFQ/xDDOlIY1zsgEVAbjBxF77q7AM3ZnwY Nohtve+MHuZUZ8C2Dd01/TUKbrZchZzAr4wjtk00fYkVbCIfa47YknjRQFB3nNwkDXpHo3sEeYR PI4RzYQjfXdl8r1DzSRNTYH+hNXW6UcH9/4uVmfwJ9hb9JchH0tA3KrnGLfj2Rc72k4FtzdV2ai SH1Xq0rjfDEtlYfdfSGh9lggQmLkG8sO2Tiw== X-Received: by 2002:a05:7301:19a5:b0:2df:919f:ce59 with SMTP id 5a478bee46e88-30449149c79mr5024715eec.19.1779596106994; Sat, 23 May 2026 21:15:06 -0700 (PDT) Received: from localhost.localdomain ([148.135.103.3]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3045245d6aesm4522133eec.26.2026.05.23.21.15.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 23 May 2026 21:15:06 -0700 (PDT) From: Qi Tang To: davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com, edumazet@google.com Cc: netdev@vger.kernel.org, fw@strlen.de, lyutoon@gmail.com, stable@vger.kernel.org, Qi Tang , Paul Moore , Simon Horman , Huw Davies , linux-security-module@vger.kernel.org Subject: [PATCH net v2 3/4] netlabel: validate CALIPSO option against skb tail in netlbl_skbuff_getattr Date: Sun, 24 May 2026 12:14:37 +0800 Message-ID: <20260524041442.2432071-4-tpluszz77@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260524041442.2432071-1-tpluszz77@gmail.com> References: <20260524041442.2432071-1-tpluszz77@gmail.com> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit netlbl_skbuff_getattr() locates the CALIPSO option in the IPv6 HBH header via calipso_optptr() and hands the bare pointer to calipso_getattr() -> calipso_opt_getattr(). The consumer re-reads calipso[1] (option data length) and calipso[6] (cat_len/4) and walks calipso + 10 for cat_len bytes via netlbl_bitmap_walk(). ipv6_hop_calipso() validates these bytes only at parse time inside ipv6_parse_hopopts(). An nftables PRE_ROUTING payload write reachable from an unprivileged user namespace can rewrite both bytes between parse and the SELinux peer-label consume path (selinux_sock_rcv_skb_compat -> selinux_netlbl_sock_rcv_skb -> netlbl_skbuff_getattr). The self-consistency check (cat_len + 8 > len) inside calipso_opt_getattr() is defeated by mutating both bytes consistently, allowing a ~232-byte slab-out-of-bounds read from calipso + 10 whose set bits become MLS categories driving the access decision. netlbl_skbuff_getattr() has the skb; gate the consume on the option fitting within skb_tail_pointer(). The IPv6 option layout is type(1) + length(1) + length bytes of data, so requiring ptr + 2 + ptr[1] <= skb_tail covers the option and its embedded bitmap. When the bounds check fails the packet has been mutated after parse, so return -EINVAL rather than fall through to the unlabeled path. Runtime confirmation (SELinux compat path with selinux=1 enforcing=0 and a CALIPSO DOI added via netlabelctl): Udp6InDatagrams increments to 1 with the mutated cat_len, showing selinux_socket_sock_rcv_skb -> netlbl_skbuff_getattr -> calipso_opt_getattr -> netlbl_bitmap_walk runs end-to-end past the option's true bound; with this patch the consume path returns -EINVAL at the bounds check and the counter stays 0. Cc: stable@vger.kernel.org Reported-by: Qi Tang Reported-by: Tong Liu Fixes: 2917f57b6bc1 ("calipso: Allow the lsm to label the skbuff directly.") Signed-off-by: Qi Tang --- net/netlabel/netlabel_kapi.c | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/net/netlabel/netlabel_kapi.c b/net/netlabel/netlabel_kapi.c index 3583fa63dd01f..d0d6220b8d59d 100644 --- a/net/netlabel/netlabel_kapi.c +++ b/net/netlabel/netlabel_kapi.c @@ -1399,11 +1399,22 @@ int netlbl_skbuff_getattr(const struct sk_buff *skb, return 0; break; #if IS_ENABLED(CONFIG_IPV6) - case AF_INET6: + case AF_INET6: { + const unsigned char *tail = skb_tail_pointer(skb); + u8 opt_data_len; + ptr = calipso_optptr(skb); - if (ptr && calipso_getattr(ptr, secattr) == 0) + if (!ptr) + break; + if (ptr + 2 > tail) + return -EINVAL; + opt_data_len = ptr[1]; /* IPv6 option data length */ + if (ptr + 2 + opt_data_len > tail) + return -EINVAL; + if (calipso_getattr(ptr, secattr) == 0) return 0; break; + } #endif /* IPv6 */ } -- 2.47.3