Page MenuHomePhorge
Differential D142

Do not try tp load invalid scripts.
ClosedPublic
Actions

Authored by • knauss on Apr 26 2016, 8:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 27, 8:53 PM
Unknown Object (File)
Mon, Oct 14, 8:25 AM
Unknown Object (File)
Mon, Oct 14, 8:25 AM
Unknown Object (File)
Mon, Oct 14, 4:39 AM
Unknown Object (File)
Mon, Oct 14, 4:39 AM
Unknown Object (File)
Thu, Oct 10, 10:54 AM
Unknown Object (File)
Thu, Oct 10, 10:54 AM
Unknown Object (File)
Thu, Oct 10, 10:54 AM
View All Files
Subscribers
None

Details

Reviewers
mollekopf
Maniphest Tasks
T1183: Sieve filter is making Kontact segfault
Commits
rKP406e300d627d: Do not try tp load invalid scripts.
rKPe485c87e0fc2: Do not try tp load invalid scripts.
Summary

It can happen, that if the server supports KEP:14, the USER file is the only
file that exists and that it is empty. In case we are at the end of the list
of avaialbe scripts, we should not try to load a next element from the list.

There is no vacation script in KEP:14 special files

Diff Detail

Repository
rKP kdepim
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• knauss updated this revision to Diff 292.Apr 26 2016, 8:06 PM
• knauss retitled this revision from to Do not try tp load invalid scripts..
• knauss updated this object.
• knauss added a reviewer: mollekopf.
• knauss added a task: T1183: Sieve filter is making Kontact segfault.
mollekopf accepted this revision.Apr 26 2016, 10:06 PM
mollekopf edited edge metadata.

Does this fix the crash as well?

This revision is now accepted and ready to land.Apr 26 2016, 10:06 PM
• knauss added a comment.Apr 26 2016, 10:15 PM

well i can't tell for sure, because I can't trigger the crash in

#5 0x00007f593c9672b0 in KUrl::setFileName(QString const&) () from
/usr/lib/libkdecore.so.5

but I searched for occurrence inside sive handling and this is the only occurence that makes sense, that it breaks here. For me it crashes with endof list (line 173) because if isLastScript() is true than mScriptPos it too high.

So in the end it - i think it makes sense to push the fix to genua and they have to test if it fix the issue.

Closed by commit rKPe485c87e0fc2: Do not try tp load invalid scripts. (authored by Sandro Knauß <sknauss@kde.org>). · Explain WhyApr 26 2016, 10:16 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
libksieve/
ksieveui/
vacation/
1 line

Diff 293

View Options

libksieve/ksieveui/vacation/vacationcheckjob.cpp

Loading...