ospf6d consistently fails when I ask it to reload its config, even
though I have a very basic test setup:

area 0.0.0.0 {
        interface em0 { passive }
        interface vether0
}

Fixing ospf6d doesn't seem trivial.  Having it fail and exit doesn't
seem to be a sufficient incentive, so I propose to disable reloading
until it is fixed.

Just a suggestion, objections (and oks) welcome.


Index: ospf6ctl/ospf6ctl.c
===================================================================
RCS file: /d/cvs/src/usr.sbin/ospf6ctl/ospf6ctl.c,v
retrieving revision 1.46
diff -u -p -r1.46 ospf6ctl.c
--- ospf6ctl/ospf6ctl.c 12 Aug 2017 22:09:54 -0000      1.46
+++ ospf6ctl/ospf6ctl.c 5 Nov 2017 14:37:40 -0000
@@ -232,10 +232,14 @@ main(int argc, char *argv[])
                done = 1;
                break;
        case RELOAD:
+#ifdef notyet
                imsg_compose(ibuf, IMSG_CTL_RELOAD, 0, 0, -1, NULL, 0);
                printf("reload request sent.\n");
                done = 1;
                break;
+#else
+               errx(1, "reload not supported");
+#endif
        }
 
        while (ibuf->w.queued)
Index: ospf6d/ospf6d.c
===================================================================
RCS file: /d/cvs/src/usr.sbin/ospf6d/ospf6d.c,v
retrieving revision 1.34
diff -u -p -r1.34 ospf6d.c
--- ospf6d/ospf6d.c     12 Aug 2017 16:27:50 -0000      1.34
+++ ospf6d/ospf6d.c     5 Nov 2017 14:36:08 -0000
@@ -561,6 +561,7 @@ ospf_redistribute(struct kroute *kr, u_i
 int
 ospf_reload(void)
 {
+#ifdef notyet
        struct area             *area;
        struct ospfd_conf       *xconf;
 
@@ -586,6 +587,9 @@ ospf_reload(void)
        /* update redistribute lists */
        kr_reload();
        return (0);
+#else
+       return (-1);
+#endif
 }
 
 int


-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to