diff options
author | Stig Bjørlykke <stig@bjorlykke.org> | 2016-02-23 09:30:48 +0100 |
---|---|---|
committer | Stig Bjørlykke <stig@bjorlykke.org> | 2016-02-23 18:30:48 +0000 |
commit | b5ac91399cccdf065d065497c83ca8e8ac864fe7 (patch) | |
tree | 79380a0c9d60c926463c5de9ad776191329a4f8c /ui/qt | |
parent | 0ef196a643daf3e9654dd118cddbec5c80f7f6ff (diff) |
Qt: Select matching tree item and bytes in Find Packet
Highlight the matching tree item and matching packet bytes when
doing a Find Packet. Added cf->search_len to correctly highlight
the matching bytes when doing a regex search.
Bug: 12157
Change-Id: I84fbdb9b43be4355e24aff3cf5f8850f1119e2bf
Reviewed-on: https://code.wireshark.org/review/14086
Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
Diffstat (limited to 'ui/qt')
-rw-r--r-- | ui/qt/byte_view_tab.cpp | 18 | ||||
-rw-r--r-- | ui/qt/packet_list.cpp | 24 | ||||
-rw-r--r-- | ui/qt/proto_tree.cpp | 13 | ||||
-rw-r--r-- | ui/qt/proto_tree.h | 1 |
4 files changed, 42 insertions, 14 deletions
diff --git a/ui/qt/byte_view_tab.cpp b/ui/qt/byte_view_tab.cpp index 128cc68523..cf405081a4 100644 --- a/ui/qt/byte_view_tab.cpp +++ b/ui/qt/byte_view_tab.cpp @@ -267,20 +267,10 @@ void ByteViewTab::protoTreeItemChanged(QTreeWidgetItem *current) { } if (cap_file_->search_in_progress && (cap_file_->hex || (cap_file_->string && cap_file_->packet_data))) { - /* In the hex view, only highlight the target bytes or string. The entire - field can then be displayed by clicking on any of the bytes in the field. */ - if (cap_file_->hex) { - const char *p = cap_file_->sfilter; - f_len = 0; - while (*p) { - if (g_ascii_isxdigit(*p++)) - f_len++; - } - f_len = (f_len + 1) / 2; - } else { - f_len = (int)strlen(cap_file_->sfilter); - } - f_start = cap_file_->search_pos - (f_len-1); + // In the hex view, only highlight the target bytes or string. The entire + // field can then be displayed by clicking on any of the bytes in the field. + f_start = cap_file_->search_pos - cap_file_->search_len + 1; + f_len = cap_file_->search_len; } else { f_start = fi->start; f_len = fi->length; diff --git a/ui/qt/packet_list.cpp b/ui/qt/packet_list.cpp index ae4bb53d18..dba7f54f7f 100644 --- a/ui/qt/packet_list.cpp +++ b/ui/qt/packet_list.cpp @@ -35,6 +35,7 @@ #include <epan/ipproto.h> #include <epan/packet.h> #include <epan/prefs.h> +#include <epan/proto.h> #include "ui/main_statusbar.h" #include "ui/packet_list_utils.h" @@ -480,6 +481,29 @@ void PacketList::selectionChanged (const QItemSelection & selected, const QItemS } byte_view_tab_->setCurrentIndex(0); } + + if (cap_file_->search_in_progress && + (cap_file_->search_pos != 0 || (cap_file_->string && cap_file_->decode_data))) + { + match_data mdata; + field_info *fi = NULL; + + if (cap_file_->string && cap_file_->decode_data) { + // The tree where the target string matched one of the labels was discarded in + // match_protocol_tree() so we have to search again in the latest tree. + if (cf_find_string_protocol_tree(cap_file_, cap_file_->edt->tree, &mdata)) { + fi = mdata.finfo; + } + } else { + // Find the finfo that corresponds to our byte. + fi = proto_find_field_from_offset(cap_file_->edt->tree, cap_file_->search_pos, + cap_file_->edt->tvb); + } + + if (fi && proto_tree_) { + proto_tree_->selectField(fi); + } + } } void PacketList::contextMenuEvent(QContextMenuEvent *event) diff --git a/ui/qt/proto_tree.cpp b/ui/qt/proto_tree.cpp index f4ecdc86d7..22e411fe88 100644 --- a/ui/qt/proto_tree.cpp +++ b/ui/qt/proto_tree.cpp @@ -545,6 +545,19 @@ void ProtoTree::itemDoubleClick(QTreeWidgetItem *item, int) { } } +void ProtoTree::selectField(field_info *fi) +{ + QTreeWidgetItemIterator iter(this); + while (*iter) { + if (fi == (*iter)->data(0, Qt::UserRole).value<field_info *>()) { + setCurrentItem(*iter); + scrollToItem(*iter); + break; + } + iter++; + } +} + /* * Editor modelines * diff --git a/ui/qt/proto_tree.h b/ui/qt/proto_tree.h index 298782cf36..a97f69c6a5 100644 --- a/ui/qt/proto_tree.h +++ b/ui/qt/proto_tree.h @@ -42,6 +42,7 @@ public: void fillProtocolTree(proto_tree *protocol_tree); void emitRelatedFrame(int related_frame, ft_framenum_type_t framenum_type = FT_FRAMENUM_NONE); void goToField(int hf_id); + void selectField(field_info *fi); void closeContextMenu(); void clear(); |