Review Request 123222: Fix assert by not calling onto an empty path for setOfflineWebApplicationCachePath()

Aleix Pol Gonzalez aleixpol at kde.org
Sat Apr 18 00:09:22 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123222/
-----------------------------------------------------------

(Updated April 18, 2015, 12:09 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 18e151f5c34cb7dabdc6c777cb79a8694744cab4 by Aleix Pol to branch master.


Repository: libksysguard


Description
-------

If we pass an empty string, we end up asserting here:
qt5/qtwebkit/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp:372

#0  0x00007fffed178512 in WTFCrash () at /home/kde-devel/frameworks/qt5/qtwebkit/Source/WTF/wtf/Assertions.cpp:345
#1  0x00007fffec78e161 in WebCore::ApplicationCacheStorage::setCacheDirectory (this=0xa07a70, cacheDirectory=...) at /home/kde-devel/frameworks/qt5/qtwebkit/Source/WebCore/loader/appcache/ApplicationCacheStorage.cpp:372
#2  0x00007fffeb5085c5 in QWebSettings::setOfflineWebApplicationCachePath (path="") at /home/kde-devel/frameworks/qt5/qtwebkit/Source/WebKit/qt/Api/qwebsettings.cpp:1142
#3  0x00007ffff79087bd in ScriptingHtmlDialog::ScriptingHtmlDialog (this=0x969720, parent=0x744080) at /home/kde-devel/frameworks/libksysguard/processui/scripting.cpp:64

So we just don't set an empty string.


Diffs
-----

  processui/scripting.cpp a0efd69 

Diff: https://git.reviewboard.kde.org/r/123222/diff/


Testing
-------

Now I don't get a crash anymore.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150418/4071f621/attachment.html>


More information about the Plasma-devel mailing list