diff options
| author | ruengeler <ruengeler@wireshark.org> | 2018-04-17 11:44:20 +0200 |
|---|---|---|
| committer | Dario Lombardo <lomato@gmail.com> | 2018-04-17 12:50:42 +0000 |
| commit | febaa4f119458c3754580d9c847da13c458345be (patch) | |
| tree | 78208732a6659463e4f4f65eecfe1b6a8fc1a436 /ui/qt | |
| parent | ad84eec866713b2f75e9b11a30468b81be7d3d5e (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>
Diffstat (limited to 'ui/qt')
| -rw-r--r-- | ui/qt/sctp_assoc_analyse_dialog.cpp | 6 |
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 02c4e1bd0b..a85cbe675a 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; } |