vat2: do not require _crc field in API messages 88/33888/2
authorFilip Tehlar <ftehlar@cisco.com>
Wed, 29 Sep 2021 14:20:49 +0000 (14:20 +0000)
committerOle Tr�an <otroan@employees.org>
Thu, 30 Sep 2021 08:28:55 +0000 (08:28 +0000)
Type: improvement

Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
Change-Id: Icc2ce594225c3197c9e5be8faa3dc2ee5b0a553e

src/vat2/main.c

index 667f473..2d14847 100644 (file)
@@ -133,17 +133,16 @@ vat2_register_function (char *name, cJSON (*f) (cJSON *),
 static int
 vat2_exec_command_by_name (char *msgname, cJSON *o)
 {
+  u32 crc = 0;
   if (filter_message (msgname))
     return 0;
 
   cJSON *crc_obj = cJSON_GetObjectItem (o, "_crc");
-  if (!crc_obj)
+  if (crc_obj)
     {
-      fprintf (stderr, "Missing '_crc' element!\n");
-      return -1;
+      char *crc_str = cJSON_GetStringValue (crc_obj);
+      crc = (u32) strtol (crc_str, NULL, 16);
     }
-  char *crc_str = cJSON_GetStringValue (crc_obj);
-  u32 crc = (u32) strtol (crc_str, NULL, 16);
 
   uword *p = hash_get_mem (function_by_name, msgname);
   if (!p)
@@ -151,7 +150,7 @@ vat2_exec_command_by_name (char *msgname, cJSON *o)
       fprintf (stderr, "No such command %s\n", msgname);
       return -1;
     }
-  if (crc != apifuncs[p[0]].crc)
+  if (crc && crc != apifuncs[p[0]].crc)
     {
       fprintf (stderr, "API CRC does not match: %s!\n", msgname);
     }