Merge pull request #47 from brave-intl/wallet-problem

Adds json parse error handler
brave · Aug 14, 2018 · f198216 · f198216
2 parents eb5bd5e + 5efd51b
commit f198216
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/bat_client.cc b/src/bat_client.cc
@@ -227,7 +227,11 @@ void BatClient::getWalletProperties() {
      return;
    }
 
-   braveledger_bat_helper::loadFromJson(properties, response);
+   bool ok = braveledger_bat_helper::loadFromJson(properties, response);
+   if (!ok) {
+     ledger_->OnWalletProperties(ledger::Result::ERROR, properties);
+     return;
+   }
    state_->walletProperties_ = properties;
    ledger_->OnWalletProperties(ledger::Result::OK, properties);
  }