Page MenuHomePhorge

Do not try tp load invalid scripts.
ClosedPublic

Authored by knauss on Apr 26 2016, 8:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 6, 12:05 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Unknown Object (File)
Feb 25 2024, 7:00 PM
Unknown Object (File)
Feb 9 2024, 10:15 PM
Unknown Object (File)
Jan 19 2024, 8:41 PM
Subscribers
None

Details

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 retitled this revision from to Do not try tp load invalid scripts..
knauss updated this object.
knauss added a reviewer: mollekopf.
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

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.

This revision was automatically updated to reflect the committed changes.