Skip to content

createVPCodecParametersString zero-pads bitDepth based on transferCharacteristics#66863

Open
charliewolfe wants to merge 1 commit into
WebKit:mainfrom
charliewolfe:eng/createVPCodecParametersString-zero-pads-bitDepth-based-on-transferCharacteristics
Open

createVPCodecParametersString zero-pads bitDepth based on transferCharacteristics#66863
charliewolfe wants to merge 1 commit into
WebKit:mainfrom
charliewolfe:eng/createVPCodecParametersString-zero-pads-bitDepth-based-on-transferCharacteristics

Conversation

@charliewolfe

@charliewolfe charliewolfe commented Jun 10, 2026

Copy link
Copy Markdown
Member

2b65059

createVPCodecParametersString zero-pads bitDepth based on transferCharacteristics
https://bugs.webkit.org/show_bug.cgi?id=316764
rdar://179210193

Reviewed by NOBODY (OOPS!).

The leading-zero guard for the two-digit bitDepth field tested configuration.transferCharacteristics
instead of configuration.bitDepth. This produced a malformed codec string. Test bitDepth.

* Source/WebCore/platform/graphics/VP9Utilities.cpp:
(WebCore::createVPCodecParametersString):

2b65059

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ⏳ 🛠 win ⏳ 🛠 ios-apple
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ⏳ 🧪 win-tests ⏳ 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ⏳ 🛠 vision-apple
✅ 🧪 ios-wk2-wpt ✅ 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🛠 ios-safer-cpp ✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 vision-sim ✅ 🛠 playstation
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv ✅ 🛠 mac-safer-cpp
✅ 🛠 tv-sim ❌ 🧪 mac-site-isolation
✅ 🛠 watch
✅ 🛠 watch-sim

…racteristics

https://bugs.webkit.org/show_bug.cgi?id=316764
rdar://179210193

Reviewed by NOBODY (OOPS!).

The leading-zero guard for the two-digit bitDepth field tested configuration.transferCharacteristics
instead of configuration.bitDepth. This produced a malformed codec string. Test bitDepth.

* Source/WebCore/platform/graphics/VP9Utilities.cpp:
(WebCore::createVPCodecParametersString):
@charliewolfe charliewolfe self-assigned this Jun 10, 2026
@charliewolfe charliewolfe added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Bugs Unclassified bugs are placed in this component until the correct component can be determined.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants