Bug 495848

Created attachment 175559 [details]
New crash information added by DrKonqi

DrKonqi auto-attaching complete backtrace.

*** Bug 499153 has been marked as a duplicate of this bug. ***

*** Bug 500188 has been marked as a duplicate of this bug. ***

I'm unable to reproduce these crashes

I need hints on how to reproduce the crashes or other info to help me debug
I'm testing with KOrganizer 6.3, if that matters

*** Bug 492807 has been marked as a duplicate of this bug. ***

can confirm this on latest user korg package

(In reply to Allen Winter from comment #4)
> I'm unable to reproduce these crashes
> 
> I need hints on how to reproduce the crashes or other info to help me debug
> I'm testing with KOrganizer 6.3, if that matters

Hi,

Some information from GDB.

The cached device pixel ratio value was stale on window update.  Please file a QTBUG which explains how to reproduce.             
org.kde.pim.incidenceeditor: Faulty editor was  "IncidenceAttendee"
org.kde.pim.incidenceeditor: Incidence  "643f089a-6a76-4691-8093-cd89d0694033"
ASSERT failure in load: "editor shouldn't be dirty", file ./src/combinedincidenceeditor.cpp, line 93

Thread 1 "korganizer" received signal SIGABRT, Aborted.
Download failed: Argument invalide.  Continuing without source file ./nptl/./nptl/pthread_kill.c.
__pthread_kill_implementation (no_tid=0, signo=6, threadid=<optimized out>) at ./nptl/pthread_kill.c:44
warning: 44     ./nptl/pthread_kill.c: Aucun fichier ou dossier de ce nom

yes, but I'm unable to reproduce.  

are you using neon too? 

which KDE versions and Qt version?

(In reply to Allen Winter from comment #8)
> yes, but I'm unable to reproduce.  
> 
> are you using neon too? 
> 
> which KDE versions and Qt version?

I'm using KDE Neon 6.3.

KDE plasma is 6.3, KDE frameworks 6.12.0 and Qt 6.8.2.

I use KDE Neon since begin of 2024 and this bug exist for me since the beginning (or may be after the last LTS update).(In reply to Allen Winter from comment #8)
> yes, but I'm unable to reproduce.  
> 
> are you using neon too? 
> 
> which KDE versions and Qt version?

I'm using KDE Neon 6.3.

KDE plasma is 6.3, KDE frameworks 6.12.0 and Qt 6.8.2.

I use KDE Neon since begin of 2024 and this bug exist for me since the beginning (or may be after the last LTS update).

In this method,  "editor->load(incidence);" is alway dirty !!!

void CombinedIncidenceEditor::load(const KCalendarCore::Incidence::Ptr &incidence)
{
    mLoadedIncidence = incidence;
    for (IncidenceEditor *editor : std::as_const(mCombinedEditors)) {
        // load() may fire dirtyStatusChanged(), reset mDirtyEditorCount to make sure
        // we don't end up with an invalid dirty count.
        editor->blockSignals(true);
        editor->load(incidence);
        editor->blockSignals(false);
 
        if (editor->isDirty()) {
            // We are going to crash due to assert. Print some useful info before crashing.
            qCWarning(INCIDENCEEDITOR_LOG) << "Faulty editor was " << editor->objectName();
            qCWarning(INCIDENCEEDITOR_LOG) << "Incidence " << (incidence ? incidence->uid() : QStringLiteral("null"));
 
            editor->printDebugInfo();
 
            Q_ASSERT_X(false, "load", "editor shouldn't be dirty");
        }
    }
 
    mWasDirty = false;
    mDirtyEditorCount = 0;
    Q_EMIT dirtyStatusChanged(false);
}

a couple months later...

since KOrganizer 6.4 are you still experiencing this crash?

only Neon users reported this one

(In reply to Allen Winter from comment #10)
> a couple months later...
> 
> since KOrganizer 6.4 are you still experiencing this crash?
> 
> only Neon users reported this one

Hi

Actually, I use Korganizer 6.4.2 and still same crash when I want to add event. I checked this crash with GDB and find that crash appears in below function "if (editor->isDirty())" but code is too complicated for me to follow why this append.


void CombinedIncidenceEditor::load(const KCalendarCore::Incidence::Ptr &incidence)
{
    mLoadedIncidence = incidence;
    for (IncidenceEditor *editor : std::as_const(mCombinedEditors)) {
        // load() may fire dirtyStatusChanged(), reset mDirtyEditorCount to make sure
        // we don't end up with an invalid dirty count.
        editor->blockSignals(true);
        editor->load(incidence);
        editor->blockSignals(false);
 
        if (editor->isDirty()) {
            // We are going to crash due to assert. Print some useful info before crashing.
            qCWarning(INCIDENCEEDITOR_LOG) << "Faulty editor was " << editor->objectName();
            qCWarning(INCIDENCEEDITOR_LOG) << "Incidence " << (incidence ? incidence->uid() : QStringLiteral("null"));
 
            editor->printDebugInfo();
 
            Q_ASSERT_X(false, "load", "editor shouldn't be dirty");
        }
    }

I installed the KDE Neon ISO into a virtual machine (virtualbox).
Then I tried to start Kontact and noticed it wasn't included with Neon.
Then I started discover and installed Kontact 

start Kontact. switch to KOrganizer part
Now I add my Google calendar

and then Add New Event from the menu.
and I encounter the same assert as reported by  emi.caruso@free.fr , et.al

somehow the Qt signal block is not working here

*** Bug 505086 has been marked as a duplicate of this bug. ***

A possibly relevant merge request was started @ https://invent.kde.org/pim/incidenceeditor/-/merge_requests/69

*** Bug 506417 has been marked as a duplicate of this bug. ***

*** Bug 485950 has been marked as a duplicate of this bug. ***

*** Bug 508427 has been marked as a duplicate of this bug. ***