Check if sync enabled before calling

Fixes brave/brave-browser#25156
brave · Sep 7, 2022 · ad5c0d1 · ad5c0d1
1 parent f9f8133
commit ad5c0d1
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/chromium_src/chrome/browser/ui/webui/settings/people_handler.cc b/chromium_src/chrome/browser/ui/webui/settings/people_handler.cc
@@ -7,17 +7,20 @@
 
 // IsSetupInProgress isn't accurate in brave sync flow especially for the first
 // time setup, we rely on it to display setup dialog
-#define BRAVE_GET_SYNC_STATUS_DICTIONARY                                       \
-  sync_status.SetBoolPath(                                                     \
-      "firstSetupInProgress",                                                  \
-      service && !disallowed_by_policy &&                                      \
-          !service->GetUserSettings()->IsFirstSetupComplete());                \
-  {                                                                            \
-    syncer::BraveSyncServiceImpl* brave_sync_service =                         \
-        static_cast<syncer::BraveSyncServiceImpl*>(service);                   \
-    bool failed_to_decrypt = false;                                            \
-    brave_sync_service->prefs().GetSeed(&failed_to_decrypt);                   \
-    sync_status.SetBoolPath("hasSyncWordsDecryptionError", failed_to_decrypt); \
+#define BRAVE_GET_SYNC_STATUS_DICTIONARY                        \
+  sync_status.SetBoolPath(                                      \
+      "firstSetupInProgress",                                   \
+      service && !disallowed_by_policy &&                       \
+          !service->GetUserSettings()->IsFirstSetupComplete()); \
+  {                                                             \
+    syncer::BraveSyncServiceImpl* brave_sync_service =          \
+        static_cast<syncer::BraveSyncServiceImpl*>(service);    \
+    if (brave_sync_service) {                                   \
+      bool failed_to_decrypt = false;                           \
+      brave_sync_service->prefs().GetSeed(&failed_to_decrypt);  \
+      sync_status.SetBoolPath("hasSyncWordsDecryptionError",    \
+                              failed_to_decrypt);               \
+    }                                                           \
   }
 
 #include "src/chrome/browser/ui/webui/settings/people_handler.cc"