From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=209.85.128.44 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793826551; b=jYQ0ngr4Vfe9TtHhvGLkySWCdvHoiw7d1aR3Pn8GY5JAXvAyzXYT2e4JHcqO7o1BP+eG PWInYrUc7dt02cBh1I9C8z30i7+FQasFqf0W0YvqItVgqCcnNeXvcVJctkVjUyAHj8QsI KhYbZvcrXOL17F0C/qcL/BQ8RtiaEtXLYICbW1xdt0sFuJ+S5xBoI7Ub5C6I6N/ektC2Q mHordD2zcIMVhDHo21x0xv5uqlFKMfMact64BTowJWYNwyWVS+jsLnVwkNeXi5u9kYiay RvPO1TGibSkbkNvoHg2Ao08Sd3MnX6iMSTg7Ls7OLvElIB/RjCwCwbFxn+yuy9uNwUA== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793826551; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Gm-Gg:X-Received:Received: From:Date:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding: Message-Id:References:In-Reply-To:To:Cc:X-Mailer; bh=729LbPMPastRfQlfVEWKWxLCR7JpRj6YVk1X4d7YWEo=; b=Y/A4UBmg1u4uLbAqyLKjuG96+QX6CBv1bSkG8gQ3KdvRscHjtOqIkILrM92veIalrSfG KNnB+snRQFCkXKbM3CsxG9qB0GBgNY+yqpHnW3AQEfjhC7FM/40zhVHrGT6FrT0JA7zSE Fl79O6PnfD1R2fzlDuZf/W2ZZ9UF/dz8pQf0mhLw/cZkErznMqF3/P/shla4SVkVj+Wmd j1KecJzd3qFkhCZ0HA8ObyWaNGETLcltnxtZ8NgNOY1mNUEC0qjA6oEtTwepYvHPBHqw4 3BA6eAEaCiJ+gBn/+SQA2aksWm0lMA31/E0qkdzQ4VeZUDCEFy4krZASIAtGQQK6asQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20260707 header.b="qW4/6f9Q"; arc=none smtp.remote-ip=209.85.128.44 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9CAA360A2614 for ; Wed, 4 Nov 2026 21:09:10 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 9CAA360A2614 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, unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20260707 header.b=qW4/6f9Q Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-4a4d7ae1ae9so11975635e9.1 for ; Wed, 04 Nov 2026 13:09:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20260707; t=1793826549; x=1794431349; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=729LbPMPastRfQlfVEWKWxLCR7JpRj6YVk1X4d7YWEo=; b=qW4/6f9QMYYNLzuX+9fq2z2jVkf1UM/thXabt8pajkTNPSX43bpePlTtAm2TfGRLbJ HcJt1RKD1b83ThkRoRG6R0VlGkC3nfTnehIAhBWA9bm5IrjeUzYvmd3wKTkyY1/yr3IS B5kNYDdWaxAtEsdwVifmNbyEAEPeRtq3OqUSzCK3QV8HrGPy00ZH7vq3h6mEOjndJ7Yj rLoam+HoGGbg8y1DVaQtdqQgwulTmfWEPFbfrvv047DnKKOScesliWUq/7qeVTuOs/Qk FmyzarFvGeu0qV1VNKXMPAGGDVZP7j5m+BPMW+dI4akrhRQkN1Rqf35mNt+DCuLwLjqt w5Wg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1793826549; x=1794431349; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=729LbPMPastRfQlfVEWKWxLCR7JpRj6YVk1X4d7YWEo=; b=CHs1p8/FcwubKLMHmTtZ59BciLbikUe9gFaTOasXDRARfnkICJwjvkqheFB7egaHHC 9zEORmVNDcVFejmeFBPS4hcT9lizLTKI0HO/a8w/ce2Cl3MS2PyUlRsmWBKYH0fizDT7 Aqr+OxXiIgkKMEtKoVUqMLGxptlxyYDi+0ymOhiApdAaVtEkbSoG/1YeShge9wdKJmxm VJ27eXvbq3qd8pB3ERjpZRUUIrkaL36nexuAPwdnNbA8af7xgZ6i6j7F1omTySgPLc26 y7CEGb5wwumEcWmahqMKsnND1xlBjavbjshsuji5lUK6uIg+q7T3Vh9KC/GcsZ23nSaB 97tw== X-Forwarded-Encrypted: i=1; AKwUvBydpwbLOChQwKCEe2SuehhD/krKT9shu4rjY3r+HwvML2ZK2MyEmwnzi1/HXXkLGEQVUrnKWyzGaCuJgcg=@vger.kernel.org X-Gm-Message-State: APuU4Z9LKJ17yuMvK0pQu8guORgcxvWZEWTIIdnTPRE+Kj+5MJbxH0jb RlDT6r7Gxq5o4bVZtEv71q7JWbYH2FG2wRzBSNV5KB9xAkz7/AXioARx X-Gm-Gg: AYBFou0esfYir98oXjomMPLFdFTFAPcSA/j6TBrD9542y6un+sfgWnpnKzKwJ67UL07 4bXlAMP4zYEfU+u5F7bW2LW9G7d9P1pW2HiikMPLotPUoRVy/e3oSZ/r6mMziDgmxUqVT/HQK5K jnce7b8otNcQVNmOU1rfjZR86vxRns1vBZ/O4kb0EaLjiNr1//oqGdOXU4w8rLv44oI8lqV6+Cj 5I0pxSnNQyZplMi2k0oUVDT0ulNEf3R7BDu1p68cXUjeXKyaPIPavVJofU53jYoxP8LiPlz0RGY Tlm2iS4jDVTlJA2w3Kk50Oob180np1sps7ZsbLgHMtayY3CzquQESeo6959akvoAdGJxhOQPG9S +t4LXq2UZpTkHa9zsw9OJrlenG5CORvqgNKEbhSvVMgh99xvktZyHU3z0peNzHBwKLEFtEeJP8R rPig6xUG69vHIZqb5J4PEXfMjZU+MQrk5ErWmbG6Ekq3vvGaKT9ugvhrYccJQeH5CZp1h9IYKD9 t7tAcZVRB9wmO4yjmdH/QWjuvjoPoiiq7OybUkFg/mztyi3sPwTsBoPei3lPub0YWIUgvoGWSXB KBFLTLQvz+3FbRGBpGjb8sp+0fG1r2NaHhdQP4Lb2qd1pQ2AwX6AGg== X-Received: by 2002:a05:600c:19c8:b0:4a3:59c9:a743 with SMTP id 5b1f17b1804b1-4ac1913ef8fmr402468605e9.27.1793826549010; Wed, 04 Nov 2026 13:09:09 -0800 (PST) Received: from localhost (ip87-106-108-193.pbiaas.com. [87.106.108.193]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4b56e58fcc6sm20061955e9.2.2026.11.04.13.09.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 04 Nov 2026 13:09:08 -0800 (PST) From: =?utf-8?q?G=C3=BCnther_Noack?= Date: Wed, 04 Nov 2026 22:08:55 +0100 Subject: [PATCH v2 09/27] selftests/landlock: Assert tracefs_extract_field() results that are used X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Message-Id: <20261104-selftests-refactor-v2-9-c52f23f7ce33@gmail.com> References: <20261104-selftests-refactor-v2-0-c52f23f7ce33@gmail.com> In-Reply-To: <20261104-selftests-refactor-v2-0-c52f23f7ce33@gmail.com> To: =?utf-8?q?Micka=C3=ABl_Sala=C3=BCn?= , =?utf-8?q?G=C3=BCnther_Noack?= , Shuah Khan Cc: linux-security-module@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, =?utf-8?q?G=C3=BCnther_Noack?= X-Mailer: b4 0.15.2 Several trace tests check tracefs_extract_field() with EXPECT_EQ() and then compare the extracted field. If the extraction fails, the field buffer is left uninitialized and the following comparison reads it. Use ASSERT_EQ() for these calls, as the other trace tests already do. Signed-off-by: Günther Noack --- tools/testing/selftests/landlock/trace_test.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/tools/testing/selftests/landlock/trace_test.c b/tools/testing/selftests/landlock/trace_test.c index cb49ce8dbeda..1795d41b779f 100644 --- a/tools/testing/selftests/landlock/trace_test.c +++ b/tools/testing/selftests/landlock/trace_test.c @@ -175,25 +175,25 @@ TEST_F(trace, create_ruleset) } /* Verify handled_fs matches what we requested. */ - EXPECT_EQ(0, + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_RULESET(TRACE_TASK), "handled_fs", field, sizeof(field))); EXPECT_STREQ("read_file", field); /* Verify handled_net matches. */ - EXPECT_EQ(0, + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_RULESET(TRACE_TASK), "handled_net", field, sizeof(field))); EXPECT_STREQ("bind_tcp", field); /* Verify that no permission is handled. */ - EXPECT_EQ(0, tracefs_extract_field( + ASSERT_EQ(0, tracefs_extract_field( buf, REGEX_CREATE_RULESET(TRACE_TASK), "handled_permissions", field, sizeof(field))); EXPECT_STREQ("", field); /* Verify version is 0 at creation (no rules added yet). */ - EXPECT_EQ(0, + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_RULESET(TRACE_TASK), "ruleset", field, sizeof(field))); /* Format is .; version is after the dot. */ @@ -363,7 +363,7 @@ TEST_F(trace, create_domain) } /* Extract the domain ID from create_domain. */ - EXPECT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_DOMAIN(TRACE_TASK), + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_DOMAIN(TRACE_TASK), "domain", domain_id, sizeof(domain_id))); @@ -371,7 +371,7 @@ TEST_F(trace, create_domain) EXPECT_NE(0, strcmp(domain_id, "0")); /* Verify parent=0 (first restriction, no prior domain). */ - EXPECT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_DOMAIN(TRACE_TASK), + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CREATE_DOMAIN(TRACE_TASK), "parent", parent_id, sizeof(parent_id))); EXPECT_STREQ("0", parent_id); @@ -387,7 +387,7 @@ TEST_F(trace, create_domain) TH_LOG("Expected check_rule_inode events\n%s", buf); } - EXPECT_EQ(0, tracefs_extract_field(buf, REGEX_CHECK_RULE_FS(TRACE_TASK), + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_CHECK_RULE_FS(TRACE_TASK), "domain", check_domain, sizeof(check_domain))); EXPECT_STREQ(domain_id, check_domain); @@ -621,7 +621,7 @@ TEST_F(trace, free_domain) } /* Verify denials count matches the single denial we triggered. */ - EXPECT_EQ(0, tracefs_extract_field(buf, REGEX_FREE_DOMAIN(KWORKER_TASK), + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_FREE_DOMAIN(KWORKER_TASK), "denials", denials_field, sizeof(denials_field))); EXPECT_STREQ("1", denials_field); @@ -906,7 +906,7 @@ TEST_F(trace, non_audit_visible_denial_counting) { TH_LOG("free_domain event not found after 10 retries"); } - EXPECT_EQ(0, tracefs_extract_field(buf, REGEX_FREE_DOMAIN(KWORKER_TASK), + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_FREE_DOMAIN(KWORKER_TASK), "denials", denials_field, sizeof(denials_field))); EXPECT_STREQ("1", denials_field); @@ -955,7 +955,7 @@ TEST_F(trace, add_rule_net_port_fields) * (landlock_net_port_attr.port). On little-endian, htons(8080) is * 36895, so this comparison catches byte-order bugs. */ - EXPECT_EQ(0, tracefs_extract_field(buf, + ASSERT_EQ(0, tracefs_extract_field(buf, REGEX_ADD_RULE_NET_PORT(TRACE_TASK), "port", field, sizeof(field))); EXPECT_STREQ("8080", field); @@ -965,7 +965,7 @@ TEST_F(trace, add_rule_net_port_fields) * net access bits are unhandled because the ruleset only handles * BIND_TCP). */ - EXPECT_EQ(0, tracefs_extract_field( + ASSERT_EQ(0, tracefs_extract_field( buf, REGEX_ADD_RULE_NET_PORT(TRACE_TASK), "access_rights", field, sizeof(field))); EXPECT_STREQ("bind_tcp|connect_tcp|bind_udp|connect_send_udp", field); -- 2.56.0