From 5ed76afa814812edaeaff2ea7b3227c18d5de5a6 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Tue, 4 Aug 2020 12:29:59 +0200 Subject: pkg/compiler: check for unused resources If a resource is never used as an input, it is not useful. It's effectively the same as using an integer. Detect such cases, they are quite confusing. Fix all existing errors in descriptions. This uncovered some interesting bugs as well, e.g. use of a completely unrelated fd subtype after copy-paste (while the resource that was supposed to be used there is completely unused). --- sys/openbsd/dev_pf.txt | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'sys/openbsd/dev_pf.txt') diff --git a/sys/openbsd/dev_pf.txt b/sys/openbsd/dev_pf.txt index 93ce99b06..c07e9d95d 100644 --- a/sys/openbsd/dev_pf.txt +++ b/sys/openbsd/dev_pf.txt @@ -10,9 +10,6 @@ include include # The following ioctl commands are restricted in neutralize(), see init.go. -_ = DIOCCLRSTATES -_ = DIOCKILLSTATES +_ = DIOCCLRSTATES, DIOCKILLSTATES -resource fd_pf[fd] - -openat$pf(fd const[AT_FDCWD], file ptr[in, string["/dev/pf"]], flags flags[open_flags], mode const[0]) fd_pf +openat$pf(fd const[AT_FDCWD], file ptr[in, string["/dev/pf"]], flags flags[open_flags], mode const[0]) fd -- cgit mrf-deployment