martinkostolny updated this revision to Diff 13539. martinkostolny added a comment.
Thanks for noticing all these security issues! Both issues should now be fixed. Regarding the KAuth dialog, I agree, but couldn't find a simple way of propagating additional info to the dialog. I didn't dig in KAuth code much but so far I didn't find a way. It looks like it isn't supported. KAuth::Action.setDetails(...) doesn't really add anything in the dialog. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5394?vs=13517&id=13539 REVISION DETAIL https://phabricator.kde.org/D5394 AFFECTED FILES src/buffer/katesecuretextbuffer.cpp src/buffer/katesecuretextbuffer_p.h src/buffer/katetextbuffer.cpp src/buffer/katetextbuffer.h To: martinkostolny, #ktexteditor, fvogt Cc: elvisangelaccio, aacid, ivan, lbeltrame, fvogt, apol, anthonyfieroni, cullmann, ltoscano, dhaumann, graesslin, davidedmundson, palant, kwrite-devel, dfaure, #frameworks, head7, kfunk, sars