Make UAT record update callbacks return a success/failure indication.

Have them return TRUE on success and FALSE on failure.  Check the return
value rather than whether the error string pointer is null or not.

Change-Id: I800a03bcd70a6bbb7b217cf7c4800e9cdcf2189c
Reviewed-on: https://code.wireshark.org/review/7222
Reviewed-by: Guy Harris <guy@alum.mit.edu>

1 files changed, 5 insertions, 4 deletions

diff --git a/epan/dissectors/packet-imf.c b/epan/dissectors/packet-imf.c
index dae55e8e06..75d367b72c 100644
--- a/epan/dissectors/packet-imf.c
+++ b/epan/dissectors/packet-imf.c
@@ -275,7 +275,7 @@ static guint num_header_fields = 0;
 
 static GHashTable *custom_field_table = NULL;
 
-static void
+static gboolean
 header_fields_update_cb(void *r, char **err)
 {
   header_field_t *rec = (header_field_t *)r;
@@ -283,13 +283,13 @@ header_fields_update_cb(void *r, char **err)
 
   if (rec->header_name == NULL) {
     *err = g_strdup("Header name can't be empty");
-    return;
+    return FALSE;
   }
 
   g_strstrip(rec->header_name);
   if (rec->header_name[0] == 0) {
     *err = g_strdup("Header name can't be empty");
-    return;
+    return FALSE;
   }
 
   /* Check for invalid characters (to avoid asserting out when
@@ -298,10 +298,11 @@ header_fields_update_cb(void *r, char **err)
   c = proto_check_field_name(rec->header_name);
   if (c) {
     *err = g_strdup_printf("Header name can't contain '%c'", c);
-    return;
+    return FALSE;
   }
 
   *err = NULL;
+  return TRUE;
 }
 
 static void *