From 6fe11c994151344c38d1b080c3f2e1280b4b2448 Mon Sep 17 00:00:00 2001
From: "Ondrej Zajicek (work)" <santiago@crfreenet.org>
Date: Wed, 29 Mar 2017 15:31:04 +0200
Subject: [PATCH] BGP: Simplify igp table options

---
 proto/bgp/bgp.c    |  7 +++++++
 proto/bgp/config.Y | 13 +++++++++----
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/proto/bgp/bgp.c b/proto/bgp/bgp.c
index ad3320be..01ac5453 100644
--- a/proto/bgp/bgp.c
+++ b/proto/bgp/bgp.c
@@ -1656,6 +1656,7 @@ bgp_postconfig(struct proto_config *CF)
     if (cc->gr_able == 0xff)
       cc->gr_able = (cf->gr_mode == BGP_GR_ABLE);
 
+    /* Default values of IGP tables */
     if ((cc->gw_mode == GW_RECURSIVE) && !cc->desc->no_igp)
     {
       if (!cc->igp_table_ip4 && (bgp_cc_is_ipv4(cc) || cc->ext_next_hop))
@@ -1663,6 +1664,12 @@ bgp_postconfig(struct proto_config *CF)
 
       if (!cc->igp_table_ip6 && (bgp_cc_is_ipv6(cc) || cc->ext_next_hop))
 	cc->igp_table_ip6 = bgp_default_igp_table(cf, cc, NET_IP6);
+
+      if (cc->igp_table_ip4 && bgp_cc_is_ipv6(cc) && !cc->ext_next_hop)
+	cf_error("Mismatched IGP table type");
+
+      if (cc->igp_table_ip6 && bgp_cc_is_ipv4(cc) && !cc->ext_next_hop)
+	cf_error("Mismatched IGP table type");
     }
 
     if (cf->multihop && (cc->gw_mode == GW_DIRECT))
diff --git a/proto/bgp/config.Y b/proto/bgp/config.Y
index 1678256b..e2dc4053 100644
--- a/proto/bgp/config.Y
+++ b/proto/bgp/config.Y
@@ -179,11 +179,16 @@ bgp_channel_item:
  | ADD PATHS TX { BGP_CC->add_path = BGP_ADD_PATH_TX; }
  | ADD PATHS bool { BGP_CC->add_path = $3 ? BGP_ADD_PATH_FULL : 0; }
  | IGP TABLE rtable {
-     if (bgp_cc_is_ipv4(BGP_CC)) BGP_CC->igp_table_ip4 = $3;
-     if (bgp_cc_is_ipv6(BGP_CC)) BGP_CC->igp_table_ip6 = $3;
+    if (BGP_CC->desc->no_igp)
+      cf_error("IGP table not allowed here");
+
+    if ($3->addr_type == NET_IP4)
+      BGP_CC->igp_table_ip4 = $3;
+    else if ($3->addr_type == NET_IP6)
+      BGP_CC->igp_table_ip6 = $3;
+    else
+      cf_error("Mismatched IGP table type");
    }
- | IGP TABLE IPV4 rtable { BGP_CC->igp_table_ip4 = $4; }
- | IGP TABLE IPV6 rtable { BGP_CC->igp_table_ip6 = $4; }
  ;
 
 bgp_channel_opts: