Any text typed into Brewday instructions display box causes crash on Mac

Bug #1145736 reported by Peter Buelow
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Brewtarget
Fix Released
Medium
Mik Firestone

Bug Description

Any text typed into the largest text box in the Brewday notes will cause an instant segfault with the following crash dump.

5 QtCore 0x0000000105674a2a qFatal(char const*, ...) + 170
6 Brewtarget 0x00000001023adf10 QList<Instruction*>::operator[](int) + 96 (qlist.h:477)
7 Brewtarget 0x00000001023afefb BrewDayScrollWidget::saveInstruction() + 75 (BrewDayScrollWidget.cpp:53)
8 Brewtarget 0x00000001025931f1 BrewDayScrollWidget::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) + 689 (moc_BrewDayScrollWidget.cxx:74)
9 QtCore 0x00000001057b1fae QMetaObject::activate(QObject*, QMetaObject const*, int, void**) + 1566

Revision history for this message
Mik Firestone (mikfire) wrote :

I'm seeing the same thing on my linux build. Looks like the same core dump too.

Revision history for this message
Mik Firestone (mikfire) wrote :

This will be fixed as soon as I get done making the database writes more sane. What's going on is that *any* input into that text edit box generates a textChanged() signal, which in turn calls the saveInstructions(). This results in a long chain, that ends up calling setInstructions() with a null string, and we dump core.

Changed in brewtarget:
importance: Undecided → Medium
status: New → Triaged
assignee: nobody → Mik Firestone (mikfire)
Revision history for this message
Philip (rocketman768) wrote :

Does this still happen in upstream/master?

Changed in brewtarget:
status: Triaged → Incomplete
Revision history for this message
Mik Firestone (mikfire) wrote :

This was closed back with merge request 21 or 22 -- I'm having a hard time finding exactly the right commit. But it was coincidental with my implementing BtTextWidgets.

Changed in brewtarget:
status: Incomplete → Fix Committed
Mik Firestone (mikfire)
Changed in brewtarget:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.