On Sun, Dec 26, 2010 at 10:35:02PM +0100, Thomas Pfaff wrote:
> On Tue, 21 Dec 2010 22:15:33 +0100
> Thomas Pfaff <[email protected]> wrote:
>
> > 1) pf(4) says DIOCSETDEBUG has enum { PF_DEBUG_NONE, PF_DEBUG_URGENT, ...
> > but these names are not in pfvar.h nor anywhere else in the source tree
> > (AFAICT). What should the legal values (or names) be?
>
> I guess these should now be one of the LOG_ names from syslog(3) so
> maybe something like this?
>
> Not all LOG_ levels are used in the code, as far as I can tell, but I
> guess that does not matter and might change in the future (so just do
> a man page reference rather than list the currently used levels).
>
fixed, thanks.
jmc
> Index: pf.4
> ===================================================================
> RCS file: /cvs/src/share/man/man4/pf.4,v
> retrieving revision 1.71
> diff -u -p -r1.71 pf.4
> --- pf.4 31 May 2010 18:33:54 -0000 1.71
> +++ pf.4 26 Dec 2010 21:30:29 -0000
> @@ -26,7 +26,7 @@
> .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
> .\" SUCH DAMAGE.
> .\"
> -.Dd $Mdocdate: May 31 2010 $
> +.Dd $Mdocdate: March 25 2010 $
> .Dt PF 4
> .Os
> .Sh NAME
> @@ -316,10 +316,9 @@ struct pfioc_natlook {
> .Ed
> .It Dv DIOCSETDEBUG Fa "u_int32_t *level"
> Set the debug level.
> -.Bd -literal
> -enum { PF_DEBUG_NONE, PF_DEBUG_URGENT, PF_DEBUG_MISC,
> - PF_DEBUG_NOISY };
> -.Ed
> +See the
> +.Xr syslog 3
> +man page for a list of valid debug levels.
> .It Dv DIOCGETSTATES Fa "struct pfioc_states *ps"
> Get state table entries.
> .Bd -literal
