Skip to content

Various undefined behaviour fixes#786

Merged
ktmf01 merged 2 commits intoxiph:masterfrom
ktmf01:fix-undefined
Jan 21, 2025
Merged

Various undefined behaviour fixes#786
ktmf01 merged 2 commits intoxiph:masterfrom
ktmf01:fix-undefined

Conversation

@ktmf01
Copy link
Collaborator

@ktmf01 ktmf01 commented Jan 21, 2025

Fixes #784

ktmf01 and others added 2 commits January 17, 2025 09:46
Fixes xiph#784

Co-authored-by: braheezy <11700503+braheezy@users.noreply.github.com>
As xiph#784 notified me of
undefined behaviour going undetected by the fuzzers, I ran the test
suite with the full undefined behaviour sanitizer enabled.

There is one bit of undefined behaviour that I cannot fix,
FLAC__metadata_get_picture allows a picture type of -1 to mean all,
but this isn't part of the enum.
@ktmf01 ktmf01 merged commit e1f62eb into xiph:master Jan 21, 2025
16 checks passed
@ktmf01 ktmf01 deleted the fix-undefined branch February 12, 2025 08:06
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.

Possible undefined behavior in find_best_partition_order_()

1 participant