diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-10-25 22:52:30 +0100 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2015-10-26 23:57:51 +0000 |
commit | 1ea43c26e68d92838a7d91dce4fe9686b09f12c7 (patch) | |
tree | 65c8b3f84f589530b16fc6159974f1813aa8968d /ui/gtk/rtp_player.c | |
parent | d141562efb18f777c122405941bd5ef8c21513b1 (diff) |
Only store frame number and not a pointer to frame_data structure in seq_analysis_item_t
Qt GUI allows to keep windows open after closing the capture file. Let's not access to freed memory.
Bug: 11573
Change-Id: I9e70d818a4b228af319961ec512b6b9725792477
Reviewed-on: https://code.wireshark.org/review/11270
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/gtk/rtp_player.c')
-rw-r--r-- | ui/gtk/rtp_player.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/gtk/rtp_player.c b/ui/gtk/rtp_player.c index fd69a27ae9..08d7344581 100644 --- a/ui/gtk/rtp_player.c +++ b/ui/gtk/rtp_player.c @@ -399,7 +399,7 @@ mark_rtp_stream_to_play(gchar *key _U_ , rtp_stream_info_t *rsi, gpointer ptr _U while (graph_list) { graph_item = (seq_analysis_item_t *)graph_list->data; - if (rsi->start_fd->num == graph_item->fd->num) { + if (rsi->start_fd->num == graph_item->frame_number) { rsi->call_num = graph_item->conv_num; /* if it is in the graph list, then check if the voip_call is selected */ voip_calls_list = g_queue_peek_nth_link(voip_calls->callsinfos, 0); |