Fix coverity errors

WE2-1003 Signed-off-by: Raul Metsma <[email protected]>
web-eid · Jul 29, 2024 · c1192ff · c1192ff
1 parent 7e462b4
commit c1192ff
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/lib/libelectronic-id b/lib/libelectronic-id
diff --git a/src/ui/webeiddialog.cpp b/src/ui/webeiddialog.cpp
@@ -137,7 +137,8 @@ WebEidDialog::WebEidDialog(QWidget* parent) : WebEidUI(parent), ui(new Private)
             ++i;
         }
         menu->show();
-        menu->move(ui->langButton->geometry().bottomRight() - menu->geometry().topRight() + QPoint(0, 2));
+        menu->move(ui->langButton->geometry().bottomRight() - menu->geometry().topRight()
+                   + QPoint(0, 2));
         connect(langGroup, qOverload<QAbstractButton*>(&QButtonGroup::buttonClicked), menu,
                 [this, menu](QAbstractButton* action) {
                     QSettings().setValue(QStringLiteral("lang"), action->property("lang"));
@@ -330,12 +331,11 @@ void WebEidDialog::onMultipleCertificatesReady(
     case CommandType::AUTHENTICATE:
         ui->selectAnotherCertificate->disconnect();
         ui->selectAnotherCertificate->setVisible(certificateAndPinInfos.size() > 1);
-        connect(ui->selectAnotherCertificate, &QPushButton::clicked, this,
-                [this, origin, certificateAndPinInfos] {
-                    ui->pinInput->clear();
-                    onMultipleCertificatesReady(origin, certificateAndPinInfos);
-                });
-        setupOK([this, origin] {
+        connect(ui->selectAnotherCertificate, &QPushButton::clicked, this, [=] {
+            ui->pinInput->clear();
+            onMultipleCertificatesReady(origin, certificateAndPinInfos);
+        });
+        setupOK([=] {
             ui->okButton->setDisabled(true);
             // Authenticate continues with the selected certificate to onSingleCertificateReady().
             if (auto* button =
@@ -371,7 +371,7 @@ void WebEidDialog::onSingleCertificateReady(const QUrl& origin,
     switch (currentCommand) {
     case CommandType::GET_SIGNING_CERTIFICATE:
         setupCertificateAndPinInfo({certAndPin});
-        setupOK([this, certAndPin] {
+        setupOK([=] {
             ui->okButton->setDisabled(true);
             emit accepted(certAndPin);
         });
@@ -540,7 +540,7 @@ void WebEidDialog::setTrText(QWidget* label, Text text) const
 void WebEidDialog::connectOkToCachePinAndEmitSelectedCertificate(
     const CardCertificateAndPinInfo& certAndPin)
 {
-    setupOK([this, certAndPin] {
+    setupOK([=] {
         ui->pinInput->hide();
         ui->pinTitleLabel->hide();
         ui->pinErrorLabel->hide();