Comment 6 for bug 1292237

The crash occurred when convertActionText() was passed a nullptr from convertToEntry(), whose nullptr 'action' argument came from ItemStore::search looping over its m_items and passing in it->value()->action().

It looks like Item::action()'s implementation explicitly returns nullptr in extreme cases, so we should probably add a nullptr safeguard to convertActionText() and fix convertToEntry() to avoid empty strings between the "||" delimiters.