initialize many attributes that could have been left without init value

done with clang-tidy

run-clang-tidy -p build -config="{Checks: '*', CheckOptions: [{key: UseAssignment, value: true}]}" -checks='-*,cppcoreguidelines-pro-type-member-init' -fix

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

diff --git a/src/3rdparty/kmessagewidget/kmessagewidget.cpp b/src/3rdparty/kmessagewidget/kmessagewidget.cpp
index 29666a6c41261a6229eb0eb892e8b67279e250a4..faa38ab1c39dc50a0e4aef0f3e6b6825784e399d 100644 (file)
--- a/src/3rdparty/kmessagewidget/kmessagewidget.cpp
+++ b/src/3rdparty/kmessagewidget/kmessagewidget.cpp
@@ -40,7 +40,7 @@ class KMessageWidgetPrivate
 public:
     void init(KMessageWidget *);
 
-    KMessageWidget *q;
+    KMessageWidget *q = nullptr;
     QFrame *content = nullptr;
     QLabel *iconLabel = nullptr;
     QLabel *textLabel = nullptr;
@@ -50,7 +50,7 @@ public:
     bool ignoreShowEventDoingAnimatedShow = false;
 
     KMessageWidget::MessageType messageType;
-    bool wordWrap;
+    bool wordWrap = false;
     QList<QToolButton *> buttons;
     QPixmap contentSnapShot;
 

diff --git a/src/3rdparty/qtlockedfile/qtlockedfile_unix.cpp b/src/3rdparty/qtlockedfile/qtlockedfile_unix.cpp
index d7460374fec47806b8ce873e427ac67cdc16881a..cb4df4ef2f85115a39c4d331a99d018504c73901 100644 (file)
--- a/src/3rdparty/qtlockedfile/qtlockedfile_unix.cpp
+++ b/src/3rdparty/qtlockedfile/qtlockedfile_unix.cpp
@@ -52,7 +52,7 @@ bool QtLockedFile::lock(LockMode mode, bool block)
     if (m_lock_mode != NoLock)
         unlock();
 
-    struct flock fl;
+    struct flock fl{};
     fl.l_whence = SEEK_SET;
     fl.l_start = 0;
     fl.l_len = 0;
@@ -82,7 +82,7 @@ bool QtLockedFile::unlock()
     if (!isLocked())
         return true;
 
-    struct flock fl;
+    struct flock fl{};
     fl.l_whence = SEEK_SET;
     fl.l_start = 0;
     fl.l_len = 0;

diff --git a/src/3rdparty/qtsingleapplication/qtsinglecoreapplication.h b/src/3rdparty/qtsingleapplication/qtsinglecoreapplication.h
index 139f9d8587e304ca6d90d2f7ec022d8c5be0eaef..ca22b57bb94c15f050e84a6ca9eab8c4eec82384 100644 (file)
--- a/src/3rdparty/qtsingleapplication/qtsinglecoreapplication.h
+++ b/src/3rdparty/qtsingleapplication/qtsinglecoreapplication.h
@@ -55,7 +55,7 @@ Q_SIGNALS:
 
 private:
     QtLocalPeer* peer;
-    bool block;
+    bool block = false;
 };
 
 } // namespace SharedTools

diff --git a/src/cmd/cmd.cpp b/src/cmd/cmd.cpp
index 3826a681bd37f2b04ab31147b2c1be52658dae46..37c456dd1ae0fea997098b89495b7c2a283ceced 100644 (file)
--- a/src/cmd/cmd.cpp
+++ b/src/cmd/cmd.cpp
@@ -71,16 +71,16 @@ struct CmdOptions
     QString user;
     QString password;
     QString proxy;
-    bool silent;
-    bool trustSSL;
-    bool useNetrc;
-    bool interactive;
-    bool ignoreHiddenFiles;
+    bool silent = false;
+    bool trustSSL = false;
+    bool useNetrc = false;
+    bool interactive = false;
+    bool ignoreHiddenFiles = false;
     QString exclude;
     QString unsyncedfolders;
-    int restartTimes;
-    int downlimit;
-    int uplimit;
+    int restartTimes = 0;
+    int downlimit = 0;
+    int uplimit = 0;
 };
 
 // we can't use csync_set_userdata because the SyncEngine sets it already.
@@ -118,7 +118,7 @@ private:
     DWORD mode = 0;
     HANDLE hStdin;
 #else
-    termios tios;
+    termios tios{};
 #endif
 };
 

diff --git a/src/common/utility.cpp b/src/common/utility.cpp
index 6613f0986a6baa160ab31d3688e177fe8607c54e..da968e53f0091f58d23d933b999052fe25ba0151 100644 (file)
--- a/src/common/utility.cpp
+++ b/src/common/utility.cpp
@@ -228,7 +228,7 @@ qint64 Utility::freeDiskSpace(const QString &path)
         return (qint64)stat.f_bavail * stat.f_frsize;
     }
 #elif defined(Q_OS_UNIX)
-    struct statvfs64 stat;
+    struct statvfs64 stat{};
     if (statvfs64(path.toLocal8Bit().data(), &stat) == 0) {
         return (qint64)stat.f_bavail * stat.f_frsize;
     }

diff --git a/src/gui/accountsettings.cpp b/src/gui/accountsettings.cpp
index 02a218e0560bb72bd9a59c209c9e2980f5c875d8..51ac7cd3507b0c5cb439eccb52a2a5f07f672ead 100644 (file)
--- a/src/gui/accountsettings.cpp
+++ b/src/gui/accountsettings.cpp
@@ -123,8 +123,8 @@ public:
     {
     }
 
-    QTreeView *folderList;
-    FolderStatusModel *model;
+    QTreeView *folderList = nullptr;
+    FolderStatusModel *model = nullptr;
 
 protected:
     bool eventFilter(QObject *watched, QEvent *event) override
@@ -1004,7 +1004,7 @@ void AccountSettings::slotSetSubFolderAvailability(Folder *folder, const QString
 void AccountSettings::displayMnemonic(const QString &mnemonic)
 {
     QDialog widget;
-    Ui_Dialog ui;
+    Ui_Dialog ui{};
     ui.setupUi(&widget);
     widget.setWindowTitle(tr("End-to-end encryption mnemonic"));
     ui.label->setText(

diff --git a/src/gui/accountsettings.h b/src/gui/accountsettings.h
index f3384ef76313c458e087bf58c57cea2e8b78678e..7a3cd2df02f475b704ef19986c29ba7caa0fcaef 100644 (file)
--- a/src/gui/accountsettings.h
+++ b/src/gui/accountsettings.h
@@ -143,8 +143,8 @@ private:
     bool _wasDisabledBefore;
     AccountState *_accountState;
     UserInfo _userInfo;
-    QAction *_toggleSignInOutAction;
-    QAction *_addAccountAction;
+    QAction *_toggleSignInOutAction = nullptr;
+    QAction *_addAccountAction = nullptr;
 
     bool _menuShown;
 

diff --git a/src/gui/accountsetupcommandlinemanager.h b/src/gui/accountsetupcommandlinemanager.h
index 585627e85801a63d0828b2b52ff076f7c1b83d7d..156ab5d9002ab2ecc2091bdc6b6ae49d7fa1f8b7 100644 (file)
--- a/src/gui/accountsetupcommandlinemanager.h
+++ b/src/gui/accountsetupcommandlinemanager.h
@@ -45,7 +45,7 @@ private:
     QUrl _serverUrl;
     QString _remoteDirPath;
     QString _localDirPath;
-    bool _isVfsEnabled;
+    bool _isVfsEnabled = false;
 };
 
 }

diff --git a/src/gui/creds/flow2auth.cpp b/src/gui/creds/flow2auth.cpp
index d0fbbca6ba281db3d5cac358e54af73e4beedf44..1526921b83ed3a9762de6fda2ffe79a278a0d899 100644 (file)
--- a/src/gui/creds/flow2auth.cpp
+++ b/src/gui/creds/flow2auth.cpp
@@ -91,7 +91,7 @@ void Flow2Auth::fetchNewToken(const TokenAction action)
 
     QObject::connect(job, &SimpleNetworkJob::finishedSignal, this, [this, action](QNetworkReply *reply) {
         auto jsonData = reply->readAll();
-        QJsonParseError jsonParseError;
+        QJsonParseError jsonParseError{};
         QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
         QString pollToken, pollEndpoint, loginUrl;
 
@@ -208,7 +208,7 @@ void Flow2Auth::slotPollTimerTimeout()
 
     QObject::connect(job, &SimpleNetworkJob::finishedSignal, this, [this](QNetworkReply *reply) {
         auto jsonData = reply->readAll();
-        QJsonParseError jsonParseError;
+        QJsonParseError jsonParseError{};
         QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
         QUrl serverUrl;
         QString loginName, appPassword;

diff --git a/src/gui/creds/flow2auth.h b/src/gui/creds/flow2auth.h
index 6b9440cfb1079ad559adf178916a44507a2bd1e9..35efb721ca9f47adb5d437055936905976e3d0c3 100644 (file)
--- a/src/gui/creds/flow2auth.h
+++ b/src/gui/creds/flow2auth.h
@@ -77,8 +77,8 @@ private:
     QString _pollToken;
     QString _pollEndpoint;
     QTimer _pollTimer;
-    qint64 _secondsLeft;
-    qint64 _secondsInterval;
+    qint64 _secondsLeft = 0LL;
+    qint64 _secondsInterval = 0LL;
     bool _isBusy;
     bool _hasToken;
     bool _enforceHttps = false;

diff --git a/src/gui/creds/oauth.cpp b/src/gui/creds/oauth.cpp
index 04b0e62f865f520fa163826c2cc7216ec6b86d44..eaa25b97ae4ffae392cc0c8af137471616627045 100644 (file)
--- a/src/gui/creds/oauth.cpp
+++ b/src/gui/creds/oauth.cpp
@@ -99,7 +99,7 @@ void OAuth::start()
                 job->setTimeout(qMin(30 * 1000ll, job->timeoutMsec()));
                 QObject::connect(job, &SimpleNetworkJob::finishedSignal, this, [this, socket](QNetworkReply *reply) {
                     auto jsonData = reply->readAll();
-                    QJsonParseError jsonParseError;
+                    QJsonParseError jsonParseError{};
                     QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
                     QString accessToken = json["access_token"].toString();
                     QString refreshToken = json["refresh_token"].toString();

diff --git a/src/gui/filedetails/shareemodel.h b/src/gui/filedetails/shareemodel.h
index 3bc1ec83c06d750f6fd72788343b779b43b6717f..94362d11b6c75d2827382a333e1900d8ac05bb88 100644 (file)
--- a/src/gui/filedetails/shareemodel.h
+++ b/src/gui/filedetails/shareemodel.h
@@ -92,7 +92,7 @@ private:
 
     QTimer _searchRateLimitingTimer;
 
-    AccountState *_accountState;
+    AccountState *_accountState = nullptr;
     QString _searchString;
     bool _shareItemIsFolder = false;
     bool _fetchOngoing = false;

diff --git a/src/gui/filedetails/sharemodel.h b/src/gui/filedetails/sharemodel.h
index 05e08fbdca264c71b3de81a2a6de509d6306774f..4e119853479e1326e6c4383deaafb6370de29ccf 100644 (file)
--- a/src/gui/filedetails/sharemodel.h
+++ b/src/gui/filedetails/sharemodel.h
@@ -196,8 +196,8 @@ private:
     // We cannot pass objects instantiated in QML using smart pointers through the property interface
     // so we have to pass the pointer here. If we kill the dialog using a smart pointer then
     // these objects will be deallocated for the entire application. We do not want that!!
-    AccountState *_accountState;
-    Folder *_folder;
+    AccountState *_accountState = nullptr;
+    Folder *_folder = nullptr;
 
     QString _localPath;
     QString _sharePath;

diff --git a/src/gui/folderwizard.h b/src/gui/folderwizard.h
index fb60caca39eb6e4e374821feb68064258ca531c5..393c0ad33ab8bb8879eace2aca3eff9f2fff42ef 100644 (file)
--- a/src/gui/folderwizard.h
+++ b/src/gui/folderwizard.h
@@ -70,7 +70,7 @@ protected slots:
 private:
     void changeStyle();
 
-    Ui_FolderWizardSourcePage _ui;
+    Ui_FolderWizardSourcePage _ui{};
     Folder::Map _folderMap;
     AccountPtr _account;
 };
@@ -114,7 +114,7 @@ private:
     LsColJob *runLsColJob(const QString &path);
     void recursiveInsert(QTreeWidgetItem *parent, QStringList pathTrail, QString path);
     bool selectByPath(QString path);
-    Ui_FolderWizardTargetPage _ui;
+    Ui_FolderWizardTargetPage _ui{};
     bool _warnWasVisible;
     AccountPtr _account;
     QTimer _lscolTimer;

diff --git a/src/gui/main.cpp b/src/gui/main.cpp
index d5d106efd0b4b76468c8b015ccd891ed0727604b..40af80ac019c5f307b2a1713d1e2fd4d82658816 100644 (file)
--- a/src/gui/main.cpp
+++ b/src/gui/main.cpp
@@ -116,7 +116,7 @@ int main(int argc, char **argv)
 // check a environment variable for core dumps
 #ifdef Q_OS_UNIX
     if (!qEnvironmentVariableIsEmpty("OWNCLOUD_CORE_DUMP")) {
-        struct rlimit core_limit;
+        struct rlimit core_limit{};
         core_limit.rlim_cur = RLIM_INFINITY;
         core_limit.rlim_max = RLIM_INFINITY;
 

diff --git a/src/gui/ocsjob.cpp b/src/gui/ocsjob.cpp
index 68c680b4dedca0ff955b29252d5969695da63fec..09f7de00c0f1e67217f6ed1bd723593c0ad13239 100644 (file)
--- a/src/gui/ocsjob.cpp
+++ b/src/gui/ocsjob.cpp
@@ -110,7 +110,7 @@ bool OcsJob::finished()
 {
     const QByteArray replyData = reply()->readAll();
 
-    QJsonParseError error;
+    QJsonParseError error{};
     QString message;
     int statusCode = 0;
     auto json = QJsonDocument::fromJson(replyData, &error);

diff --git a/src/gui/owncloudgui.h b/src/gui/owncloudgui.h
index ee38ad49693579423d48611f86bd5e0d64f191f4..41c7895ef68635d31f082c274aa70f6dc5d633bc 100644 (file)
--- a/src/gui/owncloudgui.h
+++ b/src/gui/owncloudgui.h
@@ -120,9 +120,9 @@ private:
     QDBusConnection _bus;
 #endif
 
-    QAction *_actionNewAccountWizard;
-    QAction *_actionSettings;
-    QAction *_actionEstimate;
+    QAction *_actionNewAccountWizard = nullptr;
+    QAction *_actionSettings = nullptr;
+    QAction *_actionEstimate = nullptr;
 
 
     QList<QAction *> _recentItemsActions;

diff --git a/src/gui/remotewipe.cpp b/src/gui/remotewipe.cpp
index 8a1a0b06dc71cf896da0453bd7af8d2a0379cd9b..73af19f71c0628772e26c6629bd115fbe0145a96 100644 (file)
--- a/src/gui/remotewipe.cpp
+++ b/src/gui/remotewipe.cpp
@@ -70,7 +70,7 @@ void RemoteWipe::startCheckJobWithAppPassword(QString pwd){
 void RemoteWipe::checkJobSlot()
 {
     auto jsonData = _networkReplyCheck->readAll();
-    QJsonParseError jsonParseError;
+    QJsonParseError jsonParseError{};
     QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
     bool wipe = false;
 
@@ -148,7 +148,7 @@ void RemoteWipe::notifyServerSuccessJob(AccountState *accountState, bool dataWip
 void RemoteWipe::notifyServerSuccessJobSlot()
 {
     auto jsonData = _networkReplySuccess->readAll();
-    QJsonParseError jsonParseError;
+    QJsonParseError jsonParseError{};
     QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
     if (_networkReplySuccess->error() != QNetworkReply::NoError ||
             jsonParseError.error != QJsonParseError::NoError) {

diff --git a/src/gui/selectivesyncdialog.h b/src/gui/selectivesyncdialog.h
index d44b957c5d58fb7a7e0a2a59dc813d464e52ebd4..acf8a6f4b1475b6c6095894f004510eb050d6705 100644 (file)
--- a/src/gui/selectivesyncdialog.h
+++ b/src/gui/selectivesyncdialog.h
@@ -111,9 +111,9 @@ public:
 private:
     void init(const AccountPtr &account);
 
-    SelectiveSyncWidget *_selectiveSync;
+    SelectiveSyncWidget *_selectiveSync = nullptr;
 
     Folder *_folder;
-    QPushButton *_okButton;
+    QPushButton *_okButton = nullptr;
 };
 }

diff --git a/src/gui/socketapi/socketapi.cpp b/src/gui/socketapi/socketapi.cpp
index add45e0a94ad5924517ca24190134085d733a0a5..9facf93a32bbfceeea789f76bef97de137c7988c 100644 (file)
--- a/src/gui/socketapi/socketapi.cpp
+++ b/src/gui/socketapi/socketapi.cpp
@@ -411,7 +411,7 @@ void SocketApi::slotReadSocket()
                 socketApiJob->reject(QStringLiteral("command not found"));
             }
         } else if (command.startsWith("V2/")) {
-            QJsonParseError error;
+            QJsonParseError error{};
             const auto json = QJsonDocument::fromJson(argument.toUtf8(), &error).object();
             if (error.error != QJsonParseError::NoError) {
                 qCWarning(lcSocketApi()) << "Invalid json" << argument.toString() << error.errorString();

diff --git a/src/gui/wizard/flow2authwidget.h b/src/gui/wizard/flow2authwidget.h
index cc1b01d57abbd1bb7612601c8d27928604f5c0a5..a90fcbf4957759278c7b3dc4b6e874655577cd5c 100644 (file)
--- a/src/gui/wizard/flow2authwidget.h
+++ b/src/gui/wizard/flow2authwidget.h
@@ -50,7 +50,7 @@ signals:
 private:
     Account *_account = nullptr;
     QScopedPointer<Flow2Auth> _asyncAuth;
-    Ui_Flow2AuthWidget _ui;
+    Ui_Flow2AuthWidget _ui{};
 
 protected slots:
     void slotOpenBrowser();

diff --git a/src/gui/wizard/owncloudadvancedsetuppage.h b/src/gui/wizard/owncloudadvancedsetuppage.h
index 4de4b593e4918dd8cd0a172b56ab8e82cfa8de62..2c0f23bb42b37bff347232e06d39f30fb39e6167 100644 (file)
--- a/src/gui/wizard/owncloudadvancedsetuppage.h
+++ b/src/gui/wizard/owncloudadvancedsetuppage.h
@@ -87,7 +87,7 @@ private:
     // TODO: remove when UX decision is made
     void refreshVirtualFilesAvailibility(const QString &path);
 
-    Ui_OwncloudAdvancedSetupPage _ui;
+    Ui_OwncloudAdvancedSetupPage _ui{};
     bool _checking = false;
     bool _created = false;
     bool _localFolderValid = false;

diff --git a/src/gui/wizard/owncloudoauthcredspage.h b/src/gui/wizard/owncloudoauthcredspage.h
index 339e97b025c4d1b040f070e2029b94e9c61bc6c1..2702aa3300cdbcb8c7c4fe0a4de27bf8fcd2c86f 100644 (file)
--- a/src/gui/wizard/owncloudoauthcredspage.h
+++ b/src/gui/wizard/owncloudoauthcredspage.h
@@ -56,7 +56,7 @@ public:
     QString _token;
     QString _refreshToken;
     QScopedPointer<OAuth> _asyncAuth;
-    Ui_OwncloudOAuthCredsPage _ui;
+    Ui_OwncloudOAuthCredsPage _ui{};
 
 protected slots:
     void slotOpenBrowser();

diff --git a/src/gui/wizard/owncloudsetuppage.h b/src/gui/wizard/owncloudsetuppage.h
index 590bc289f63036f70c10a240ca017a9af482396e..e9c5da0d2c0d07be53f03e61377d710e5a339f62 100644 (file)
--- a/src/gui/wizard/owncloudsetuppage.h
+++ b/src/gui/wizard/owncloudsetuppage.h
@@ -78,7 +78,7 @@ private:
     void customizeStyle();
     void setupServerAddressDescriptionLabel();
 
-    Ui_OwncloudSetupPage _ui;
+    Ui_OwncloudSetupPage _ui{};
 
     QString _oCUrl;
     QString _ocUser;

diff --git a/src/gui/wizard/owncloudwizard.h b/src/gui/wizard/owncloudwizard.h
index c720ab0ed79c6a6f98663ae09a8d55009ba3e42e..a586ee2615800b32a7fe1497d35b345786af49e7 100644 (file)
--- a/src/gui/wizard/owncloudwizard.h
+++ b/src/gui/wizard/owncloudwizard.h
@@ -125,7 +125,7 @@ private:
     OwncloudOAuthCredsPage *_browserCredsPage;
     Flow2AuthCredsPage *_flow2CredsPage;
     OwncloudAdvancedSetupPage *_advancedSetupPage;
-    OwncloudWizardResultPage *_resultPage;
+    OwncloudWizardResultPage *_resultPage = nullptr;
     AbstractCredentialsWizardPage *_credentialsPage = nullptr;
     WebViewPage *_webViewPage = nullptr;
 

diff --git a/src/gui/wizard/webviewpage.h b/src/gui/wizard/webviewpage.h
index 23667975e44c520980fe07d0dc5a35fceb39adfe..cbb6959ec7bf9a4298605cf5db90bfb037a3b6a1 100644 (file)
--- a/src/gui/wizard/webviewpage.h
+++ b/src/gui/wizard/webviewpage.h
@@ -40,7 +40,7 @@ private:
     QString _user;
     QString _pass;
 
-    bool _useSystemProxy;
+    bool _useSystemProxy = false;
 
     QSize _originalWizardSize;
 };

diff --git a/src/libsync/bandwidthmanager.h b/src/libsync/bandwidthmanager.h
index 15e08f997b67c752a152255c27c869ed323bba59..dac0d78a1e2b0fe5cf037eeed92a387664232628 100644 (file)
--- a/src/libsync/bandwidthmanager.h
+++ b/src/libsync/bandwidthmanager.h
@@ -96,7 +96,7 @@ private:
     GETFileJob *_relativeLimitCurrentMeasuredJob;
 
     // for measuring how much progress we made at start
-    qint64 _relativeDownloadLimitProgressAtMeasuringRestart;
+    qint64 _relativeDownloadLimitProgressAtMeasuringRestart = 0LL;
 
     qint64 _currentDownloadLimit;
 };

diff --git a/src/libsync/clientsideencryptionjobs.cpp b/src/libsync/clientsideencryptionjobs.cpp
index 2e52a02a99176ac277900ec553270bb40dd09458..843419fbcb59f3e481ce2ab9b6206f505cd2ab07 100644 (file)
--- a/src/libsync/clientsideencryptionjobs.cpp
+++ b/src/libsync/clientsideencryptionjobs.cpp
@@ -54,7 +54,7 @@ bool GetMetadataApiJob::finished()
         emit error(_fileId, retCode);
         return true;
     }
-    QJsonParseError error;
+    QJsonParseError error{};
     auto json = QJsonDocument::fromJson(reply()->readAll(), &error);
     emit jsonReceived(json, reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt());
     return true;
@@ -285,7 +285,7 @@ bool LockEncryptFolderApiJob::finished()
         return true;
     }
 
-    QJsonParseError error;
+    QJsonParseError error{};
     const auto json = QJsonDocument::fromJson(reply()->readAll(), &error);
     const auto obj = json.object().toVariantMap();
     const auto token = obj["ocs"].toMap()["data"].toMap()["e2e-token"].toByteArray();
@@ -364,7 +364,7 @@ bool StorePrivateKeyApiJob::finished()
     if (retCode != 200)
         qCInfo(lcStorePrivateKeyApiJob()) << "Sending private key ended with"  << path() << errorString() << retCode;
 
-    QJsonParseError error;
+    QJsonParseError error{};
     auto json = QJsonDocument::fromJson(reply()->readAll(), &error);
     emit jsonReceived(json, reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt());
     return true;
@@ -401,7 +401,7 @@ bool SignPublicKeyApiJob::finished()
 {
     qCInfo(lcStorePrivateKeyApiJob()) << "Sending CSR ended with"  << path() << errorString() << reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute);
 
-    QJsonParseError error;
+    QJsonParseError error{};
     auto json = QJsonDocument::fromJson(reply()->readAll(), &error);
     emit jsonReceived(json, reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt());
     return true;

diff --git a/src/libsync/clientsideencryptionjobs.h b/src/libsync/clientsideencryptionjobs.h
index f5cf6fbb6a2138c7c8b941fd7ed8fded5f670ff6..7fcbac10d1d76d944175572a49c4619c1131029f 100644 (file)
--- a/src/libsync/clientsideencryptionjobs.h
+++ b/src/libsync/clientsideencryptionjobs.h
@@ -193,7 +193,7 @@ signals:
 private:
     QByteArray _fileId;
     QByteArray _token;
-    QBuffer *_tokenBuf;
+    QBuffer *_tokenBuf = nullptr;
     QPointer<SyncJournalDb> _journalDb;
 };
 

diff --git a/src/libsync/creds/httpcredentials.cpp b/src/libsync/creds/httpcredentials.cpp
index 21ec6b8232f15a6a97932c0bd19f55eefb0dae94..fe570897a07e5ab988da51cd772e98000e7a4abc 100644 (file)
--- a/src/libsync/creds/httpcredentials.cpp
+++ b/src/libsync/creds/httpcredentials.cpp
@@ -430,7 +430,7 @@ bool HttpCredentials::refreshAccessToken()
     job->setTimeout(qMin(30 * 1000ll, job->timeoutMsec()));
     QObject::connect(job, &SimpleNetworkJob::finishedSignal, this, [this](QNetworkReply *reply) {
         auto jsonData = reply->readAll();
-        QJsonParseError jsonParseError;
+        QJsonParseError jsonParseError{};
         QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
         QString accessToken = json["access_token"].toString();
         if (jsonParseError.error != QJsonParseError::NoError || json.isEmpty()) {

diff --git a/src/libsync/creds/keychainchunk.h b/src/libsync/creds/keychainchunk.h
index b6bc50e458fb2b04f92f5fc3393a7112c958a448..8e09ba66adf7ce297959d8cc136918929ab9cfb4 100644 (file)
--- a/src/libsync/creds/keychainchunk.h
+++ b/src/libsync/creds/keychainchunk.h
@@ -74,7 +74,7 @@ public:
 
 protected:
     QString _serviceName;
-    Account *_account;
+    Account *_account = nullptr;
     QString _key;
     bool _insecureFallback = false;
     bool _autoDelete = true;

diff --git a/src/libsync/networkjobs.cpp b/src/libsync/networkjobs.cpp
index 19951674dbbe878a44528d576bf372a5bb384e21..f1f73bdfa20e922c78c824a6dd1ef06a4e43f691 100644 (file)
--- a/src/libsync/networkjobs.cpp
+++ b/src/libsync/networkjobs.cpp
@@ -530,7 +530,7 @@ bool CheckServerJob::finished()
         qCWarning(lcCheckServerJob) << "error: status.php replied " << httpStatus << body;
         emit instanceNotFound(reply());
     } else {
-        QJsonParseError error;
+        QJsonParseError error{};
         auto status = QJsonDocument::fromJson(body, &error);
         // empty or invalid response
         if (error.error != QJsonParseError::NoError || status.isNull()) {
@@ -862,7 +862,7 @@ bool JsonApiJob::finished()
     if(reply()->rawHeaderList().contains("ETag"))
         emit etagResponseHeaderReceived(reply()->rawHeader("ETag"), statusCode);
 
-    QJsonParseError error;
+    QJsonParseError error{};
     auto json = QJsonDocument::fromJson(jsonStr.toUtf8(), &error);
     // empty or invalid response and status code is != 304 because jsonStr is expected to be empty
     if ((error.error != QJsonParseError::NoError || json.isNull()) && httpStatusCode != notModifiedStatusCode) {

diff --git a/src/libsync/progressdispatcher.cpp b/src/libsync/progressdispatcher.cpp
index 85b99fb66ab770f14cee3096c6c1e8b62be267cc..abc820fb5fbf88b610653b04d1aa3bbe9307a6b3 100644 (file)
--- a/src/libsync/progressdispatcher.cpp
+++ b/src/libsync/progressdispatcher.cpp
@@ -369,7 +369,7 @@ void ProgressInfo::recomputeCompletedSize()
 
 ProgressInfo::Estimates ProgressInfo::Progress::estimates() const
 {
-    Estimates est;
+    Estimates est{};
     est.estimatedBandwidth = _progressPerSec;
     if (_progressPerSec != 0) {
         est.estimatedEta = qRound64(static_cast<double>(_total - _completed) / _progressPerSec) * 1000;

diff --git a/src/libsync/progressdispatcher.h b/src/libsync/progressdispatcher.h
index 3276586cd942e76f4b7683826c55fe18388c1127..9ff71b7e3086f23bcd45edc0de829b9509bdc877 100644 (file)
--- a/src/libsync/progressdispatcher.h
+++ b/src/libsync/progressdispatcher.h
@@ -227,11 +227,11 @@ private:
     Progress _fileProgress;
 
     // All size from completed jobs only.
-    qint64 _totalSizeOfCompletedJobs;
+    qint64 _totalSizeOfCompletedJobs = 0LL;
 
     // The fastest observed rate of files per second in this sync.
-    double _maxFilesPerSecond;
-    double _maxBytesPerSecond;
+    double _maxFilesPerSecond = 0.0;
+    double _maxBytesPerSecond = 0.0;
 };
 
 namespace Progress {

diff --git a/src/libsync/propagateupload.cpp b/src/libsync/propagateupload.cpp
index 829dc529d5485998309db5c0639ad964e1950004..658c9060d24c0f7dfec1ff22014889f75f40364e 100644 (file)
--- a/src/libsync/propagateupload.cpp
+++ b/src/libsync/propagateupload.cpp
@@ -130,7 +130,7 @@ bool PollJob::finished()
     }
 
     QByteArray jsonData = reply()->readAll().trimmed();
-    QJsonParseError jsonParseError;
+    QJsonParseError jsonParseError{};
     QJsonObject json = QJsonDocument::fromJson(jsonData, &jsonParseError).object();
     qCInfo(lcPollJob) << ">" << jsonData << "<" << reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt() << json << jsonParseError.errorString();
     if (jsonParseError.error != QJsonParseError::NoError) {

diff --git a/src/libsync/userstatusconnector.h b/src/libsync/userstatusconnector.h
index 1af208ea0bf8a2edce466244d8930cdec62ae831..20d12347f8422885e19eaa26a3ba36b0d5308163 100644 (file)
--- a/src/libsync/userstatusconnector.h
+++ b/src/libsync/userstatusconnector.h
@@ -94,7 +94,7 @@ private:
     QString _message;
     QString _icon;
     OnlineStatus _state = OnlineStatus::Online;
-    bool _messagePredefined;
+    bool _messagePredefined = false;
     Optional<ClearAt> _clearAt;
 };
 

diff --git a/test/syncenginetestutils.h b/test/syncenginetestutils.h
index 7f1c3e2f6248f8e7f1a8ad5a613eb3b8523816de..8d24e2863b30ec48dad0add6a48ab0882404cfd8 100644 (file)
--- a/test/syncenginetestutils.h
+++ b/test/syncenginetestutils.h
@@ -304,8 +304,8 @@ class FakeGetReply : public FakeReply
     Q_OBJECT
 public:
     const FileInfo *fileInfo;
-    char payload;
-    int size;
+    char payload = 0;
+    int size = 0;
     bool aborted = false;
 
     FakeGetReply(FileInfo &remoteRootFileInfo, QNetworkAccessManager::Operation op, const QNetworkRequest &request, QObject *parent);

diff --git a/test/testchecksumvalidator.cpp b/test/testchecksumvalidator.cpp
index 10be1f0d7dd16fc0e7ceb73af3f764768446605b..3d2bb076d644d83617010951431797560a113aca 100644 (file)
--- a/test/testchecksumvalidator.cpp
+++ b/test/testchecksumvalidator.cpp
@@ -28,8 +28,8 @@ using namespace OCC::Utility;
         ValidateChecksumHeader::FailureReason _expectedFailureReason = ValidateChecksumHeader::FailureReason::Success;
         QByteArray     _expected;
         QByteArray     _expectedType;
-        bool           _successDown;
-        bool           _errorSeen;
+        bool           _successDown = false;
+        bool           _errorSeen = false;
 
     public slots:
 

diff --git a/test/testxmlparse.cpp b/test/testxmlparse.cpp
index 201c3a449978aa792c54e7ac54207fa10c02d497..8bc987a525e3fc7756d72719253c148b5ebae068 100644 (file)
--- a/test/testxmlparse.cpp
+++ b/test/testxmlparse.cpp
@@ -15,7 +15,7 @@ class TestXmlParse : public QObject
     Q_OBJECT
 
 private:
-  bool _success;
+  bool _success = false;
   QStringList _subdirs;
   QStringList _items;