Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intermittent crash if Confirmations library is called before being instantiated #7728

Closed
tmancey opened this issue Jan 14, 2020 · 1 comment · Fixed by brave/brave-core#4370

Comments

@tmancey
Copy link
Contributor

tmancey commented Jan 14, 2020

Description

Intermittent crash if Confirmations library was called from the ads or ledger libraries before being instantiated, this is due to a race condition.

Steps to Reproduce

  1. Unable to reproduce

Actual result:

Ledger library intermittently crashes if Confirmations has been not initialized when fetching transaction history or a timed event is triggered due to a race condition

Expected result:

Ledger library should fail gracefully if Confirmations is not initialized

Reproduces how often:

No steps to reproduce

Brave version (brave://version info)

Version 1.5.29 Chromium: 79.0.3945.117 (Developer Build) (64-bit)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

@tmancey tmancey added bug crash priority/P1 A very extremely bad problem. We might push a hotfix for it. feature/ads labels Jan 14, 2020
@tmancey tmancey added this to the 1.5.x - Nightly milestone Jan 14, 2020
@tmancey tmancey self-assigned this Jan 14, 2020
@tmancey tmancey changed the title Intermittent crash if Confirmations library was called before being instantiated Intermittent crash if Confirmations library is called before being instantiated Jan 14, 2020
@tmancey tmancey added the QA/Yes label Jan 14, 2020
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Mar 10, 2020

Verification passed on

Brave 1.5.109 Chromium: 80.0.3987.132 (Official Build) beta (64-bit)
Revision fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verification passed with

Brave 1.5.109 Chromium: 80.0.3987.132 (Official Build) beta (64-bit)
Revision fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974}
OS macOS Version 10.14.6 (Build 18G3020)
  • Verified test plan from Intermittent crash if Confirmations library is called before being instantiated brave-core#4370
  • Confirmed no crashes observed in the console log
  • Confirmed ads can be viewed and users are rewarded for a fresh install
  • Confirmed ads can be viewed and users are rewarded on an upgrade profile
  • Confirmed ads can be viewed and users are rewarded using a restored wallet
  • Confirmed ads can be viewed and users are rewarded after ads are switched off and then on

Verification passed on

Brave 1.5.111 Chromium: 80.0.3987.132 (Official Build) (64-bit)
Revision fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974}
OS Ubuntu 18.04 LTS
  • Verified test plan from Intermittent crash if Confirmations library is called before being instantiated brave-core#4370
  • Confirmed no crashes observed in the console log
  • Confirmed ads can be viewed and users are rewarded for a fresh install
  • Confirmed ads can be viewed and users are rewarded on an upgrade profile
  • Confirmed ads can be viewed and users are rewarded using a restored wallet
  • Confirmed ads can be viewed and users are rewarded after ads are switched off and then on

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants