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

Launching VPN support page will crash app if set as default #23492

Closed
Uni-verse opened this issue Jun 15, 2022 · 3 comments · Fixed by brave/brave-core#13851
Closed

Launching VPN support page will crash app if set as default #23492

Uni-verse opened this issue Jun 15, 2022 · 3 comments · Fixed by brave/brave-core#13851

Comments

@Uni-verse
Copy link
Contributor

Description

Loading the VPN Support page will crash app if brave was previously set as default. It will open URL initially if brave is not default browser and user is prompted with the open with modal and selects "Always". If user tries to open again after that, it will result in crash. Also, selecting "Just Once" from the open with modal with result in same crash.

Note: VPN menu is disabled on Beta, Nightly

Case 1: Set brave as default, open VPN Support page
Case 2: Skip default, open VPN Support page, select "Just Once" from open with prompt

Reproducible on 1.39.x, 1.40.x, 1.41.x

Steps to reproduce

Precondition: VPN setting enabled through dev/qa menu on Beta/Nightly

  1. Clean install
  2. Set brave as default browser
  3. Navigate to Menu > Settings > Brave Firewall + VPN
  4. Tap on VPN Support
  5. Observe

Actual result

Support_crash1.mp4
Support_crash3.mp4

--- beginning of crash
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: FATAL EXCEPTION: main
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: Process: com.brave.browser_beta, PID: 2845
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.res.Resources$Theme android.content.Context.getTheme()' on a null object reference
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at Q33.a(chromium-MonochromePublic6432.aab-beta-414009624:355)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at androidx.preference.Preference.<init>(chromium-MonochromePublic6432.aab-beta-414009624:314)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at tf3.<init>(chromium-MonochromePublic6432.aab-beta-414009624:325)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at org.chromium.chrome.browser.settings.BraveMainPreferencesBase.c4(chromium-MonochromePublic6432.aab-beta-414009624:182)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at oG.run(Unknown Source:2)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:938)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:99)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at android.os.Looper.loopOnce(Looper.java:226)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:313)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:8582)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:563)
06-15 13:37:28.250 10824  2845  2845 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1133)
06-15 13:37:28.266  1000   737   737 D io_stats: !@   8,0 r 4993518 130680836 w 5549344 102408648 d 1049554 306078732 f 0 0 iot 5352904 0 th 0 0 0 pt 0 inp 0 0 365898.460
06-15 13:37:28.583  1000  1230  3581 I DropBoxManagerService: add tag=data_app_crash isTagEnabled=true flags=0x2
06-15 13:37:28.585  1000  1230  1586 W BroadcastQueue: Background execution not allowed: receiving Intent { act=android.intent.action.DROPBOX_ENTRY_ADDED flg=0x10 (has extras) } to com.google.android.gms/.stats.service.DropBoxEntryAddedReceiver
06-15 13:37:28.585  1000  1230  2449 W ActivityManager: crash : com.brave.browser_beta,10824

Expected result

VPN support page should open in new tab

Issue reproduces how often

Easily, follow STR

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? Yes
  • Can you reproduce this issue with the current Play Store Beta version? Yes
  • Can you reproduce this issue with the current Play Store Nightly version? Yes

Device details

  • Install type (ARM, x86): ARM
  • Device type (Phone, Tablet, Phablet): Samsung GS 21 & Tab S7
  • Android version: 12

Brave version

Release 1.39.123, Chromium 102.0.5005.125
Beta 1.40.x
Nightly 1.41.x

Additional information

@Uni-verse Uni-verse added bug QA/Yes OS/Android Fixes related to Android browser functionality feature/vpn labels Jun 15, 2022
@kjozwiak kjozwiak added the crash label Jun 16, 2022
@kjozwiak
Copy link
Member

kjozwiak commented Jun 16, 2022

Using the STR/Cases outlined via #23492 (comment), I reproduced the crash using the following devices:

  • reproduced on Pixel 6 running Android 13 using 1.39.123 Chromium: 102.0.5005.125
  • reproduced on Samsung S10+ running Android 12 using 1.39.123 Chromium: 102.0.5005.125

Pixel 6 Example

screen-20220616-004110.mp4

Samsung S10+ Example

Screen_Recording_20220616-005155_Brave.mp4

@kjozwiak
Copy link
Member

The above will require 1.40.105 or newer for 1.40.x verification 👍

@kjozwiak
Copy link
Member

kjozwiak commented Jun 20, 2022

Verification PASSED on Pixel 6 running Android 13 using the following build(s):

Brave | 1.40.105 Chromium: 103.0.5060.53 (Official Build) (64-bit)
--- | ---
Revision | a1711811edd74ff1cf2150f36ffa3b0dae40b17f-refs/branch-heads/5060@{#853}
OS | Android 13

Went through the STR/Cases outlined via #23492 (comment) & #23492 (comment) and couldn't reproduce the original issue. Also ensured that you can enable VPN and restore a subscription without any issues.


Verification PASSED on Galaxy Tab S8 Ultra running Android 12 using the following build(s):

Brave | 1.40.105 Chromium: 103.0.5060.53 (Official Build) (64-bit)
--- | ---
Revision | a1711811edd74ff1cf2150f36ffa3b0dae40b17f-refs/branch-heads/5060@{#853}
OS | Android 12; Build/SP1A.210812.016

Went through the STR/Cases outlined via #23492 (comment) & #23492 (comment) and couldn't reproduce the original issue. Also ensured that you can enable VPN and restore a subscription without any issues.

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.

2 participants