[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

Anders Broman  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|IN_PROGRESS |RESOLVED

--- Comment #8 from Anders Broman  ---
No response, assume fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #7 from Jaap Keuter  ---
(In reply to Guy Harris from comment #1)

> Now, if I run a recent Wireshark build from the master branch, pop up the
> About dialog, and select Folders, I see the folders in question.  If I then
> close the dialog and enable OID name resolution, I get an *immediate* crash
> with:
> [STACK TRACE]

Is this problem averted now?

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #6 from Gerrit Code Review  ---
Change 26555 merged by Anders Broman:
OID name resolv: Fix MIB/PIB path presentations

https://code.wireshark.org/review/26555

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #5 from Gerrit Code Review  ---
Change 26555 had a related patch set uploaded by Jaap Keuter:
OID name resolv: Fix MIB/PIB path presentations

https://code.wireshark.org/review/26555

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #4 from Jaap Keuter  ---
(In reply to Guy Harris from comment #3)
> (In reply to Jaap Keuter from comment #2)
> > My scope of comment was the MIB/PIB paths. These are absent in this case. Of
> > course other folders are visible.
> 
> I.e., what you meant was:

That is an accurate representation of my observations.

(In reply to Jaap Keuter from comment #2)
> (In reply to Guy Harris from comment #1)
> > (In reply to Jaap Keuter from comment #0)
> > Now, if I run a recent Wireshark build from the master branch, pop up the
> > About dialog, and select Folders, I see the folders in question.  If I then
> > close the dialog and enable OID name resolution, I get an *immediate* crash
> > with:
> > 
> 
> That would be an alternative scenario, which I have to try out as well.

I've tried it (on my Debian platform), but failed to reproduce that behaviour.
I recon it ties into the 'wrong way' libsmi is being used in these scenario's,
something that this bug is meant to address.

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #3 from Guy Harris  ---
(In reply to Jaap Keuter from comment #2)
> My scope of comment was the MIB/PIB paths. These are absent in this case. Of
> course other folders are visible.

I.e., what you meant was:

With OID name resolution off (unchecked)
Qt: About Wireshark|Folders shows no MIB/PIB folders (ok)
GTK: About Wireshark|Folders shows no MIB/PIB folders (ok)
Tshark: -G folders shows no MIB/PIB folders (ok)

With OID name resolution on (checked)
Qt: About Wireshark|Folders shows user MIB/PIB folders twice (bug)
GTK: About Wireshark|Folders shows user MIB/PIB folders twice, all user paths
are concatenated (bug)
Tshark: -G folders shows no MIB/PIB folders (bug)

With OID name resolution off (unchecked), toggled to on (checked), without
restart
Qt: About Wireshark|Folder causes crash (bug)
GTK: About Wireshark|Folder causes crash (bug)
Tshark: N/A

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #2 from Jaap Keuter  ---
(In reply to Guy Harris from comment #1)
> (In reply to Jaap Keuter from comment #0)
> > With OID name resolution off (unchecked)
> > Qt: About Wireshark|Folders shows no folders (ok)
> > GTK: About Wireshark|Folders shows no folders (ok)
> > Tshark: -G folders shows no folders (ok)
> 
> None of those seem OK to me - there are a lot of folders that have nothing
> to do with OID name resolution, and those should be shown.
> 

My scope of comment was the MIB/PIB paths. These are absent in this case. Of
course other folders are visible.

> Now, if I run a recent Wireshark build from the master branch, pop up the
> About dialog, and select Folders, I see the folders in question.  If I then
> close the dialog and enable OID name resolution, I get an *immediate* crash
> with:
> 

That would be an alternative scenario, which I have to try out as well.

-- 
You are receiving this mail because:
You are watching all bug changes.___
Sent via:Wireshark-bugs mailing list 
Archives:https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
 mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

[Wireshark-bugs] [Bug 14539] OID name resolution, various problems

[bugzilla-daemon]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14539

--- Comment #1 from Guy Harris  ---
(In reply to Jaap Keuter from comment #0)
> With OID name resolution off (unchecked)
> Qt: About Wireshark|Folders shows no folders (ok)
> GTK: About Wireshark|Folders shows no folders (ok)
> Tshark: -G folders shows no folders (ok)

None of those seem OK to me - there are a lot of folders that have nothing to
do with OID name resolution, and those should be shown.

Now, if I run a recent Wireshark build from the master branch, pop up the About
dialog, and select Folders, I see the folders in question.  If I then close the
dialog and enable OID name resolution, I get an *immediate* crash with:

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   org.qt-project.QtCore   0x00010d83fcb4
QSortFilterProxyModel::parent(QModelIndex const&) const + 84
1   org.qt-project.QtCore   0x00010d834455
QIdentityProxyModel::parent(QModelIndex const&) const + 53
2   org.qt-project.QtCore   0x00010d816dc3
QPersistentModelIndex::parent() const + 35
3   org.qt-project.QtCore   0x00010d82a769
qt_PersistentModelIndexLessThan(QPersistentModelIndex const&,
QPersistentModelIndex const&) + 41
4   org.qt-project.QtCore   0x00010d8319bb void
std::__1::__stable_sort_move(QPersistentModelIndex*,
QPersistentModelIndex*, bool (*&)(QPersistentModelIndex const&,
QPersistentModelIndex const&),
std::__1::iterator_traits::difference_type,
std::__1::iterator_traits::value_type*) + 59
5   org.qt-project.QtCore   0x00010d831810 void
std::__1::__stable_sort(QPersistentModelIndex*,
QPersistentModelIndex*, bool (*&)(QPersistentModelIndex const&,
QPersistentModelIndex const&),
std::__1::iterator_traits::difference_type,
std::__1::iterator_traits::value_type*, long) + 480
6   org.qt-project.QtCore   0x00010d82a4cd
QItemSelectionModelPrivate::_q_layoutChanged(QList
const&, QAbstractItemModel::LayoutChangeHint) + 1853
7   org.qt-project.QtCore   0x00010d82eb40
QItemSelectionModel::qt_static_metacall(QObject*, QMetaObject::Call, int,
void**) + 1888
8   org.qt-project.QtCore   0x00010d899b82
QMetaObject::activate(QObject*, int, int, void**) + 2994
9   org.qt-project.QtCore   0x00010d91a263
QAbstractItemModel::layoutChanged(QList const&,
QAbstractItemModel::LayoutChangeHint) + 67
10  org.qt-project.QtCore   0x00010d83590c
QIdentityProxyModelPrivate::_q_sourceLayoutChanged(QList
const&, QAbstractItemModel::LayoutChangeHint) + 1036
11  org.qt-project.QtCore   0x00010d835ede
QIdentityProxyModel::qt_static_metacall(QObject*, QMetaObject::Call, int,
void**) + 878
12  org.qt-project.QtCore   0x00010d899b82
QMetaObject::activate(QObject*, int, int, void**) + 2994
13  org.qt-project.QtCore   0x00010d91a263
QAbstractItemModel::layoutChanged(QList const&,
QAbstractItemModel::LayoutChangeHint) + 67
14  org.qt-project.QtCore   0x00010d84175b
QSortFilterProxyModel::invalidate() + 187
15  org.wireshark.Wireshark 0x0001063cac18
InterfaceSortFilterModel::resetPreferenceData() + 440 (qstring.h:1053)
16  org.qt-project.QtCore   0x00010d899b82
QMetaObject::activate(QObject*, int, int, void**) + 2994
17  org.wireshark.Wireshark 0x0001063177b8
WiresharkApplication::flushAppSignals() + 168 (qlist.h:106)
18  org.wireshark.Wireshark 0x0001062239bb
MainWindow::showPreferencesDialog(QString) + 171
19  org.wireshark.Wireshark 0x000106223a87
MainWindow::on_actionEditPreferences_triggered() + 71 (qstring.h:1053)
20  org.wireshark.Wireshark 0x000106431312
MainWindow::qt_metacall(QMetaObject::Call, int, void**) + 82
21  org.qt-project.QtCore   0x00010d899aa9
QMetaObject::activate(QObject*, int, int, void**) + 2777
22  org.qt-project.QtWidgets0x000106a8c7ef
QAction::activate(QAction::ActionEvent) + 271
23  org.qt-project.QtWidgets0x000106a8ccf5
QAction::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) + 565
24  org.qt-project.QtCore   0x00010d899b82
QMetaObject::activate(QObject*, int, int, void**) + 2994
25  org.qt-project.QtCore   0x00010d892753
QObject::event(QEvent*) + 755
26  org.qt-project.QtWidgets0x000106a956fb
QApplicationPrivate::notify_helper(QObject*, QEvent*) + 251
27  org.qt-project.QtWidgets0x000106a98ade
QApplication::notify(QObject*, QEvent*) + 8318
28  org.qt-project.QtCore   0x00010d869e8b
QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) + 971
29  libqcocoa.dylib 0x00010e31408e
QCocoaEventDispatcherPrivate::processPostedEvents() + 190
30