diff options
author | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2021-10-12 10:27:21 +0100 |
---|---|---|
committer | Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2021-10-13 04:21:31 +0000 |
commit | abcadce44fab895afc4aa01531122972c9ef9ede (patch) | |
tree | 76211448f3a69792e9a7f1c4d9cb9c3392a10017 /epan/dissectors | |
parent | 01e858e0a3efef54957efc0f4cae0106660dded3 (diff) |
BPV7: Address check_static.py issues
epan/dissectors/packet-bpv7.c (00000000000010d0 T bp_block_canonical_free) is not referred to so could be static?
epan/dissectors/packet-bpv7.c (0000000000000e50 T bp_creation_ts_new) is not referred to so could be static?
One function was not called at all - the other only in this dissector so could be static.
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-bpv7.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/epan/dissectors/packet-bpv7.c b/epan/dissectors/packet-bpv7.c index 9c7d6a6af6..c20d7a8563 100644 --- a/epan/dissectors/packet-bpv7.c +++ b/epan/dissectors/packet-bpv7.c @@ -466,7 +466,7 @@ static void file_scope_delete(gpointer ptr) { wmem_free(wmem_file_scope(), ptr); } -bp_creation_ts_t * bp_creation_ts_new(wmem_allocator_t *alloc) { +static bp_creation_ts_t * bp_creation_ts_new(wmem_allocator_t *alloc) { bp_creation_ts_t *obj = wmem_new0(alloc, bp_creation_ts_t); return obj; } @@ -547,12 +547,6 @@ bp_block_canonical_t * bp_block_canonical_new(wmem_allocator_t *alloc, guint64 b return obj; } -void bp_block_canonical_free(wmem_allocator_t *alloc, bp_block_canonical_t *obj) { - wmem_free(alloc, obj->sec.data_i); - wmem_free(alloc, obj->sec.data_c); - wmem_free(alloc, obj); -} - static guint64 * guint64_new(wmem_allocator_t *alloc, const guint64 val) { guint64 *obj = wmem_new(alloc, guint64); *obj = val; |