Skip to content

Don't crash on empty protocol requested

ivan tkachenko requested to merge work/ratijas/empty-url into master

Be more fault-tolerant to a dummy input from client code, and don't crash when an empty url/protocol is requested. There is already an infrastructure for handling null return value in place, e.g. in the KProtocolInfo::extraFields method, since it can not be guaranteed that a requested protocol really exists.

This patch prevents icon applet from crashing when attempted to be configured when launched via plasmoidviewer.

By the way, this is how currently the file properties dialog looks like for such an empty url in plasmoidviewer:

image

Still better than SEGFAULT Core dumped

Merge request reports