Skip to content

Conversation

@jonoomph
Copy link
Member

@jonoomph jonoomph commented Apr 13, 2023

Protect child_clip_id from accessing NULL parentClip() pointer - which causes a crash when OpenShot attempts to update the JSON of an Object Detection effect with a child ID. Also fixing a GitHub CI issue, which is breaking the GitHub Checks/Actions.

@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #915 (2715d1c) into develop (c472a24) will decrease coverage by 0.53%.
The diff coverage is 13.04%.

@@             Coverage Diff             @@
##           develop     #915      +/-   ##
===========================================
- Coverage    54.21%   53.68%   -0.53%     
===========================================
  Files          190      182       -8     
  Lines        16592    16800     +208     
===========================================
+ Hits          8995     9019      +24     
- Misses        7597     7781     +184     
Impacted Files Coverage Δ
src/effects/ObjectDetection.cpp 0.00% <0.00%> (ø)
src/effects/Tracker.cpp 11.04% <0.00%> (-1.06%) ⬇️
src/TrackedObjectBBox.cpp 65.51% <66.66%> (-0.32%) ⬇️
src/Clip.cpp 69.89% <100.00%> (+0.10%) ⬆️
src/TrackedObjectBase.h 58.33% <100.00%> (+8.33%) ⬆️

... and 93 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jonoomph
Copy link
Member Author

@jonoomph jonoomph merged commit 9665093 into develop Apr 16, 2023
@jonoomph jonoomph deleted the sentry-invalid-property-data branch April 16, 2023 21:12
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