public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
* [tests/selinux] pr775-checkpolicy-revdeps: test if sepolgen-ifgen complains about illegal character
@ 2026-09-11 13:21 Milos Malik
  0 siblings, 0 replies; only message in thread
From: Milos Malik @ 2026-09-11 13:21 UTC (permalink / raw)
  To: git-commits

            A new commit has been pushed.

            Repo   : tests/selinux
            Branch : pr775-checkpolicy-revdeps
            Commit : 55c2934207b077ed907a65b424ac99fab988d56f
            Author : Milos Malik <mmalik@redhat.com>
            Date   : 2024-05-06T08:54:27+00:00
            Stats  : +12/-0 in 2 file(s)
            URL    : https://src.fedoraproject.org/tests/selinux/c/55c2934207b077ed907a65b424ac99fab988d56f?branch=pr775-checkpolicy-revdeps

            Log:
            test if sepolgen-ifgen complains about illegal character

Several bug reports pointed out that the installation, reinstallation
or upgrade of the selinux-policy-devel package produces the following
error message:

  Illegal character '"'

Further investigation revealed that there are unexpected/redundant
parentheses in the /usr/share/selinux/devel/include/contrib/virt.if
file, which also cause the following error messages when the
sepolgen-ifgen command is executed:

  Missing interface definition for virt_pid_filetrans
  Missing interface definition for virt_systemctl

The TC is able to reproduces the problem when executed on a not yet
fixed package. The TC passes when the problem is fixed.

The TC covers BZ#2254169, BZ#2254206, BZ#2277925.

---
diff --git a/selinux-policy/interface-definitions/main.fmf b/selinux-policy/interface-definitions/main.fmf
index e54b879..8befb8d 100644
--- a/selinux-policy/interface-definitions/main.fmf
+++ b/selinux-policy/interface-definitions/main.fmf
@@ -6,6 +6,7 @@ component:
 recommend:
   - /usr/bin/sepolgen-ifgen
   - selinux-policy
+  - selinux-policy-devel
   - selinux-policy-targeted
 test: ./runtest.sh
 framework: beakerlib
@@ -19,6 +20,10 @@ tag:
 link:
   - verifies: https://issues.redhat.com/browse/RHEL-2616
   - verifies: https://issues.redhat.com/browse/RHEL-16185
+  - verifies: https://bugzilla.redhat.com/show_bug.cgi?id=2254169
+  - verifies: https://bugzilla.redhat.com/show_bug.cgi?id=2254206
+  - verifies: https://bugzilla.redhat.com/show_bug.cgi?id=2277925
+  - verifies: https://issues.redhat.com/browse/RHEL-34769
 adjust:
   - enabled: false
     when: distro == rhel-4, rhel-5

diff --git a/selinux-policy/interface-definitions/runtest.sh b/selinux-policy/interface-definitions/runtest.sh
index 6d80041..215404a 100755
--- a/selinux-policy/interface-definitions/runtest.sh
+++ b/selinux-policy/interface-definitions/runtest.sh
@@ -13,6 +13,13 @@ rlJournalStart
         rlRun "grep -i missing output.txt" 1
     rlPhaseEnd
 
+    rlPhaseStartTest "bz#2254169 + bz#2254206 + bz#2277925"
+        rlRun "yum -y reinstall selinux-policy-devel >& output.txt"
+        rlRun "grep -i illegal output.txt" 1
+        rlRun "sepolgen-ifgen --verbose >& output.txt"
+        rlRun "grep -i illegal output.txt" 1
+    rlPhaseEnd
+
     rlPhaseStartCleanup
         rm -f output.txt
     rlPhaseEnd

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2026-09-11 13:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-11 13:21 [tests/selinux] pr775-checkpolicy-revdeps: test if sepolgen-ifgen complains about illegal character Milos Malik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox