aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorruengeler <ruengeler@wireshark.org>2018-04-17 11:44:20 +0200
committerDario Lombardo <lomato@gmail.com>2018-04-17 12:50:42 +0000
commitfebaa4f119458c3754580d9c847da13c458345be (patch)
tree78208732a6659463e4f4f65eecfe1b6a8fc1a436
parentad84eec866713b2f75e9b11a30468b81be7d3d5e (diff)
Qt: Fix segfault bug when analysing SCTP packets
Change-Id: Id444318621ffa61ee5edb84d40de372e8b7f60ef Fixes: v2.5.2rc0-26-gecda3d0901 ("tap-sctp: remove guint32 unneeded allocations.") Reviewed-on: https://code.wireshark.org/review/26977 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Dario Lombardo <lomato@gmail.com>
-rw-r--r--ui/qt/sctp_assoc_analyse_dialog.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/ui/qt/sctp_assoc_analyse_dialog.cpp b/ui/qt/sctp_assoc_analyse_dialog.cpp
index 02c4e1b..a85cbe6 100644
--- a/ui/qt/sctp_assoc_analyse_dialog.cpp
+++ b/ui/qt/sctp_assoc_analyse_dialog.cpp
@@ -66,10 +66,10 @@ sctp_assoc_info_t* SCTPAssocAnalyseDialog::findAssocForPacket(capture_file* cf)
assoc = (sctp_assoc_info_t*)(list->data);
framelist = g_list_first(assoc->frame_numbers);
+ guint32 fn;
while (framelist) {
- guint32 *fn;
- fn = (guint32 *)framelist->data;
- if (*fn == fdata->num) {
+ fn = GPOINTER_TO_UINT(framelist->data);
+ if (fn == fdata->num) {
frame_found = TRUE;
break;
}