# HG changeset patch # User Andre Heinecke # Date 1411059191 -7200 # Node ID 2a1206932f538bbf135fa08b0c650969db4626cf # Parent b67681a61e3e472780dc6bbc2e83cf5e22f8e762 (issue107) Comment all TODO's and FIXME's with issues as documented in issue107 diff -r b67681a61e3e -r 2a1206932f53 cinst/main.c --- a/cinst/main.c Thu Sep 18 18:30:54 2014 +0200 +++ b/cinst/main.c Thu Sep 18 18:53:11 2014 +0200 @@ -211,7 +211,6 @@ int main (int argc, char **argv) { - /* TODO handle wchar arguments on Windows or do conversion dance */ char **to_install = NULL, **to_remove = NULL, **all_valid_certs = NULL; diff -r b67681a61e3e -r 2a1206932f53 cmake/FindNSS.cmake --- a/cmake/FindNSS.cmake Thu Sep 18 18:30:54 2014 +0200 +++ b/cmake/FindNSS.cmake Thu Sep 18 18:53:11 2014 +0200 @@ -7,7 +7,6 @@ # - Try to find Mozilla NSS include(FindPkgConfig) -# FIXME: maybe a minimal version would be wise... pkg_check_modules (NSS nss) if (NOT NSS_FOUND) diff -r b67681a61e3e -r 2a1206932f53 common/linuxlockfile.c --- a/common/linuxlockfile.c Thu Sep 18 18:30:54 2014 +0200 +++ b/common/linuxlockfile.c Thu Sep 18 18:53:11 2014 +0200 @@ -32,7 +32,7 @@ lk.l_len = 0; if (fcntl(fd, F_SETLK, &lk) != -1) { - /* FIXME: For extra security we should check if there is + /* FIXME (issue139): For extra security we should check if there is already a pid in the file. If so we should check in /proc/$PID if there is still a process of the same name as ours running... */ diff -r b67681a61e3e -r 2a1206932f53 common/listutil.c --- a/common/listutil.c Thu Sep 18 18:30:54 2014 +0200 +++ b/common/listutil.c Thu Sep 18 18:53:11 2014 +0200 @@ -203,14 +203,10 @@ } if (ret == READ_FILE_UNREADABLE) { - /* TODO: work with errno ? */ - /* errsv = errno; */ - /* perror("read_and_verify_list(), READ_FILE_UNREADABLE:"); */ return SeekFailed; } if (ret == READ_FILE_READ_FAILED) { - /* TODO: work with ferror() or feof() ? */ return ReadFailed; } return UnknownError; diff -r b67681a61e3e -r 2a1206932f53 common/util.c --- a/common/util.c Thu Sep 18 18:30:54 2014 +0200 +++ b/common/util.c Thu Sep 18 18:53:11 2014 +0200 @@ -761,7 +761,7 @@ if (err == ERROR_ALREADY_EXISTS) { /* Verify that the directory has the correct rights */ - // TODO + // TODO (issue138) retval = true; goto done; } diff -r b67681a61e3e -r 2a1206932f53 packaging/trustbridge.nsi --- a/packaging/trustbridge.nsi Thu Sep 18 18:30:54 2014 +0200 +++ b/packaging/trustbridge.nsi Thu Sep 18 18:53:11 2014 +0200 @@ -273,7 +273,7 @@ ExpandEnvStrings $0 %LOCALAPPDATA% nsExec::ExecToLog '"$INSTDIR\cinst.exe" "list=$0\BSI\TrustBridge\list-installed.txt" "choices=uninstall"' skip_certs: - RMDir /r "$INSTDIR" ; TODO include uninstall files + RMDir /r "$INSTDIR" ; TODO (issue137) include uninstall files ; !include "filelist-un.nsh" ; Delete "$INSTDIR\Uninstall.exe" ; RMDir "$INSTDIR" diff -r b67681a61e3e -r 2a1206932f53 ui/certificate.cpp --- a/ui/certificate.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/certificate.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -142,7 +142,6 @@ /* We read the file using Qt to avoid filename encoding problems * on Windows */ - /* TODO change qDebug errors into messageboxes */ QFile certificateFile(file_name); QByteArray fileContent; QList retval; @@ -173,7 +172,6 @@ if (ret > 0) { qDebug() << "Some certificates could not be parsed."; - /* Maybe return here? */ } x509_crt *iter = &chain; diff -r b67681a61e3e -r 2a1206932f53 ui/createinstallerdialog.cpp --- a/ui/createinstallerdialog.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/createinstallerdialog.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -435,7 +435,7 @@ if (entry.suffix() == "exe") { if (!signFile(targetPath)) { showErrorMessage(tr("Failed to sign binaries with osslsigncode.\n" - "Please check that %1 is a valid code signing certificate and that" + "Please check that %1 is a valid code signing certificate and that " "osslsigncode can be found in the PATH.").arg(mCertFile->text())); mProgress.cancel(); return NULL; diff -r b67681a61e3e -r 2a1206932f53 ui/installwrapper.cpp --- a/ui/installwrapper.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/installwrapper.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -61,8 +61,6 @@ void InstallWrapper::run() { - /* TODO: We need errorcodes here so that we can see if a user - * cancled the UAC elevation */ QTemporaryFile choicesFile; QFileInfo cinstProcInfo = getCinstProcInfo(); @@ -171,7 +169,7 @@ fclose(vres.fptr); if (retval != 0) { - /* TODO make this nicer */ + /* TODO (issue135) make this nicer */ emit error (tr("The process failed with return code. %1").arg(retval)); return; } @@ -239,7 +237,7 @@ if (installerProcess.exitCode() == 0) { qDebug() << "output: " << installerProcess.readAllStandardOutput(); } else { - /* TODO handle errors defined by errorcodes.h */ + /* TODO (issue135) handle errors defined by errorcodes.h */ qDebug() << "Installer Process returned: " << installerProcess.exitCode(); qDebug() << "output: " << installerProcess.readAllStandardOutput(); return; diff -r b67681a61e3e -r 2a1206932f53 ui/l10n/trustbridge_de_DE.ts --- a/ui/l10n/trustbridge_de_DE.ts Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/l10n/trustbridge_de_DE.ts Thu Sep 18 18:53:11 2014 +0200 @@ -165,33 +165,33 @@ Auführungsfehler: %1 - + Error monitoring process: %1 Fehlerüberwachung: %1 - + Certificate installation timed out. Zeitüberschreitung bei der Zertifikatsinstallation. - + Failed to check process status: %1 Fehler in der Prozess-Statusprüfung: %1 - + The process failed with return code. %1 Der Prozess ist fehlgeschlagen. Fehlercode: %1 - + Failed to start installer process. Fehler beim Starten des Installer-Prozess. - - + + Failed to complete installation. Fehler beim Abschließen der Installation. @@ -203,19 +203,19 @@ Neue Vorschläge für Wurzelzertifikate sind verfügbar. Klicken Sie hier zum Installieren. - - + + Check for Updates Neue Empfehlungen suchen - - + + Quit Beenden - + TrustBridge TrustBridge @@ -241,7 +241,7 @@ Empfehlungen - + Revoked certificates Abgeratene @@ -256,41 +256,41 @@ Aktualisierungen (%1/%2) - - + + Quit without saving Beenden ohne Schreiben - - + + Remove revoked certificates (%1/%2) Abgeratene Wurzelzertifikate entfernen (%1/%2) - + Trusted certificates Empfohlene Wurzelzertifikate - + The following list of trusted root certificates is managed by the BSI. The BSI validates independently the authenticity, security and actuality of these certificates. Die folgenden Wurzelzertifikate wurden bisher vom BSI zur Installation vorgeschlagen. Sie können erkennen, welche Sie bereits geschrieben haben. - + Please choose the certificates you want to trust or untrust. TrustBridge will install these certificates for your secure communication for email and internet. Legen Sie fest, ob Sie der Empfehlung ganz oder teilweise folgen möchten. TrustBridge wird die Änderungen an den Wurzelzertifikaten vornehmen. - + Trusted certificates Empfohlene Wurzelzertifikate - + Information and help Informationen @@ -317,27 +317,27 @@ Aktualisierungen einspielen - - + + Install new trusted certificates (%1/%2) Neue, empfohlene Wurzelzertifikate installieren (%1/%2) - - - - - + + + + + Show details Details einblenden - + Revoked certificates Abgeratene Wurzelzertifikate - + Certificates can be corrupted or stolen and misused in many ways. Therefore the BSI recommends to remove all revoked certificates from your system. Wurzelzertifikate können veraltet sein, korrumpiert, gestohlen oder missbraucht werden. Die Wurzelzertifikate , von denen das BSI abrät, sollten umgehend entfernt werden. @@ -350,7 +350,7 @@ Abweichend zu behandelnde Wurzelzertifikate (%1) - + Trust in your digital communication Vertrauen in Ihre digitale Kommunikation @@ -359,8 +359,8 @@ Änderungen - - + + Certificates unchanged Wurzelzertifikate unverändert @@ -377,75 +377,80 @@ Es wird empfohlen, die nachfolgenden Änderungen an Ihren Wurzelzertifikaten zu übernehmen. - + Apply changes Änderungen schreiben - + Version: Version: - - + + Show recommendations + + + + + An updated certificate list is available. Neue Vorschläge für Wurzelzertifikate sind verfügbar. - + Click here to install. Klicken Sie hier zum Installieren. - - + + An update for %1 is available. Eine Aktualisierung für %1 ist verfügbar. - + Click here to download and install the update. Hier klicken, um Download und Installation zu starten. - + Failed to create update process. - + This could be caused by not enough disk space or invalid permissions. - + Downloading update... - + Failed to check for updates: Fehler bei Updateprüfung: - + You should apply the following, recommended changes to your certificate stores: Es wird empfohlen, die nachfolgenden Änderungen an Ihren Zertifikatsspeichern vorzunehmen: - + You can apply the following, changes to your certificate stores: Sie können die nachfolgenden Änderungen an Ihren Zertifikatsspeichern vornehmen: - + There are currently no changes for your certificate stores. Es liegen keine neuen Empfehlungen vor. - - + + Install certificates again Wurzelzertifikate erneut schreiben @@ -498,7 +503,7 @@ Letzte erfolgreiche Prüfung nach Aktualisierungen: %1 - + Sucessfully checked for updates. Suche nach neuen Empfehlungen erfolgreich. @@ -509,141 +514,141 @@ Hier klicken, um Download und Installation zu starten. - + TrustBridge is a root certificate installer for Windows and GNU/Linux.<br/> TrustBridge ist ein Wurzelzertifikatsinstaller für Windows und GNU/Linux.<br/> - + The root certificate lists are managed by the German <a href="https://www.bsi.bund.de">Federal Office for Information Security (BSI)</a>.<br/><br/> Die Wurzelzertifikate werden vom <a href="https://www.bsi.bund.de">Bundesamt für Sicherheit in der Informationstechnik (BSI)</a> vorgeschlagen.<br/><br/> - + The software was developed by the companies <a href="http://www.intevation.de">Intevation GmbH</a> and <a href="http://www.dn-systems.de">DN-Systems GmbH</a>, <br> contracted by the German Federal Office for Information Security (BSI).<br/><br/> Die Software wurde von den Unternehmen <a href="http://www.intevation.de">Intevation GmbH</a> und <a href="http://www.dn-systems.de">DN-Systems GmbH</a> entwickelt, <br> beauftragt vom Bundesamt für Sicherheit in der Informationstechnik (BSI).<br/><br/> - + TrustBridge is Free Software licensed under GNU GPL v2+.<br/><br/>Copyright (C) 2014 by Bundesamt für Sicherheit in der Informationstechnik TrustBridge ist Freie Software, lizensiert unter der GNU GPL v2+.<br/><br/>(C) 2014. Die Rechte liegen beim Bundesamt für Sicherheit in der Informationstechnik. - + Show Help Hilfe anzeigen - + Proxy settings Proxy-Einstellungen - + Last update check: Letzte Suche: - + No connection with the updateserver. Keine Verbindung zum Updateserver. - + Update Aktualisieren - - - + + + Details Details - + The following unsecure certificates were revoked by the BSI. Already uninstalled certificates cannot be reinstalled. It is recommended that you select all certificates to uninstall if you still have revoked certificates installed. Von den folgenden, ehemals empfohlenen Wurzelzertifikaten, rät das BSI ab. Über diese Anwendung können sie auch nicht mehr installiert werden. Markieren Sie verbleibende Wurzelzertifikate zur Löschung, sobald Sie können. - + Version Version - + Pending changes Ausstehende Änderungen - + Manual changes (%1) Manuelle Änderungen (%1) - + Certificate list from: Zertifikatsliste vom: - + Currently installed certificate list: Aktuell installierte Zertifikatsliste: - + No certificate list installed. Keine Zertifikatsliste installiert. - - - + + + Certificate will be installed. - - + + Certifcate is not installed. - - + + Certificate is installed. - - - - - + + + + + Certificate will be removed. - - + + Certificate has not been removed. - - + + Certificate has been removed. - + Certificate will not be installed. - + Certificate will not be removed. @@ -657,18 +662,18 @@ Sie können die nachfolgenden Änderungen an Ihren Zertifikatsspeichern vornehmen. - + Pending changes (%1) Ausstehende Änderungen (%1) - + New, recommended changes (%1/%2) Neue, empfohlene Änderungen (%1/%2) - + No new recommendations Keine neuen Empfehlungen @@ -689,37 +694,37 @@ Neue empfohlene Änderungen (%1) - + Error executing update Fehler bei der Aktualisierung - + Installation with standard user account Installation mit Standardbenutzerkonto - + Windows will now ask you to confirm each root certificate modification because TrustBridge does not have the necessary privileges to install root certificates into the Windows certificate store silently. Windows wird Sie nun bitten, jede Wurzelzertifikatsänderung zu bestätigen. Grund dafür: TrustBridge besitzt nicht die nötigen Privilegien, um Wurzelzertifikate ohne Nachfrage in den Windows-Zertifikatsspeicher zu installieren. - + Installing certificates... Wurzelzertifikate werden geändert... - + Error! Fehler! - + Failed to find the manual Fehler beim Finden des Handbuchs - + TrustBridge error TrustBridge Fehler @@ -792,28 +797,28 @@ Automatische Aktualisierungsprüfung von TrustBridge. - + TrustBridge-Updater Used as filename for the updater. Only use ASCII please. - + Hide details Details ausblenden - + Less Weniger - + Show details Details einblenden - + Details Details @@ -821,12 +826,12 @@ TrayIcon - + Install update Herunterladen und installieren - + Remind me later Später erinnern diff -r b67681a61e3e -r 2a1206932f53 ui/mainwindow.cpp --- a/ui/mainwindow.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/mainwindow.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -271,7 +271,7 @@ verifyListData(); if (!mListToInstall.isValid()) { /* Downloader provided invalid files */ - /* TODO: Error count. Error handling. Otherwise + /* TODO (issue38): Error count. Error handling. Otherwise * we can go into an endless loop here */ /* Retry the download again in 10 - 20 minutes */ @@ -571,7 +571,7 @@ void MainWindow::downloaderError(const QString &message, SSLConnection::ErrorCode error) { - /* TODO logging and handle error according to a plan */ + /* TODO (issue38) handle error according to a plan */ syslog_error_printf ("Failed to check for updates: %s", message.toUtf8().constData()); #ifdef IS_TAG_BUILD /* During tag build it should never happen that an url checked is not available @@ -1209,7 +1209,7 @@ void MainWindow::loadCertificateList() { - /* TODO: if nothing is available (neither old nor new) add some progress + /* TODO (issue134): if nothing is available (neither old nor new) add some progress * indication */ mInstallList->clear(); mRemoveList->clear(); diff -r b67681a61e3e -r 2a1206932f53 ui/processhelp.h --- a/ui/processhelp.h Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/processhelp.h Thu Sep 18 18:53:11 2014 +0200 @@ -34,6 +34,9 @@ /** * @brief check if another process with the same name exists * +* Under GNU/Linux this only works for processes that have a +* processName.pid file in QStandardPaths::DataLocation. +* * @param[in] processName name of the process to look for. * * @returns true if one or more processes (other than the current process) exist diff -r b67681a61e3e -r 2a1206932f53 ui/processhelp_linux.cpp --- a/ui/processhelp_linux.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/processhelp_linux.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -23,15 +23,12 @@ } const QList ProcessHelp::getProcessesIdForName(const QString &processName) { - // TODO + // TODO (issue39) Q_UNUSED(processName); return QList(); } bool ProcessHelp::otherProcessesExist(const QString &processName) { - // FIXME: We are using lock file semantics on GNU Linux so the - // name of this method is rather misleading. - QDir dataDir(QStandardPaths::writableLocation(QStandardPaths::DataLocation)); dataDir.mkpath("."); QString lockFilePath = dataDir.filePath(processName + ".pid"); @@ -44,7 +41,7 @@ } void ProcessHelp::activateWindowForProcess(const QString &executableName) { - // TODO + // TODO (issue136) Q_UNUSED(executableName); return; } diff -r b67681a61e3e -r 2a1206932f53 ui/sslconnection.cpp --- a/ui/sslconnection.cpp Thu Sep 18 18:30:54 2014 +0200 +++ b/ui/sslconnection.cpp Thu Sep 18 18:53:11 2014 +0200 @@ -27,7 +27,7 @@ { if (certificate.isEmpty()) { #ifdef RELEASE_BUILD - /* TODO Change certificate here in case of release build */ + /* TODO (issue95) Change certificate here in case of release build */ QFile certResource(":certs/ssl-test"); #else QFile certResource(":certs/ssl-test");