Skip to content

Conversation

@Sakoram
Copy link
Collaborator

@Sakoram Sakoram commented Oct 23, 2025

The status of frames was changed to TX_FRAME_FREE before the frame_done() finished modifing the frame, which made it possible that a frame obtained by the get_frame() function was modified after it had been retrieved.

Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

Great catch
LGTM
( ദ്ദി ˙ᗜ˙ )

@Sakoram Sakoram force-pushed the Fix_race_condition_in_pipeline branch from b73061d to ecbacba Compare October 23, 2025 14:25
The status of frames was changed to TX_FRAME_FREE before the
frame_done() finished modifing the frame, which made it possible that a
frame obtained by the get_frame() function was modified after it had
been retrieved.

Signed-off-by: Kasiewicz, Marek <[email protected]>
@Sakoram Sakoram force-pushed the Fix_race_condition_in_pipeline branch from ecbacba to 3162420 Compare October 29, 2025 08:38
@Sakoram
Copy link
Collaborator Author

Sakoram commented Oct 29, 2025

continue in #1308

@Sakoram Sakoram closed this Oct 29, 2025
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