Revert unnecessary std::invalid_argument handling

WE2-479

Signed-off-by: Mart Somermaa <[email protected]>

Author: mrts

src/controller/controller.cpp

@@ -101,19 +101,6 @@ void Controller::run()
 
         startCommandExecution();
 
-    } catch (const std::invalid_argument& exc) {
-        if (isInStdinMode) {
-            // Pass invalid argument message to the caller just in case it may be interested
-            // The result will be {"invalid-argument" : message}
-            // Command parameter is only used if exception will be raised during json creation
-            writeResponseToStdOut(isInStdinMode, {{QStringLiteral("invalid-argument"), exc.what()}},
-                                  "invalid-argument");
-            // Exit directly here
-            disposeUI();
-            exit();
-            return;
-        }
-        onCriticalFailure(exc.what());
     } catch (const std::exception& error) {
         onCriticalFailure(error.what());
     }

src/ui/webeiddialog.cpp

@@ -292,9 +292,9 @@ QString WebEidDialog::getPin()
     // QString uses QAtomicPointer internally and is thread-safe.
     // There should be only single reference and this is transferred to the caller for safety
     QString ret = pin;
-	// Cannot use eraseData because we have to force Qt copy-on-write
-	pin.fill('\0');
-	pin.clear();
+    // Cannot use eraseData because we have to force Qt copy-on-write
+    pin.fill('\0');
+    pin.clear();
     return ret;
 }