D8106: Make sure there's not a path within the filename

[Albert Vaca Cintora]

albertvaka requested changes to this revision.
albertvaka added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> shareplugin.cpp:65
>  }
>  
> +QString cleanFilename(const QString &filename)

Make this function static. Also, I think you want to return filename instead of QString(). If I understand correctly, if there is no '/' at all in the filename, now it returns empty string.

REPOSITORY
  R224 KDE Connect

REVISION DETAIL
  https://phabricator.kde.org/D8106

To: apol, #kde_connect, aacid, albertvaka
Cc: albertvaka
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20171003/b272dffd/attachment.html>