meven added a comment.
In D25432#565595 <https://phabricator.kde.org/D25432#565595>, @trufanov wrote: > Hmm, I've just `git grep` the repo and found another place that is most probably affected by the same problem in FTP slave; > https://github.com/KDE/kio/blob/d03551cd0367ac7760224c56bda943c937ba5dac/src/ioslaves/ftp/ftp.cpp#L320 > > Could you check out if it's a QVariant with QByteArray type and fix it on your own? > > Also now I think the code would be more self-descriptionly if I would use QByteArray() instead of QString() as a default value for mapConfig().value() in my fix. > So if the second place is a bug and you think the same about default values feel free to improve my commit in HTTP with your fix too. Good point this needs to be fixed as well since mapConfig QVariant will always be a QByteArray QVariant : https://github.com/KDE/kio/blob/f2a3a78972b2674269b2179a8866ac5480f7c51e/src/core/slavebase.cpp#L1199 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D25432 To: trufanov, meven, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
