diff options
| author | lidl <lidl@FreeBSD.org> | 2015-11-05 17:37:14 +0000 |
|---|---|---|
| committer | lidl <lidl@FreeBSD.org> | 2015-11-05 17:37:14 +0000 |
| commit | 47674eb3ca595f821b75939cad1928bd470e94dc (patch) | |
| tree | 75c924b6bbae038bfc974cfa12f78c568ed05696 | |
| parent | 3a7badaac7812e351eac63f3e6b5cddc6ae55496 (diff) | |
| download | FreeBSD-src-47674eb3ca595f821b75939cad1928bd470e94dc.zip FreeBSD-src-47674eb3ca595f821b75939cad1928bd470e94dc.tar.gz | |
Restrict 520.pfdenied to only list rules that blocked traffic.
Before this change, the 520.pfdenied script listed all rules that
matched /^block/ in the rule. Restrict the printed output to only
those rules that result in packets being dropped.
PR: conf/187224
Approved by: rpaulo (mentor)
Differential Revision: https://reviews.freebsd.org/D4068
| -rwxr-xr-x | etc/periodic/security/520.pfdenied | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/etc/periodic/security/520.pfdenied b/etc/periodic/security/520.pfdenied index 7a32bf2..3fea360 100755 --- a/etc/periodic/security/520.pfdenied +++ b/etc/periodic/security/520.pfdenied @@ -44,7 +44,7 @@ rc=0 if check_yesno_period security_status_pfdenied_enable then TMP=`mktemp -t security` - if pfctl -sr -v 2>/dev/null | nawk '{if (/^block/) {buf=$0; getline; gsub(" +"," ",$0); print buf$0;} }' > ${TMP}; then + if pfctl -sr -v 2>/dev/null | nawk '{if (/^block/) {buf=$0; getline; gsub(" +"," ",$0); if ($5 > 0) print buf$0;} }' > ${TMP}; then check_diff new_only pf ${TMP} "${host} pf denied packets:" fi rc=$? |
