diff options
| author | Guy Harris <guy@alum.mit.edu> | 2015-02-17 16:40:28 -0800 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2015-02-18 00:41:09 +0000 |
| commit | 859511db14feae071cc1e6e1889ae39bfd276faa (patch) | |
| tree | a6ef5501110220c93d583439f5f0257c243cde64 /plugins | |
| parent | 3f765b3ef94fd6044279d47d0b30b46c344e5943 (diff) | |
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>
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/stats_tree/pinfo_stats_tree.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/stats_tree/pinfo_stats_tree.c b/plugins/stats_tree/pinfo_stats_tree.c index b54a5f7091..42c461e296 100644 --- a/plugins/stats_tree/pinfo_stats_tree.c +++ b/plugins/stats_tree/pinfo_stats_tree.c @@ -65,16 +65,17 @@ static void *uat_plen_record_copy_cb(void *n, const void *o, size_t siz _U_) { return n; } -static void +static gboolean uat_plen_record_update_cb(void *r, char **err) { uat_plen_record_t *rec = (uat_plen_record_t*)r; if (rec->packet_range->nranges < 1) { *err = g_strdup("Invalid range string"); - return; + return FALSE; } *err = NULL; + return TRUE; } static void uat_plen_record_free_cb(void*r) { |