## Summary Exit (crash?) when expanding tree - `Internetwork Packet eXchange` or `IPX Routing Information Protocol` ## Steps to reproduce Related to Network name resolution: `View->Name Resolution->Resolve Network Addresses` Disabling Network name resolution prevents the crash/allows tree expansion. ## What is the current bug behavior? (What actually happens) ## What is the expected correct behavior? (What you should see instead) ## Sample capture file [test/captures/ipx_rip.pcap](https://gitlab.com/wireshark/wireshark/-/blob/master/test/captures/ipx_rip.pcap) #1795 - FormatShowcase1.cap: "Showcase Traffic" (HTTP, other mixed traffic) in libpcap format #143 - Memory exhaustion in NCP ## Relevant logs and/or screenshots ``` libwsutil.dll!wmem_map_lookup(_wmem_map_t * map, const void * key) Line 259 (c:\Development\wireshark\wsutil\wmem\wmem_map.c:259) libwireshark.dll!ipxnet_name_lookup(_wmem_allocator_t * allocator, const unsigned int addr) Line 2129 (c:\Development\wireshark\epan\addr_resolv.c:2129) libwireshark.dll!get_ipxnet_name(_wmem_allocator_t * allocator, const unsigned int addr) Line 3401 (c:\Development\wireshark\epan\addr_resolv.c:3401) libwireshark.dll!proto_item_fill_label(field_info * fi, char * label_str) Line 9326 (c:\Development\wireshark\epan\proto.c:9326) Wireshark.exe!ProtoNode::labelText() Line 55 (c:\Development\wireshark\ui\qt\utils\proto_node.cpp:55) Wireshark.exe!ProtoTreeModel::data(const QModelIndex & index, int role) Line 98 (c:\Development\wireshark\ui\qt\models\proto_tree_model.cpp:98) Qt6Cored.dll!QAbstractItemModel::multiData(const QModelIndex & index, QModelRoleDataSpan roleDataSpan) Line 3704 (c:\Users\qt\work\qt\qtbase\src\corelib\itemmodels\qabstractitemmodel.cpp:3704) Qt6Widgetsd.dll!QModelIndex::multiData(QModelRoleDataSpan roleDataSpan) Line 532 (c:\Users\qt\work\qt\qtbase\src\corelib\itemmodels\qabstractitemmodel.h:532) Qt6Widgetsd.dll!QStyledItemDelegate::initStyleOption(QStyleOptionViewItem * option, const QModelIndex & index) Line 297 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qstyleditemdelegate.cpp:297) Qt6Widgetsd.dll!QStyledItemDelegate::sizeHint(const QStyleOptionViewItem & option, const QModelIndex & index) Line 427 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qstyleditemdelegate.cpp:427) Qt6Widgetsd.dll!QTreeView::indexRowSizeHint(const QModelIndex & index) Line 3029 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:3029) Qt6Widgetsd.dll!QTreeViewPrivate::itemHeight(int item) Line 3507 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:3507) Qt6Widgetsd.dll!QTreeViewPrivate::updateScrollBars() Line 3747 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:3747) Qt6Widgetsd.dll!QTreeView::updateGeometries() Line 2867 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:2867) Qt6Widgetsd.dll!QTreeViewPrivate::expandOrCollapseItemAtPos(const QPoint & pos) Line 1401 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:1401) Qt6Widgetsd.dll!QTreeView::mousePressEvent(QMouseEvent * event) Line 1937 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qtreeview.cpp:1937) Qt6Widgetsd.dll!QWidget::event(QEvent * event) Line 8694 (c:\Users\qt\work\qt\qtbase\src\widgets\kernel\qwidget.cpp:8694) Qt6Widgetsd.dll!QFrame::event(QEvent * e) Line 550 (c:\Users\qt\work\qt\qtbase\src\widgets\widgets\qframe.cpp:550) Qt6Widgetsd.dll!QAbstractScrollArea::viewportEvent(QEvent * e) Line 1097 (c:\Users\qt\work\qt\qtbase\src\widgets\widgets\qabstractscrollarea.cpp:1097) Qt6Widgetsd.dll!QAbstractItemView::viewportEvent(QEvent * event) Line 1793 (c:\Users\qt\work\qt\qtbase\src\widgets\itemviews\qabstractitemview.cpp:1793) ``` [epan/addr_resolv.c](https://gitlab.com/wireshark/wireshark/-/blob/master/epan/addr_resolv.c#L2129): ``` ipxnet_name_lookup(wmem_allocator_t *allocator, const guint addr) { hashipxnet_t *tp; ipxnet_t *ipxnet; tp = (hashipxnet_t *)wmem_map_lookup(ipxnet_hash_table, GUINT_TO_POINTER(addr)); ``` [wsutil/wmem/wmem_map.c](https://gitlab.com/wireshark/wireshark/-/blob/master/wsutil/wmem/wmem_map.c#L259): ``` wmem_map_lookup(wmem_map_t *map, const void *key) { wmem_map_item_t *item; /* Make sure we have a table */ if (map->table == NULL) { return NULL; } Exception has occurred: W32/0xC0000005 Unhandled exception thrown: read access violation. map was nullptr. ``` ## Build information ``` 3.6.7 (v3.6.7-0-g4a304d7ec222) Compiled (64-bit) using Microsoft Visual Studio 2019 (VC++ 14.31, build 31107), with Qt 5.15.2, with libpcap, with GLib 2.66.4, with zlib 1.2.11, with Lua 5.2.4, with GnuTLS 3.6.3 and PKCS #11 support, with Gcrypt 1.8.3, with MIT Kerberos, with MaxMind DB resolver, with nghttp2 1.44.0, with brotli, with LZ4, with Zstandard, with Snappy, with libxml2 2.9.10, with libsmi 0.4.8, with QtMultimedia, with automatic updates using WinSparkle 0.5.7, with AirPcap, with SpeexDSP (using bundled resampler), with Minizip. Running on 64-bit Windows 10 (21H1), build 19043, with Intel(R) Core(TM) i5-4300U CPU @ 1.90GHz (with SSE4.2), with 12191 MB of physical memory, with GLib 2.66.4, with Qt 5.15.2, with Npcap version 1.60, based on libpcap version 1.10.2-PRE-GIT, with c-ares 1.17.0, with GnuTLS 3.6.3, with Gcrypt 1.8.3, with nghttp2 1.44.0, with brotli 1.0.9, with LZ4 1.9.3, with Zstandard 1.4.0, without AirPcap, with light display mode, without HiDPI, with LC_TYPE=English_United States.utf8, binary plugins supported (21 loaded). ```