Skip to content
This repository has been archived by the owner on Sep 1, 2021. It is now read-only.

Crash when adding atoms to a molecule. #179

Closed
cryos opened this issue Mar 5, 2008 · 6 comments
Closed

Crash when adding atoms to a molecule. #179

cryos opened this issue Mar 5, 2008 · 6 comments

Comments

@cryos
Copy link
Owner

cryos commented Mar 5, 2008

gcc-Version 4.2.2 20071128
Qt 4.3.4
OpenBabel from today
Avogadro r1157

I am opening a file (a .cml file from out testfiles, doesn't matter which). After adding atoms at random I am getting this crash, usually after the second atom is added.

100% reproducable.

(gdb) bt
#0 0xffffe410 in __kernel_vsyscall ()
#1 0xb6e455f5 in raise () from /lib/i686/libc.so.6
#2 0xb6e47041 in abort () from /lib/i686/libc.so.6
#3 0xb6e3e70e in __assert_fail () from /lib/i686/libc.so.6
#4 0xb719e459 in Avogadro::PrimitiveItemModel::removePrimitive () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#5 0xb719e802 in Avogadro::PrimitiveItemModel::qt_metacall () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#6 0xb730d447 in QMetaObject::activate () from /usr/lib/qt4/lib/libQtCore.so.4
#7 0xb730dfb5 in QMetaObject::activate () from /usr/lib/qt4/lib/libQtCore.so.4
#8 0xb71cb190 in Avogadro::Molecule::primitiveRemoved () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#9 0xb71cbf08 in Avogadro::Molecule::DestroyBond () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#10 0xb7e28358 in OpenBabel::OBMol::DeleteBond (this=0x82a83dc, bond=0x82cfadc) at mol.cpp:2700
#11 0xb7e29ad1 in OpenBabel::OBMol::DeleteHydrogen (this=0x82a83dc, atom=0x82ce594) at mol.cpp:1781
#12 0xb7e2a421 in OpenBabel::OBMol::DeleteHydrogens (this=0x82a83dc, atom=0x82cdb74) at mol.cpp:1757
#13 0xb5fb47eb in Avogadro::AddBondDrawCommand::redo () from /home/kde4/kde/build/avo/libavogadro/src/tools/libdrawtool.so
#14 0xb7199956 in Avogadro::UndoSequence::redo () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#15 0xb7be3517 in QUndoStack::push () from /usr/lib/qt4/lib/libQtGui.so.4
#16 0xb71a9149 in Avogadro::GLWidget::mouseReleaseEvent () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#17 0xb77359f4 in QWidget::event () from /usr/lib/qt4/lib/libQtGui.so.4
#18 0xb7d6467d in QGLWidget::event () from /usr/lib/qt4/lib/libQtOpenGL.so.4
#19 0xb71a9270 in Avogadro::GLWidget::event () from /home/kde4/kde/build/avo/libavogadro/src/libavogadro.so.0
#20 0xb76ef1c2 in QApplicationPrivate::notify_helper () from /usr/lib/qt4/lib/libQtGui.so.4
#21 0xb76f5e18 in QApplication::notify () from /usr/lib/qt4/lib/libQtGui.so.4
#22 0xb72fb378 in QCoreApplication::notifyInternal () from /usr/lib/qt4/lib/libQtCore.so.4
#23 0xb774845d in QTreeView::scrollTo () from /usr/lib/qt4/lib/libQtGui.so.4
#24 0xb7747b5c in QApplication::x11ProcessEvent () from /usr/lib/qt4/lib/libQtGui.so.4
#25 0xb776b91a in QTreeView::scrollTo () from /usr/lib/qt4/lib/libQtGui.so.4
#26 0xb72fa7e3 in QEventLoop::processEvents () from /usr/lib/qt4/lib/libQtCore.so.4
#27 0xb72fa8ea in QEventLoop::exec () from /usr/lib/qt4/lib/libQtCore.so.4
#28 0xb72fcc46 in QCoreApplication::exec () from /usr/lib/qt4/lib/libQtCore.so.4
#29 0xb76ee884 in QApplication::exec () from /usr/lib/qt4/lib/libQtGui.so.4
#30 0x08071506 in main ()

Reported by: @cniehaus

@cryos
Copy link
Owner Author

cryos commented Mar 5, 2008

Logged In: YES
user_id=1500648
Originator: NO

is this with "auto add-hydrogens" enabled?

Original comment by: @milkypostman

@cryos
Copy link
Owner Author

cryos commented Mar 10, 2008

Logged In: YES
user_id=684210
Originator: YES

Is this the same bug?

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0xb6d236d0 (LWP 15676)]
0xb71185c3 in Avogadro::Primitive::type () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
(gdb) bt
#0 0xb71185c3 in Avogadro::Primitive::type () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#1 0xb70eeab4 in Avogadro::PrimitiveItemModel::parent () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#2 0xb72924ae in QAbstractItemModelPrivate::rowsAboutToBeRemoved () from /usr/lib/libQtCore.so.4
#3 0xb7292680 in QAbstractItemModel::beginRemoveRows () from /usr/lib/libQtCore.so.4
#4 0xb70ef580 in Avogadro::PrimitiveItemModel::engineChanged () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#5 0xb70efacb in Avogadro::PrimitiveItemModel::qt_metacall () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#6 0xb72a899a in QMetaObject::activate () from /usr/lib/libQtCore.so.4
#7 0xb72a8de2 in QMetaObject::activate () from /usr/lib/libQtCore.so.4
#8 0xb711c251 in Avogadro::Engine::changed () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#9 0xb711c90c in Avogadro::Engine::removePrimitive () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#10 0xb67e8986 in Avogadro::OrbitalEngine::removePrimitive () from /home/kde-devel/kde/build/avo-git/libavogadro/src/engines/liborbitalengine.so
#11 0xb70fda71 in Avogadro::GLWidget::removePrimitive () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#12 0xb710016e in Avogadro::GLWidget::qt_metacall () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#13 0xb72a899a in QMetaObject::activate () from /usr/lib/libQtCore.so.4
#14 0xb72a8de2 in QMetaObject::activate () from /usr/lib/libQtCore.so.4
#15 0xb7118860 in Avogadro::Molecule::primitiveRemoved () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#16 0xb71195d8 in Avogadro::Molecule::DestroyBond () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#17 0xb7e2df88 in OpenBabel::OBMol::DeleteBond (this=0x83648dc, bond=0x833dddc) at mol.cpp:2684
#18 0xb7e2f9b2 in OpenBabel::OBMol::DeleteHydrogen (this=0x83648dc, atom=0x833dd64) at mol.cpp:1765
#19 0xb7e30301 in OpenBabel::OBMol::DeleteHydrogens (this=0x83648dc, atom=0x852b4ec) at mol.cpp:1741
#20 0xb698e7ab in Avogadro::AddBondDrawCommand::redo () from /home/kde-devel/kde/build/avo-git/libavogadro/src/tools/libdrawtool.so
#21 0xb7c35ce8 in QUndoStack::push () from /usr/lib/libQtGui.so.4
#22 0xb710043f in Avogadro::GLWidget::mouseReleaseEvent () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#23 0xb7741cca in QWidget::event () from /usr/lib/libQtGui.so.4
#24 0xb7d223c2 in QGLWidget::event () from /usr/lib/libQtOpenGL.so.4
#25 0xb710055a in Avogadro::GLWidget::event () from /home/kde-devel/kde/build/avo-git/libavogadro/src/libavogadro.so.0
#26 0xb76f2bec in QApplicationPrivate::notify_helper () from /usr/lib/libQtGui.so.4
#27 0xb76fa195 in QApplication::notify () from /usr/lib/libQtGui.so.4
#28 0xb7298429 in QCoreApplication::notifyInternal () from /usr/lib/libQtCore.so.4
#29 0xb76f92d0 in QApplicationPrivate::sendMouseEvent () from /usr/lib/libQtGui.so.4
#30 0xb775496f in QETWidget::translateMouseEvent () from /usr/lib/libQtGui.so.4
#31 0xb775403b in QApplication::x11ProcessEvent () from /usr/lib/libQtGui.so.4
#32 0xb777851c in QEventDispatcherX11::processEvents () from /usr/lib/libQtGui.so.4
#33 0xb72976bd in QEventLoop::processEvents () from /usr/lib/libQtCore.so.4
#34 0xb7297851 in QEventLoop::exec () from /usr/lib/libQtCore.so.4
#35 0xb729999a in QCoreApplication::exec () from /usr/lib/libQtCore.so.4
#36 0xb76f22e7 in QApplication::exec () from /usr/lib/libQtGui.so.4
#37 0x0808b38e in main ()

Original comment by: @cniehaus

@cryos
Copy link
Owner Author

cryos commented Mar 10, 2008

Logged In: YES
user_id=684210
Originator: YES

is this with "auto add-hydrogens" enabled?

Do you mean "Adjust Hydrogens"? Yes, that option is active.

Original comment by: @cniehaus

@cryos
Copy link
Owner Author

cryos commented May 12, 2008

Logged In: YES
user_id=21420
Originator: NO

Does this bug still exist in the current trunk?

Original comment by: @ghutchis

@cryos
Copy link
Owner Author

cryos commented May 13, 2008

Logged In: YES
user_id=21420
Originator: NO

I believe this is fixed in the trunk. I'm marking this "fixed" pending confirmation from Carsten. If it isn't fixed, please speak up Carsten so we can re-open and re-visit the bug.

Original comment by: @ghutchis

@cryos
Copy link
Owner Author

cryos commented May 28, 2008

Logged In: YES
user_id=1312539
Originator: NO

This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

Original comment by: sf-robot

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants