Skip to content

fix(video): uninitialized lock - #748

Open
nickolay168 wants to merge 1 commit into
TokTok:masterfrom
nickolay168:nickolay168/fix_null_ptr
Open

fix(video): uninitialized lock#748
nickolay168 wants to merge 1 commit into
TokTok:masterfrom
nickolay168:nickolay168/fix_null_ptr

Conversation

@nickolay168

@nickolay168 nickolay168 commented Sep 2, 2026

Copy link
Copy Markdown

Fix the issue: #711
@grisutheguru, Thank you for the fix!


This change is Reviewable

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Tip

Preview URL:

@github-actions github-actions Bot added the bug Bug fix for the user, not a fix to a build script label Sep 2, 2026
@Green-Sky Green-Sky added this to the v1.18.6 milestone Sep 3, 2026
@Green-Sky

Copy link
Copy Markdown
Member

Technically the correct commit message is fix(video): uninitialized lock

@Green-Sky

Copy link
Copy Markdown
Member

Hm, tried to speed things up bc
Maintainers are allowed to edit this pull request.
but
ERROR: Permission to nickolay168/qTox_legacy.git denied to Green-Sky.

@nickolay168
nickolay168 force-pushed the nickolay168/fix_null_ptr branch from 6df437d to a6ed3bb Compare September 4, 2026 04:44
@nickolay168 nickolay168 changed the title fix(security): Fix uninitialized value. fix(video): uninitialized lock Sep 4, 2026
@nickolay168

Copy link
Copy Markdown
Author

Hm, tried to speed things up bc Maintainers are allowed to edit this pull request. but ERROR: Permission to nickolay168/qTox_legacy.git denied to Green-Sky.

Fixed. The Alpine build has failed, I have created the PR to fix: TokTok/ci-tools#211

@Green-Sky

Copy link
Copy Markdown
Member

Thanks, please rebase and then we wait on appimagetool update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

2 participants