On Thu, May 14, 2020 at 04:10:42PM +0200, Denis Fondras wrote:
> IMSG_IFADD is never used, wipe it.
In ospfd we have IMSG_RECONF_IFACE for this. Once we start adding
reload functionality we can bring that over to ospf6d.
OK remi@
>
> Index: ospf6d.h
> ===
> RCS file: /cvs/src/usr.sbin/ospf6d/ospf6d.h,v
> retrieving revision 1.46
> diff -u -p -r1.46 ospf6d.h
> --- ospf6d.h 5 Apr 2020 18:19:04 - 1.46
> +++ ospf6d.h 14 May 2020 13:52:08 -
> @@ -103,7 +103,6 @@ enum imsg_type {
> IMSG_KROUTE_CHANGE,
> IMSG_KROUTE_DELETE,
> IMSG_IFINFO,
> - IMSG_IFADD,
> IMSG_IFDELETE,
> IMSG_IFADDRNEW,
> IMSG_IFADDRDEL,
> Index: ospfe.c
> ===
> RCS file: /cvs/src/usr.sbin/ospf6d/ospfe.c,v
> retrieving revision 1.61
> diff -u -p -r1.61 ospfe.c
> --- ospfe.c 2 Jan 2020 10:16:46 - 1.61
> +++ ospfe.c 14 May 2020 13:52:08 -
> @@ -327,18 +327,6 @@ ospfe_dispatch_main(int fd, short event,
> log_warnx("interface %s down", iface->name);
> }
> break;
> - case IMSG_IFADD:
> - if ((iface = malloc(sizeof(struct iface))) == NULL)
> - fatal(NULL);
> - memcpy(iface, imsg.data, sizeof(struct iface));
> -
> - LIST_INIT(&iface->nbr_list);
> - TAILQ_INIT(&iface->ls_ack_list);
> - RB_INIT(&iface->lsa_tree);
> -
> - LIST_INSERT_HEAD(&iface->area->iface_list, iface,
> - entry);
> - break;
> case IMSG_IFDELETE:
> if (imsg.hdr.len != IMSG_HEADER_SIZE +
> sizeof(ifindex))
> Index: rde.c
> ===
> RCS file: /cvs/src/usr.sbin/ospf6d/rde.c,v
> retrieving revision 1.86
> diff -u -p -r1.86 rde.c
> --- rde.c 5 Apr 2020 18:19:04 - 1.86
> +++ rde.c 14 May 2020 13:52:09 -
> @@ -734,17 +734,6 @@ rde_dispatch_parent(int fd, short event,
> orig_intra_area_prefix_lsas(iface->area);
>
> break;
> - case IMSG_IFADD:
> - if ((iface = malloc(sizeof(struct iface))) == NULL)
> - fatal(NULL);
> - memcpy(iface, imsg.data, sizeof(struct iface));
> -
> - LIST_INIT(&iface->nbr_list);
> - TAILQ_INIT(&iface->ls_ack_list);
> - RB_INIT(&iface->lsa_tree);
> -
> - LIST_INSERT_HEAD(&iface->area->iface_list, iface,
> entry);
> - break;
> case IMSG_IFDELETE:
> if (imsg.hdr.len != IMSG_HEADER_SIZE +
> sizeof(ifindex))
>