vppapigen: missing crcs in user-defined types 77/27677/8
authorOle Troan <ot@cisco.com>
Thu, 25 Jun 2020 12:27:46 +0000 (14:27 +0200)
committerAndrew Yourtchenko <ayourtch@gmail.com>
Thu, 23 Jul 2020 11:33:30 +0000 (11:33 +0000)
make the change backwards compatible

Change-Id: I379691aa5972f99b2144deda4b7ef9e6ba4af67d
Type: fix
Signed-off-by: Ole Troan <ot@cisco.com>
src/tools/vppapigen/test_vppapigen.py
src/tools/vppapigen/vppapigen.py

index cff2400..a8e5263 100755 (executable)
@@ -1,10 +1,11 @@
 #!/usr/bin/env python3
 
 import unittest
-from vppapigen import VPPAPI, Option, ParseError, Union
+from vppapigen import VPPAPI, Option, ParseError, Union, foldup_crcs, global_types
+import vppapigen
 
 # TODO
-# - test parsing of options, typedefs, enums, defines, CRC
+# - test parsing of options, typedefs, enums, defines
 # - test JSON, C output
 
 
@@ -140,5 +141,109 @@ class TestService(unittest.TestCase):
         self.assertEqual(s['Service'][0].reply, 'show_version_reply')
 
 
+def get_crc(apistring, name):
+    vppapigen.global_types = {}
+    parser = vppapigen.VPPAPI()
+    r = parser.parse_string(apistring)
+    s = parser.process(r)
+    foldup_crcs(s['Define'])
+    d = [f for f in s['Define'] if f.name == name]
+    return d[0].crc
+
+
+class TestCRC(unittest.TestCase):
+    def test_crc(self):
+        test_string = '''
+         typedef list { u8 foo; };
+         autoreply define foo { u8 foo; vl_api_list_t l;};
+        '''
+        crc = get_crc(test_string, 'foo')
+
+        # modify underlaying type
+        test_string = '''
+         typedef list { u8 foo2; };
+         autoreply define foo { u8 foo;  vl_api_list_t l;};
+        '''
+        crc2 = get_crc(test_string, 'foo')
+        self.assertNotEqual(crc, crc2)
+
+        # two user-defined types
+        test_string = '''
+         typedef address { u8 foo2; };
+         typedef list { u8 foo2; vl_api_address_t add; };
+         autoreply define foo { u8 foo;  vl_api_list_t l;};
+        '''
+        crc3 = get_crc(test_string, 'foo')
+
+        test_string = '''
+         typedef address { u8 foo3; };
+         typedef list { u8 foo2; vl_api_address_t add; };
+         autoreply define foo { u8 foo;  vl_api_list_t l;};
+        '''
+        crc4 = get_crc(test_string, 'foo')
+        self.assertNotEqual(crc3, crc4)
+
+        test_string = '''
+         typedef address { u8 foo3; };
+         typedef list { u8 foo2; vl_api_address_t add; u8 foo3; };
+         autoreply define foo { u8 foo;  vl_api_list_t l;};
+        '''
+        crc5 = get_crc(test_string, 'foo')
+        self.assertNotEqual(crc4, crc5)
+
+        test_string = '''
+typedef ip6_address
+{
+  u8 foo;
+};
+typedef srv6_sid_list
+{
+  u8 num_sids;
+  u32 weight;
+  u32 sl_index;
+  vl_api_ip6_address_t sids[16];
+};
+autoreply define sr_policy_add
+{
+  u32 client_index;
+  u32 context;
+  vl_api_ip6_address_t bsid_addr;
+  u32 weight;
+  bool is_encap;
+  bool is_spray;
+  u32 fib_table;
+  vl_api_srv6_sid_list_t sids;
+};
+'''
+
+        crc = get_crc(test_string, 'sr_policy_add')
+
+        test_string = '''
+typedef ip6_address
+{
+  u8 foo;
+};
+typedef srv6_sid_list
+{
+  u8 num_sids;
+  u32 weight;
+  vl_api_ip6_address_t sids[16];
+};
+autoreply define sr_policy_add
+{
+  u32 client_index;
+  u32 context;
+  vl_api_ip6_address_t bsid_addr;
+  u32 weight;
+  bool is_encap;
+  bool is_spray;
+  u32 fib_table;
+  vl_api_srv6_sid_list_t sids;
+};
+'''
+        crc2 = get_crc(test_string, 'sr_policy_add')
+
+        self.assertNotEqual(crc, crc2)
+
 if __name__ == '__main__':
     unittest.main(verbosity=2)
index 94e770e..8b6297e 100755 (executable)
@@ -932,17 +932,312 @@ def foldup_blocks(block, crc):
             t = global_types[b.fieldtype]
             try:
                 crc = crc_block_combine(t.block, crc)
-                return foldup_blocks(t.block, crc)
+                crc = foldup_blocks(t.block, crc)
             except AttributeError:
                 pass
     return crc
 
 
+# keep the CRCs of the existing types of messages compatible with the
+# old "erroneous" way of calculating the CRC. For that - make a pointed
+# adjustment of the CRC function.
+# This is the purpose of the first element of the per-message dictionary.
+# The second element is there to avoid weakening the duplicate-detecting
+# properties of crc32. This way, if the new way of calculating the CRC
+# happens to collide with the old (buggy) way - we will still get
+# a different result and fail the comparison.
+
+fixup_crc_dict = {
+        "abf_policy_add_del": { 0xc6131197: 0xee66f93e },
+        "abf_policy_details": { 0xb7487fa4: 0x6769e504 },
+        "acl_add_replace": { 0xee5c2f18: 0x1cabdeab },
+        "acl_details": { 0x95babae0: 0x7a97f21c },
+        "macip_acl_add": { 0xce6fbad0: 0xd648fd0a },
+        "macip_acl_add_replace": { 0x2a461dd4: 0xe34402a7 },
+        "macip_acl_details": { 0x27135b59: 0x57c7482f },
+        "dhcp_proxy_config": { 0x4058a689: 0x6767230e },
+        "dhcp_client_config": { 0x1af013ea: 0x959b80a3 },
+        "dhcp_compl_event": { 0x554a44e5: 0xe908fd1d },
+        "dhcp_client_details": { 0x3c5cd28a: 0xacd82f5a },
+        "dhcp_proxy_details": { 0xdcbaf540: 0xce16f044 },
+        "dhcp6_send_client_message": { 0xf8222476: 0xf6f14ef0 },
+        "dhcp6_pd_send_client_message": { 0x3739fd8d: 0x64badb8 },
+        "dhcp6_reply_event": { 0x85b7b17e: 0x9f3af9e5 },
+        "dhcp6_pd_reply_event": { 0x5e878029: 0xcb3e462b },
+        "ip6_add_del_address_using_prefix": { 0x3982f30a: 0x9b3d11e0 },
+        "gbp_bridge_domain_add": { 0x918e8c01: 0x8454bfdf },
+        "gbp_bridge_domain_details": { 0x51d51be9: 0x2acd15f9 },
+        "gbp_route_domain_add": { 0x204c79e1: 0x2d0afe38 },
+        "gbp_route_domain_details": { 0xa78bfbca: 0x8ab11375 },
+        "gbp_endpoint_add": { 0x7b3af7de: 0x9ce16d5a },
+        "gbp_endpoint_details": { 0x8dd8fbd3: 0x8aecb60 },
+        "gbp_endpoint_group_add": { 0x301ddf15: 0x8e0f4054 },
+        "gbp_endpoint_group_details": { 0xab71d723: 0x8f38292c },
+        "gbp_subnet_add_del": { 0xa8803c80: 0x888aca35 },
+        "gbp_subnet_details": { 0xcbc5ca18: 0x4ed84156 },
+        "gbp_contract_add_del": { 0xaa8d652d: 0x553e275b },
+        "gbp_contract_details": { 0x65dec325: 0x2a18db6e },
+        "gbp_ext_itf_add_del": { 0x7606d0e1: 0x12ed5700 },
+        "gbp_ext_itf_details": { 0x519c3d3c: 0x408a45c0 },
+        "gtpu_add_del_tunnel": { 0xca983a2b: 0x9a9c0426 },
+        "gtpu_tunnel_update_tteid": { 0x79f33816: 0x8a2db108 },
+        "gtpu_tunnel_details": { 0x27f434ae: 0x4535cf95 },
+        "igmp_listen": { 0x19a49f1e: 0x3f93a51a },
+        "igmp_details": { 0x38f09929: 0x52f12a89 },
+        "igmp_event": { 0x85fe93ec: 0xd7696eaf },
+        "igmp_group_prefix_set": { 0x5b14a5ce: 0xd4f20ac5 },
+        "igmp_group_prefix_details": { 0x259ccd81: 0xc3b3c526 },
+        "ikev2_set_responder": { 0xb9aa4d4e: 0xf0d3dc80 },
+        "vxlan_gpe_ioam_export_enable_disable": { 0xd4c76d3a: 0xe4d4ebfa },
+        "ioam_export_ip6_enable_disable": { 0xd4c76d3a: 0xe4d4ebfa },
+        "vxlan_gpe_ioam_vni_enable": { 0xfbb5fb1: 0x997161fb },
+        "vxlan_gpe_ioam_vni_disable": { 0xfbb5fb1: 0x997161fb },
+        "vxlan_gpe_ioam_transit_enable": { 0x3d3ec657: 0x553f5b7b },
+        "vxlan_gpe_ioam_transit_disable": { 0x3d3ec657: 0x553f5b7b },
+        "udp_ping_add_del": { 0xfa2628fc: 0xc692b188 },
+        "l3xc_update": { 0xe96aabdf: 0x787b1d3 },
+        "l3xc_details": { 0xbc5bf852: 0xd4f69627 },
+        "sw_interface_lacp_details": { 0xd9a83d2f: 0x745ae0ba },
+        "lb_conf": { 0x56cd3261: 0x22ddb739 },
+        "lb_add_del_vip": { 0x6fa569c7: 0xd15b7ddc },
+        "lb_add_del_as": { 0x35d72500: 0x78628987 },
+        "lb_vip_dump": { 0x56110cb7: 0xc7bcb124 },
+        "lb_vip_details": { 0x1329ec9b: 0x8f39bed },
+        "lb_as_details": { 0x8d24c29e: 0x9c39f60e },
+        "mactime_add_del_range": { 0xcb56e877: 0x101858ef },
+        "mactime_details": { 0xda25b13a: 0x44921c06 },
+        "map_add_domain": { 0x249f195c: 0x7a5a18c9 },
+        "map_domain_details": { 0x796edb50: 0xfc1859dd },
+        "map_param_add_del_pre_resolve": { 0xdae5af03: 0x17008c66 },
+        "map_param_get_reply": { 0x26272c90: 0x28092156 },
+        "memif_details": { 0xda34feb9: 0xd0382c4c },
+        "dslite_add_del_pool_addr_range": { 0xde2a5b02: 0xc448457a },
+        "dslite_set_aftr_addr": { 0x78b50fdf: 0x1e955f8d },
+        "dslite_get_aftr_addr_reply": { 0x8e23608e: 0x38e30db1 },
+        "dslite_set_b4_addr": { 0x78b50fdf: 0x1e955f8d },
+        "dslite_get_b4_addr_reply": { 0x8e23608e: 0x38e30db1 },
+        "nat44_add_del_address_range": { 0x6f2b8055: 0xd4c7568c },
+        "nat44_address_details": { 0xd1beac1: 0x45410ac4 },
+        "nat44_add_del_static_mapping": { 0x5ae5f03e: 0xe165e83b },
+        "nat44_static_mapping_details": { 0x6cb40b2: 0x1a433ef7 },
+        "nat44_add_del_identity_mapping": { 0x2faaa22: 0x8e12743f },
+        "nat44_identity_mapping_details": { 0x2a52a030: 0x36d21351 },
+        "nat44_add_del_interface_addr": { 0x4aed50c0: 0xfc835325 },
+        "nat44_interface_addr_details": { 0xe4aca9ca: 0x3e687514 },
+        "nat44_user_session_details": { 0x2cf6e16d: 0x1965fd69 },
+        "nat44_add_del_lb_static_mapping": { 0x4f68ee9d: 0x53b24611 },
+        "nat44_lb_static_mapping_add_del_local": { 0x7ca47547: 0x2910a151 },
+        "nat44_lb_static_mapping_details": { 0xed5ce876: 0x2267b9e8 },
+        "nat44_del_session": { 0x15a5bf8c: 0x4c49c387 },
+        "nat_det_add_del_map": { 0x1150a190: 0x112fde05 },
+        "nat_det_map_details": { 0xad91dc83: 0x88000ee1 },
+        "nat_det_close_session_out": { 0xf6b259d1: 0xc1b6cbfb },
+        "nat_det_close_session_in": { 0x3c68e073: 0xa10ef64 },
+        "nat64_add_del_pool_addr_range": { 0xa3b944e3: 0x21234ef3 },
+        "nat64_add_del_static_bib": { 0x1c404de5: 0x90fae58a },
+        "nat64_bib_details": { 0x43bc3ddf: 0x62c8541d },
+        "nat64_st_details": { 0xdd3361ed: 0xc770d620 },
+        "nat66_add_del_static_mapping": { 0x3ed88f71: 0xfb64e50b },
+        "nat66_static_mapping_details": { 0xdf39654b: 0x5c568448 },
+        "nsh_add_del_map": { 0xa0f42b0: 0x898d857d },
+        "nsh_map_details": { 0x2fefcf49: 0xb34ac8a1 },
+        "nsim_cross_connect_enable_disable": { 0x9c3ead86: 0x16f70bdf },
+        "pppoe_add_del_session": { 0xf6fd759e: 0x46ace853 },
+        "pppoe_session_details": { 0x4b8e8a4a: 0x332bc742 },
+        "stn_add_del_rule": { 0x224c6edd: 0x53f751e6 },
+        "stn_rules_details": { 0xa51935a6: 0xb0f6606c },
+        "svs_route_add_del": { 0xe49bc63c: 0xd39e31fc },
+        "svs_details": { 0x6282cd55: 0xb8523d64 },
+        "vmxnet3_details": { 0x6a1a5498: 0x829ba055 },
+        "vrrp_vr_add_del": { 0xc5cf15aa: 0x6dc4b881 },
+        "vrrp_vr_details": { 0x46edcebd: 0x412fa71 },
+        "vrrp_vr_set_peers": { 0x20bec71f: 0xbaa2e52b },
+        "vrrp_vr_peer_details": { 0x3d99c108: 0xabd9145e },
+        "vrrp_vr_track_if_add_del": { 0xd67df299: 0x337f4ba4 },
+        "vrrp_vr_track_if_details": { 0x73c36f81: 0x99bcca9c },
+        "proxy_arp_add_del": { 0x1823c3e7: 0x85486cbd },
+        "proxy_arp_details": { 0x5b948673: 0x9228c150 },
+        "bfd_udp_get_echo_source_reply": { 0xe3d736a1: 0x1e00cfce },
+        "bfd_udp_add": { 0x939cd26a: 0x7a6d1185 },
+        "bfd_udp_mod": { 0x913df085: 0x783a3ff6 },
+        "bfd_udp_del": { 0xdcb13a89: 0x8096514d },
+        "bfd_udp_session_details": { 0x9fb2f2d: 0x60653c02 },
+        "bfd_udp_session_set_flags": { 0x4b4bdfd: 0xcf313851 },
+        "bfd_udp_auth_activate": { 0x21fd1bdb: 0x493ee0ec },
+        "bfd_udp_auth_deactivate": { 0x9a05e2e0: 0x99978c32 },
+        "bier_route_add_del": { 0xfd02f3ea: 0xf29edca0 },
+        "bier_route_details": { 0x4008caee: 0x39ee6a56 },
+        "bier_disp_entry_add_del": { 0x9eb80cb4: 0x648323eb },
+        "bier_disp_entry_details": { 0x84c218f1: 0xe5b039a9 },
+        "bond_create": { 0xf1dbd4ff: 0x48883c7e },
+        "bond_enslave": { 0xe7d14948: 0x76ecfa7 },
+        "sw_interface_bond_details": { 0xbb7c929b: 0xf5ef2106 },
+        "pipe_create_reply": { 0xb7ce310c: 0xd4c2c2b3 },
+        "pipe_details": { 0xc52b799d: 0x43ac107a },
+        "tap_create_v2": { 0x2d0d6570: 0x445835fd },
+        "sw_interface_tap_v2_details": { 0x1e2b2a47: 0xe53c16de },
+        "sw_interface_vhost_user_details": { 0xcee1e53: 0x98530df1 },
+        "virtio_pci_create": { 0x1944f8db: 0xa9f1370c },
+        "sw_interface_virtio_pci_details": { 0x6ca9c167: 0x16187f3a },
+        "p2p_ethernet_add": { 0x36a1a6dc: 0xeeb8e717 },
+        "p2p_ethernet_del": { 0x62f81c8c: 0xb62c386 },
+        "geneve_add_del_tunnel": { 0x99445831: 0x976693b5 },
+        "geneve_tunnel_details": { 0x6b16eb24: 0xe27e2748 },
+        "gre_tunnel_add_del": { 0xa27d7f17: 0x6efc9c22 },
+        "gre_tunnel_details": { 0x24435433: 0x3bfbf1 },
+        "sw_interface_set_flags": { 0xf5aec1b8: 0x6a2b491a },
+        "sw_interface_event": { 0x2d3d95a7: 0xf709f78d },
+        "sw_interface_details": { 0x6c221fc7: 0x17b69fa2 },
+        "sw_interface_add_del_address": { 0x5463d73b: 0x5803d5c4 },
+        "sw_interface_set_unnumbered": { 0x154a6439: 0x938ef33b },
+        "sw_interface_set_mac_address": { 0xc536e7eb: 0x6aca746a },
+        "sw_interface_set_rx_mode": { 0xb04d1cfe: 0x780f5cee },
+        "sw_interface_rx_placement_details": { 0x9e44a7ce: 0xf6d7d024 },
+        "create_subif": { 0x790ca755: 0xcb371063 },
+        "ip_neighbor_add_del": { 0x607c257: 0x105518b6 },
+        "ip_neighbor_dump": { 0xd817a484: 0xcd831298 },
+        "ip_neighbor_details": { 0xe29d79f0: 0x870e80b9 },
+        "want_ip_neighbor_events": { 0x73e70a86: 0x1a312870 },
+        "ip_neighbor_event": { 0xbdb092b2: 0x83933131 },
+        "ip_route_add_del": { 0xb8ecfe0d: 0xc1ff832d },
+        "ip_route_details": { 0xbda8f315: 0xd1ffaae1 },
+        "ip_route_lookup": { 0x710d6471: 0xe2986185 },
+        "ip_route_lookup_reply": { 0x5d8febcb: 0xae99de8e },
+        "ip_mroute_add_del": { 0x85d762f3: 0xf6627d17 },
+        "ip_mroute_details": { 0x99341a45: 0xc1cb4b44 },
+        "ip_address_details": { 0xee29b797: 0xb1199745 },
+        "ip_unnumbered_details": { 0xcc59bd42: 0xaa12a483 },
+        "mfib_signal_details": { 0x6f4a4cfb: 0x64398a9a },
+        "ip_punt_redirect": { 0x6580f635: 0xa9a5592c },
+        "ip_punt_redirect_details": { 0x2cef63e7: 0x3924f5d3 },
+        "ip_container_proxy_add_del": { 0x7df1dff1: 0x91189f40 },
+        "ip_container_proxy_details": { 0xa8085523: 0xee460e8 },
+        "ip_source_and_port_range_check_add_del": { 0x92a067e3: 0x8bfc76f2 },
+        "sw_interface_ip6_set_link_local_address": { 0x1c10f15f: 0x2931d9fa },
+        "ip_reassembly_enable_disable": { 0xeb77968d: 0x885c85a6 },
+        "set_punt": { 0xaa83d523: 0x83799618 },
+        "punt_socket_register": { 0x95268cbf: 0xc8cd10fa },
+        "punt_socket_details": { 0xde575080: 0x1de0ce75 },
+        "punt_socket_deregister": { 0x98fc9102: 0x98a444f4 },
+        "sw_interface_ip6nd_ra_prefix": { 0x82cc1b28: 0xe098785f },
+        "ip6nd_proxy_add_del": { 0xc2e4a686: 0x3fdf6659 },
+        "ip6nd_proxy_details": { 0x30b9ff4a: 0xd35be8ff },
+        "ip6_ra_event": { 0x364c1c5: 0x47e8cfbe },
+        "set_ipfix_exporter": { 0x5530c8a0: 0x69284e07 },
+        "ipfix_exporter_details": { 0xdedbfe4: 0x11e07413 },
+        "ipip_add_tunnel": { 0x2ac399f5: 0xa9decfcd },
+        "ipip_6rd_add_tunnel": { 0xb9ec1863: 0x56e93cc0 },
+        "ipip_tunnel_details": { 0xd31cb34e: 0x53236d75 },
+        "ipsec_spd_entry_add_del": { 0x338b7411: 0x9f384b8d },
+        "ipsec_spd_details": { 0x5813d7a2: 0xf2222790 },
+        "ipsec_sad_entry_add_del": { 0xab64b5c6: 0xb8def364 },
+        "ipsec_tunnel_protect_update": { 0x30d5f133: 0x143f155d },
+        "ipsec_tunnel_protect_del": { 0xcd239930: 0xddd2ba36 },
+        "ipsec_tunnel_protect_details": { 0x21663a50: 0xac6c823b },
+        "ipsec_tunnel_if_add_del": { 0x20e353fa: 0x2b135e68 },
+        "ipsec_sa_details": { 0x345d14a7: 0xb30c7f41 },
+        "l2_xconnect_details": { 0x472b6b67: 0xc8aa6b37 },
+        "l2_fib_table_details": { 0xa44ef6b8: 0xe8d2fc72 },
+        "l2fib_add_del": { 0xeddda487: 0xf29d796c },
+        "l2_macs_event": { 0x44b8fd64: 0x2eadfc8b },
+        "bridge_domain_details": { 0xfa506fd: 0x979f549d },
+        "l2_interface_pbb_tag_rewrite": { 0x38e802a8: 0x612efa5a },
+        "l2_patch_add_del": { 0xa1f6a6f3: 0x522f3445 },
+        "sw_interface_set_l2_xconnect": { 0x4fa28a85: 0x1aaa2dbb },
+        "sw_interface_set_l2_bridge": { 0xd0678b13: 0x2e483cd0 },
+        "bd_ip_mac_add_del": { 0x257c869: 0x5f2b84e2 },
+        "bd_ip_mac_details": { 0x545af86a: 0xa52f8044 },
+        "l2_arp_term_event": { 0x6963e07a: 0x85ff71ea },
+        "l2tpv3_create_tunnel": { 0x15bed0c2: 0x596892cb },
+        "sw_if_l2tpv3_tunnel_details": { 0x50b88993: 0x1dab5c7e },
+        "lisp_add_del_local_eid": { 0x4e5a83a2: 0x21f573bd },
+        "lisp_add_del_map_server": { 0xce19e32d: 0x6598ea7c },
+        "lisp_add_del_map_resolver": { 0xce19e32d: 0x6598ea7c },
+        "lisp_use_petr": { 0xd87dbad9: 0x9e141831 },
+        "show_lisp_use_petr_reply": { 0x22b9a4b0: 0xdcad8a81 },
+        "lisp_add_del_remote_mapping": { 0x6d5c789e: 0xfae8ed77 },
+        "lisp_add_del_adjacency": { 0x2ce0e6f6: 0xcf5edb61 },
+        "lisp_locator_details": { 0x2c620ffe: 0xc0c4c2a7 },
+        "lisp_eid_table_details": { 0x1c29f792: 0x4bc32e3a },
+        "lisp_eid_table_dump": { 0x629468b5: 0xb959b73b },
+        "lisp_adjacencies_get_reply": { 0x807257bf: 0x3f97bcdd },
+        "lisp_map_resolver_details": { 0x3e78fc57: 0x82a09deb },
+        "lisp_map_server_details": { 0x3e78fc57: 0x82a09deb },
+        "one_add_del_local_eid": { 0x4e5a83a2: 0x21f573bd },
+        "one_add_del_map_server": { 0xce19e32d: 0x6598ea7c },
+        "one_add_del_map_resolver": { 0xce19e32d: 0x6598ea7c },
+        "one_use_petr": { 0xd87dbad9: 0x9e141831 },
+        "show_one_use_petr_reply": { 0x84a03528: 0x10e744a6 },
+        "one_add_del_remote_mapping": { 0x6d5c789e: 0xfae8ed77 },
+        "one_add_del_l2_arp_entry": { 0x1aa5e8b3: 0x33209078 },
+        "one_l2_arp_entries_get_reply": { 0xb0dd200f: 0xb0a47bbe },
+        "one_add_del_ndp_entry": { 0xf8a287c: 0xd1629a2f },
+        "one_ndp_entries_get_reply": { 0x70719b1a: 0xbd34161 },
+        "one_add_del_adjacency": { 0x9e830312: 0xe48e7afe },
+        "one_locator_details": { 0x2c620ffe: 0xc0c4c2a7 },
+        "one_eid_table_details": { 0x1c29f792: 0x4bc32e3a },
+        "one_eid_table_dump": { 0xbd190269: 0x95151038 },
+        "one_adjacencies_get_reply": { 0x85bab89: 0xa8ed89a5 },
+        "one_map_resolver_details": { 0x3e78fc57: 0x82a09deb },
+        "one_map_server_details": { 0x3e78fc57: 0x82a09deb },
+        "one_stats_details": { 0x2eb74678: 0xff6ef238 },
+        "gpe_add_del_fwd_entry": { 0xf0847644: 0xde6df50f },
+        "gpe_fwd_entries_get_reply": { 0xc4844876: 0xf9f53f1b },
+        "gpe_fwd_entry_path_details": { 0x483df51a: 0xee80b19a },
+        "gpe_add_del_native_fwd_rpath": { 0x43fc8b54: 0x812da2f2 },
+        "gpe_native_fwd_rpaths_get_reply": { 0x7a1ca5a2: 0x79d54eb9 },
+        "sw_interface_set_lldp": { 0x57afbcd4: 0xd646ae0f },
+        "mpls_ip_bind_unbind": { 0xc7533b32: 0x48249a27 },
+        "mpls_tunnel_add_del": { 0x44350ac1: 0xe57ce61d },
+        "mpls_tunnel_details": { 0x57118ae3: 0xf3c0928e },
+        "mpls_route_add_del": { 0x8e1d1e07: 0x343cff54 },
+        "mpls_route_details": { 0x9b5043dc: 0xd0ac384c },
+        "policer_add_del": { 0x2b31dd38: 0xcb948f6e },
+        "policer_details": { 0x72d0e248: 0xa43f781a },
+        "qos_store_enable_disable": { 0xf3abcc8b: 0x3507235e },
+        "qos_store_details": { 0x3ee0aad7: 0x38a6d48 },
+        "qos_record_enable_disable": { 0x2f1a4a38: 0x25b33f88 },
+        "qos_record_details": { 0xa425d4d3: 0x4956ccdd },
+        "session_rule_add_del": { 0xe4895422: 0xe31f9443 },
+        "session_rules_details": { 0x28d71830: 0x304b91f0 },
+        "sw_interface_span_enable_disable": { 0x23ddd96b: 0xacc8fea1 },
+        "sw_interface_span_details": { 0x8a20e79f: 0x55643fc },
+        "sr_mpls_steering_add_del": { 0x64acff63: 0x7d1b0a0b },
+        "sr_mpls_policy_assign_endpoint_color": { 0xe7eb978: 0x5e1c5c13 },
+        "sr_localsid_add_del": { 0x5a36c324: 0x26fa3309 },
+        "sr_policy_add": { 0x44ac92e8: 0xec79ee6a },
+        "sr_policy_mod": { 0xb97bb56e: 0xe531a102 },
+        "sr_steering_add_del": { 0xe46b0a0f: 0x3711dace },
+        "sr_localsids_details": { 0x2e9221b9: 0x6a6c0265 },
+        "sr_policies_details": { 0xdb6ff2a1: 0x7ec2d93 },
+        "sr_steering_pol_details": { 0xd41258c9: 0x1c1ee786 },
+        "syslog_set_sender": { 0xb8011d0b: 0xbb641285 },
+        "syslog_get_sender_reply": { 0x424cfa4e: 0xd3da60ac },
+        "tcp_configure_src_addresses": { 0x67eede0d: 0x4b02b946 },
+        "teib_entry_add_del": { 0x8016cfd2: 0x5aa0a538 },
+        "teib_details": { 0x981ee1a1: 0xe3b6a503 },
+        "udp_encap_add": { 0xf74a60b1: 0x61d5fc48 },
+        "udp_encap_details": { 0x8cfb9c76: 0x87c82821 },
+        "vxlan_gbp_tunnel_add_del": { 0x6c743427: 0x8c819166 },
+        "vxlan_gbp_tunnel_details": { 0x66e94a89: 0x1da24016 },
+        "vxlan_gpe_add_del_tunnel": { 0xa645b2b0: 0x7c6da6ae },
+        "vxlan_gpe_tunnel_details": { 0x968fc8b: 0x57712346 },
+        "vxlan_add_del_tunnel": { 0xc09dc80: 0xa35dc8f5 },
+        "vxlan_tunnel_details": { 0xc3916cb1: 0xe782f70f },
+        "vxlan_offload_rx": { 0x9cc95087: 0x89a1564b },
+        "log_details": { 0x3d61cc0: 0x255827a1 },
+}
+
+
 def foldup_crcs(s):
     for f in s:
         f.crc = foldup_blocks(f.block,
                               binascii.crc32(f.crc) & 0xffffffff)
 
+        # fixup the CRCs to make the fix seamless
+        if f.name in fixup_crc_dict:
+            if f.crc in fixup_crc_dict.get(f.name):
+                f.crc = fixup_crc_dict.get(f.name).get(f.crc)
 
 #
 # Main