Skip to content

fix access violation when exiting viewer with playback device - #12201

Merged
maloel merged 1 commit into
realsenseai:developmentfrom
maloel:playback-crash
Sep 18, 2023
Merged

fix access violation when exiting viewer with playback device#12201
maloel merged 1 commit into
realsenseai:developmentfrom
maloel:playback-crash

Conversation

@maloel

@maloel maloel commented Sep 17, 2023

Copy link
Copy Markdown
Contributor

Add playback device
Close viewer

It's on exit, so user doesn't see much, but the debugger will complain and it's correct to crash.

Problem is we're using a reference to a variable that's declared outside a lambda that goes away.

Tracked on [LRS-880]

Add playback device
Close viewer

It's on exit, so user doesn't see much, but the debugger will complain
and it's correct to crash.

Problem is we're using a reference to a variable that's declared outside
a lambda that goes away.
@maloel
maloel requested a review from Nir-Az September 17, 2023 05:58

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@maloel
maloel merged commit 71d1e14 into realsenseai:development Sep 18, 2023
@maloel
maloel deleted the playback-crash branch September 18, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants