From e715be9787847c7fc369fe7266e9f4e82e45b7ee Mon Sep 17 00:00:00 2001 From: Albert S Date: Mon, 30 May 2022 19:39:43 +0200 Subject: [PATCH] gui: Fix dispatch of SandboxedProcessor There was an off-by-one, the SandboxedProcessor was only passed 'process', not the path to the file. No processor was found for 'process', thus 'nothingProcessor' was returned. Therefore, we never sandboxed (because we never had to process anything). The sandboxing would have failed though, because we need to launch QCoreApplication, not QApplication. The CLI was never affected. --- gui/main.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gui/main.cpp b/gui/main.cpp index 0a1c110..707b0d0 100644 --- a/gui/main.cpp +++ b/gui/main.cpp @@ -89,16 +89,16 @@ int main(int argc, char *argv[]) if(arg == "process") { Common::setupAppInfo(); - QApplication a(argc, argv); + QCoreApplication a(argc, argv); QStringList args = a.arguments(); - if(args.length() < 1) + if(args.length() < 3) { qDebug() << "Filename is required"; return 1; } - QString file = args.at(1); + QString file = args.at(2); SandboxedProcessor processor(file); return processor.process(); }