From b3053bcbb66a5835a101deb3f1b85bb567e8e632 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20G=C3=B6ttsche?= <cgzones@googlemail.com>
Date: Fri, 24 May 2024 19:59:24 +0200
Subject: [PATCH] Support CIDR address notation in nodecon statement

Supported since checkpolicy 3.7[1].

[1]: https://github.com/SELinuxProject/selinux/commit/804e52b7f8a3c8649615211a961ef8189fe73f39
---
 src/lex.l                             |  2 ++
 src/parse.y                           | 10 ++++++++++
 tests/sample_policy_files/uncommon.te |  2 ++
 3 files changed, 14 insertions(+)

diff --git a/src/lex.l b/src/lex.l
index 6d18611f..47c36d6c 100644
--- a/src/lex.l
+++ b/src/lex.l
@@ -156,7 +156,9 @@ userdebug_or_eng { return USERDEBUG_OR_ENG; }
 [a-zA-Z\$\/][a-zA-Z0-9_\$\*\/\-]* { yylval->string = xstrdup(yytext); return STRING; }
 [0-9a-zA-Z\$\/][a-zA-Z0-9_\$\*\/\-]* { yylval->string = xstrdup(yytext); return NUM_STRING; }
 [0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3} { yylval->string = xstrdup(yytext); return IPV4; }
+[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\/[0-9]{1,2} { yylval->string = xstrdup(yytext); return IPV4_CIDR; }
 ([0-9A-Fa-f]{1,4})?\:([0-9A-Fa-f\:])*\:([0-9A-Fa-f]{1,4})?(\:[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3})? { yylval->string = xstrdup(yytext); return IPV6; }
+([0-9A-Fa-f]{1,4})?\:([0-9A-Fa-f\:])*\:([0-9A-Fa-f]{1,4})?(\:[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3})?\/[0-9]{1,3} { yylval->string = xstrdup(yytext); return IPV6_CIDR; }
 \"[a-zA-Z0-9_\.\-\:~\$\[\]\/]*\" { yylval->string = xstrdup(yytext); return QUOTED_STRING; }
 \-[\-ldbcsp][ \t] { return FILE_TYPE_SPECIFIER; }
 \( { return OPEN_PAREN; }
diff --git a/src/parse.y b/src/parse.y
index 0fb9c300..03b7c566 100644
--- a/src/parse.y
+++ b/src/parse.y
@@ -86,7 +86,9 @@
 %token <string> STRING;
 %token <string> NUM_STRING;
 %token <string> IPV4;
+%token <string> IPV4_CIDR;
 %token <string> IPV6;
+%token <string> IPV6_CIDR;
 %token <string> NUMBER;
 %token <string> QUOTED_STRING;
 %token <string> VERSION_NO;
@@ -909,6 +911,8 @@ netifcon:
 
 nodecon:
 	NODECON two_ip_addrs context
+	|
+	NODECON cidr_addr context
 	;
 
 two_ip_addrs:
@@ -917,6 +921,12 @@ two_ip_addrs:
 	IPV6 IPV6 { free($1); free($2); }
 	;
 
+cidr_addr:
+	IPV4_CIDR { free($1); }
+	|
+	IPV6_CIDR { free($1); }
+	;
+
 fs_use:
 	FS_USE_TRANS STRING context SEMICOLON { free($2); }
 	|
diff --git a/tests/sample_policy_files/uncommon.te b/tests/sample_policy_files/uncommon.te
index 0131ffc9..6fb108b3 100644
--- a/tests/sample_policy_files/uncommon.te
+++ b/tests/sample_policy_files/uncommon.te
@@ -46,8 +46,10 @@ optional_policy(`
 netifcon lo gen_context(system_u:object_r:lo_netif_t,s0 - mls_systemhigh) gen_context(system_u:object_r:unlabeled_t,s0 - mls_systemhigh)
 
 nodecon 127.0.0.1 255.255.255.255 gen_context(system_u:object_r:system_t:s0)
+nodecon 127.0.0.0/24 gen_context(system_u:object_r:system_t:s0)
 nodecon ::5 ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff gen_context(system_u:object_r:system_t:s0)
 nodecon ::ffff:127.0.0.1 ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff gen_context(system_u:object_r:lo_node_t,s0)
+nodecon ::1/128 gen_context(system_u:object_r:lo_node_t,s0)
 
 if (!bool_one) {
 	allow foo_t bar_t:file open;