diff options
Diffstat (limited to 'kde-apps/kio-extras/files/kio-extras-17.04.3-mtp.patch')
-rw-r--r-- | kde-apps/kio-extras/files/kio-extras-17.04.3-mtp.patch | 87 |
1 files changed, 0 insertions, 87 deletions
diff --git a/kde-apps/kio-extras/files/kio-extras-17.04.3-mtp.patch b/kde-apps/kio-extras/files/kio-extras-17.04.3-mtp.patch deleted file mode 100644 index 04aff7dcdbd7..000000000000 --- a/kde-apps/kio-extras/files/kio-extras-17.04.3-mtp.patch +++ /dev/null @@ -1,87 +0,0 @@ -From bdd0e54ee303559106d2ac746f9a3d167bd89146 Mon Sep 17 00:00:00 2001 -From: Kevin Funk <kfunk@kde.org> -Date: Tue, 11 Jul 2017 03:03:21 +0200 -Subject: mtp: Simplify code a bit - -Only one branch can be taken per call, make this clear by using -if/else-if instead of a series of ifs. ---- - mtp/kio_mtp.cpp | 9 ++------- - 1 file changed, 2 insertions(+), 7 deletions(-) - -diff --git a/mtp/kio_mtp.cpp b/mtp/kio_mtp.cpp -index 0532820..9d8585f 100644 ---- a/mtp/kio_mtp.cpp -+++ b/mtp/kio_mtp.cpp -@@ -593,15 +593,12 @@ void MTPSlave::copy(const QUrl &src, const QUrl &dest, int, JobFlags flags) - { - qCDebug(LOG_KIO_MTP) << src.path() << dest.path(); - -- // mtp:/// to mtp:/// - if (src.scheme() == QLatin1String("mtp") && dest.scheme() == QLatin1String("mtp")) { - qCDebug(LOG_KIO_MTP) << "Copy on device: Not supported"; - // MTP doesn't support moving files directly on the device, so we have to download and then upload... - - error(ERR_UNSUPPORTED_ACTION, i18n("Cannot copy/move files on the device itself")); -- } -- // file:/// tp mtp:/// -- if (src.scheme() == QLatin1String("file") && dest.scheme() == QLatin1String("mtp")) { -+ } else if (src.scheme() == QLatin1String("file") && dest.scheme() == QLatin1String("mtp")) { - int check = checkUrl(dest); - switch (check) { - case 0: -@@ -680,9 +677,7 @@ void MTPSlave::copy(const QUrl &src, const QUrl &dest, int, JobFlags flags) - } - - qCDebug(LOG_KIO_MTP) << "Sent file"; -- } -- // mtp:/// to file:/// -- if (src.scheme() == QLatin1String("mtp") && dest.scheme() == QLatin1String("file")) { -+ } else if (src.scheme() == QLatin1String("mtp") && dest.scheme() == QLatin1String("file")) { - int check = checkUrl(src); - switch (check) { - case 0: --- -cgit v0.11.2 -From f7b9b827ed7a23403913a4e356b0549d9deaf351 Mon Sep 17 00:00:00 2001 -From: Kevin Funk <kfunk@kde.org> -Date: Tue, 11 Jul 2017 03:04:21 +0200 -Subject: A couple of fixes for the MTP kio slave (2 patches) - -Summary: -mtp: Simplify code a bit - -Only one branch can be taken per call, make this clear by using -if/else-if instead of a series of ifs. - -mtp: Don't crash if getPath returns something invalid - -This happens when you try to copy a file from a device which just got -locked again (I think...) - -Reviewers: elvisangelaccio - -Reviewed By: elvisangelaccio - -Differential Revision: https://phabricator.kde.org/D6620 ---- - mtp/kio_mtp.cpp | 4 ++++ - 1 file changed, 4 insertions(+) - -diff --git a/mtp/kio_mtp.cpp b/mtp/kio_mtp.cpp -index 9d8585f..541268a 100644 ---- a/mtp/kio_mtp.cpp -+++ b/mtp/kio_mtp.cpp -@@ -705,6 +705,10 @@ void MTPSlave::copy(const QUrl &src, const QUrl &dest, int, JobFlags flags) - } - - QPair<void *, LIBMTP_mtpdevice_t *> pair = getPath(src.path()); -+ if (!pair.first) { -+ error(ERR_COULD_NOT_READ, src.path()); -+ return; -+ } - - LIBMTP_mtpdevice_t *device = pair.second; - LIBMTP_file_t *source = (LIBMTP_file_t *) pair.first; --- -cgit v0.11.2 |