OK florian@

On Mon, Oct 29, 2018 at 11:27:15PM +0100, Remi Locherer wrote:
> Hi,
> 
> ospf6d does not support reloading so its parent proc does not need
> filesystem access with the exception of the control socket cleanup on
> exit. Once we teach it how to reload the config it is easy to unveil "/"
> readonly as I just did for ospfd.
> 
> OK?
> 
> Remi
> 
> 
> cvs diff: Diffing .
> Index: ospf6d.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/ospf6d/ospf6d.c,v
> retrieving revision 1.39
> diff -u -p -r1.39 ospf6d.c
> --- ospf6d.c  1 Sep 2018 19:21:10 -0000       1.39
> +++ ospf6d.c  29 Oct 2018 22:20:45 -0000
> @@ -274,6 +274,11 @@ main(int argc, char *argv[])
>               fatalx("control socket setup failed");
>       main_imsg_compose_ospfe_fd(IMSG_CONTROLFD, 0, control_fd);
>  
> +     if (unveil(ospfd_conf->csock, "c") == -1)
> +             fatal("unveil");
> +     if (unveil(NULL, NULL) == -1)
> +             fatal("unveil");
> +
>       if (kr_init(!(ospfd_conf->flags & OSPFD_FLAG_NO_FIB_UPDATE),
>           ospfd_conf->rdomain) == -1)
>               fatalx("kr_init failed");
> 

-- 
I'm not entirely sure you are real.

Reply via email to