diff options
| author | Evan Huus <eapache@gmail.com> | 2014-04-03 08:55:06 -0400 |
|---|---|---|
| committer | Evan Huus <eapache@gmail.com> | 2014-04-03 12:55:42 +0000 |
| commit | dc84709705644272dce313b8fc40c4ed2e5300f7 (patch) | |
| tree | c51531c7fda32f5b3f078b592385c19bff745fe6 | |
| parent | 5e4769dc87900f83b63a2b4ce7aabd7275dc6bff (diff) | |
please checkAPI
Change-Id: I2dbd1fcb47b75300e753b5ef2dc60e68ca1b5fa5
Reviewed-on: https://code.wireshark.org/review/941
Reviewed-by: Evan Huus <eapache@gmail.com>
| -rw-r--r-- | epan/dissectors/packet-rtp.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/epan/dissectors/packet-rtp.c b/epan/dissectors/packet-rtp.c index 01b02c9335..ee8b0fb890 100644 --- a/epan/dissectors/packet-rtp.c +++ b/epan/dissectors/packet-rtp.c @@ -900,7 +900,7 @@ rtp_dyn_payloads_table_steal_func(gpointer key _U_, gpointer value, gpointer use if (rtp_dyn_payload->ref_count == 0) { /* this shouldn't happen */ - g_error("rtp_dyn_payload cannot be free'd because it should already have been!\n"); + DPRINT(("rtp_dyn_payload cannot be free'd because it should already have been!\n")); } else if (rtp_dyn_payload->table) { /* each member was created with a wmem file scope, so there's no point in calling the @@ -919,8 +919,6 @@ static gboolean wmem_rtp_dyn_payload_destroy_cb(wmem_allocator_t *allocator _U_, wmem_cb_event_t event _U_, void *user_data _U_) { - g_assert(rtp_dyn_payloads); - DPRINT(("destroying %u remaining rtp_dyn_payload_t's", g_hash_table_size(rtp_dyn_payloads))); /* each member was created with a wmem file scope, so there's no point in calling the @@ -1071,9 +1069,8 @@ rtp_dyn_payload_free(rtp_dyn_payload_t *rtp_dyn_payload) #endif /* remove it from the single rtp_dyn_payloads GHashTable */ - g_assert(rtp_dyn_payloads); if (!g_hash_table_remove(rtp_dyn_payloads, rtp_dyn_payload)) { - g_error("rtp_dyn_payload not found in rtp_dyn_payloads table to remove!"); + DPRINT(("rtp_dyn_payload not found in rtp_dyn_payloads table to remove!")); } /* destroy the table GHashTable in it - this automatically deletes the |