DeviceController audio memory leak

What happened and what did you expect to happen?

I was profiling a simple React component that only imports the chime JS sdk. It simply creates a new DefaultDeviceController when mounted, and calls the .destroy() method on unmount.

It seems that AudioBufferSourceNode persists and a new one is created every time the component is mounted.

This was discovered using heap snapshots in Chrome.

Heap snapshot of initially unmounted component that is mounted and unmounted, repeated twice (Unmounted, Mounted, Unmounted, Mounted, Unmounted):

Have you reviewed our existing documentation?

• GitHub issues
• FAQs page
• README page
• Feature-specific guides

Reproduction steps

1. Create a new DefaultDeviceController
2. Call .destroy() on the created instance
3. Create a new instance of DefaultDeviceController again
4. Memory leak is observed with an additional AudioBufferSourceNode

Amazon Chime SDK for JavaScript version

2.17.0

What browsers are you seeing the problem on?

Chrome

Browser version

94.0.4606.71

Meeting and Attendee ID Information.

No response

Browser console logs

N/A

[emopti-jrufer]

I am running into the same issue. I am not using using the React components but rather using the SDK in an Angular app. What I discovered when trying to figure out why the memory in our app was increasing after every meeting is that the “AudioBufferSourceNode” objects were being created during the teardown of the meeting. Initially followed the convention used in the examples for tearing down by calling MeetingSession.audioVideo.chooseXXXDevice(null). The chooseAudioInputDevice(null) is creating a the empty audio device. Next tried calling MeetingSession.destroy(), which is not documented anywhere, instead of calling chooseAudioInputDevice(null). This resulted in the same behavior because it is doing the same thing.

Overall there needs to be a short circuit when calling chooseAudioInputDevice(null) during destroy so it does not create new resources that are never cleaned up. The following reference is what is creating the empty device during destroy.

https://github.com/aws/amazon-chime-sdk-js/blob/17a5fc7163cf020deeaa90ff45ee414188504862/src/devicecontroller/DefaultDeviceController.ts#L1465

[xuesichao]

@emopti-jrufer Hi, thanks for providing such a detailed observation. We are working on a fix now.