From 4da6fd05a23487d51129305db9cca0e0bc09a04d Mon Sep 17 00:00:00 2001 From: lupoDharkael Date: Tue, 3 Oct 2017 17:19:41 +0200 Subject: [PATCH] Fix parser segfault Passing an argument with a double dashed prefix was causing a crash. 'flameshot --gui' instead of 'flameshot gui' now it's handled correctly. --- src/cli/commandlineparser.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cli/commandlineparser.cpp b/src/cli/commandlineparser.cpp index 78b663ad..d9dfba1e 100644 --- a/src/cli/commandlineparser.cpp +++ b/src/cli/commandlineparser.cpp @@ -157,7 +157,7 @@ bool CommandLineParser::processOptions(const QStringList &args, break; } } - if (optionIt == actualNode->options.end()) { + if (optionIt == endIt) { QString argName = actualNode->argument.name(); if (argName.isEmpty()) { argName = qApp->applicationName();