aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJean-Baptiste Mardelle <[email protected]>2015-12-05 14:30:42 +0100
committerJean-Baptiste Mardelle <[email protected]>2015-12-05 14:30:42 +0100
commit258cb55ce76ef9817044e9161eb8bab1293f19ee (patch)
treecc7d0c68bd3daeb92b4f2b8ea93c2b67769dbc8e
parent8c44cb598b4f0916e564a4f04ddfa974f2547d37 (diff)
Don't start a process with an url in it, use more secure argument form
-rw-r--r--src/utils/freesound.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/utils/freesound.cpp b/src/utils/freesound.cpp
index e574978..b44807b 100644
--- a/src/utils/freesound.cpp
+++ b/src/utils/freesound.cpp
@@ -253,7 +253,7 @@ void FreeSound::slotParseResults(KJob* job)
* @return
*/
bool FreeSound::startItemPreview(QListWidgetItem *item)
-{
+{
if (!item)
return false;
const QString url = m_metaInfo.value("itemPreview");
@@ -263,7 +263,7 @@ bool FreeSound::startItemPreview(QListWidgetItem *item)
if (m_previewProcess->state() != QProcess::NotRunning) {
m_previewProcess->close();
}
- m_previewProcess->start(KdenliveSettings::ffplaypath() + " " + url + " -nodisp -autoexit");
+ m_previewProcess->start(KdenliveSettings::ffplaypath(), QStringList() <<url<<"-nodisp"<<"-autoexit");
}
return true;
}