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

[Bug]: Application crashes with EXCEPTION_ACCESS_VIOLATION_READ #32120

Closed
3 tasks done
juniopereirab opened this issue Dec 7, 2021 · 4 comments
Closed
3 tasks done

[Bug]: Application crashes with EXCEPTION_ACCESS_VIOLATION_READ #32120

juniopereirab opened this issue Dec 7, 2021 · 4 comments
Labels
blocked/need-repro Needs a test case to reproduce the bug bug 🪲

Comments

@juniopereirab
Copy link

Preflight Checklist

Electron Version

15.3.3

What operating system are you using?

Windows

Operating System Version

Windows 8 NT 6.3.9600 17415

What arch are you using?

x64

Last Known Working Electron version

No response

Expected Behavior

Application does not crash on Windows 8 and 7

Actual Behavior

Application crashes

Testcase Gist URL

No response

Additional Information

I am making a fancy notifier.

Basically I have an external API which is connected with my Electron Application using Websocket Protocol. Every a certain amount of time my API checks if a new order has arrived and if it does the API make an EMIT call to Electron App.

When the electron app receives that event, it calls that event from renderer process to the main.
And the main process restore or minimize the window with the notification sound.

Captura de Tela 2021-12-06 às 21 44 41

I setTImeout the restore function because at #21614 this was the cause of this bug. But the bug persists anyway.

I have configured a integration with BugSplat to see a potencial cause for this errors. And that's what I've got.

Stack Key

electron.exe!void base::ObserverListblink::MemoryUsageMonitor::Observer,0,1,base::internal::CheckedObserverAdapter::AddObserver(class blink::MemoryUsageMonitor::Observer *)(271)

Exception Code

EXCEPTION_ACCESS_VIOLATION_READ

Function

electron.exe!void base::ObserverListblink::MemoryUsageMonitor::Observer,0,1,base::internal::CheckedObserverAdapter::AddObserver(class blink::MemoryUsageMonitor::Observer *)

Location

observer_list.h(271)

Explanation

EXCEPTION_ACCESS_VIOLATION_READ

Captura de Tela 2021-12-06 às 21 37 16

And also a lot of others information

Captura de Tela 2021-12-06 às 21 38 07

Captura de Tela 2021-12-06 às 21 38 29

Captura de Tela 2021-12-06 às 21 39 33

@juniopereirab
Copy link
Author

It happens that on Preload file, I use path.join() function to get a audio file, and use the Audio() constructor from JavaScript to play a sound.
And I think the preload file was trying to access the audio's folder and emit this error, crashing the application.

@juniopereirab
Copy link
Author

After a lot of work around this problem, I figured out that the Electron App crashes when I call the function "play()" from an HTML Audio Element... At the renderer process it occurs.

I try to use the "sound-play" library at the main process but I had no success.

@codebytere
Copy link
Member

Thanks for reporting this and helping to make Electron better!

Because of time constraints, triaging code with third-party dependencies is usually not feasible for a small team like Electron's.

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

I'm adding the blocked/need-repro label for this reason. After you make a test case, please link to it in a followup comment. This issue will be closed in 10 days if the above is not addressed.

@codebytere codebytere added the blocked/need-repro Needs a test case to reproduce the bug label Jan 10, 2022
@codebytere
Copy link
Member

Fixed by #32046

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked/need-repro Needs a test case to reproduce the bug bug 🪲
Projects
None yet
Development

No branches or pull requests

2 participants