+void
> +pfctl_reset(int dev, int opts)
> +{
> +     struct pfctl    pf;
> +     struct pfr_buffer t;
> +     int             i;
> +
> +     pf.dev = dev;
> +     pfctl_init_options(&pf);
> +
> +     /* Force reset upon pfctl_load_options() */
> +     pf.debug_set = 1;
> +     pf.reass_set = 1;
> +     pf.syncookieswat_set = 1;
> +     pf.ifname = strdup("none");
> +     if (pf.ifname == NULL)
> +             warn("%s: Warning: can't reset loginterface\n", __func__);
                                                        ^^^^^^^^
                                                do you really need this
                                                extra newline here?
                                                warn() itself already includes
                                                one.
> +     else
> +             pf.ifname_set = 1;
> +
> +     memset(&t, 0, sizeof(t));
> +     t.pfrb_type = PFRB_TRANS;
> +     if (pfctl_trans(dev, &t, DIOCXBEGIN, 0))
> +             warn("%s, DIOCXBEGIN", __func__);
> +
> +     for (i = 0; pf_limits[i].name; i++)
> +             pf.limit_set[pf_limits[i].index] = 1;
> +
> +     for (i = 0; pf_timeouts[i].name; i++)
> +             pf.timeout_set[pf_timeouts[i].timeout] = 1;
> +
> +     pfctl_load_options(&pf);
> +
> +     if (pfctl_trans(dev, &t, DIOCXCOMMIT, 0))
> +             warn("%s, DIOCXCOMMIT", __func__);
> +
> +     pfctl_clear_interface_flags(dev, opts);
> +}

Reply via email to