Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Buffer Serialize struct length #199

Closed
wants to merge 1 commit into from

Conversation

jeremy-prater
Copy link
Contributor

@jeremy-prater jeremy-prater commented Jan 9, 2025

I noticed that rmp-serde and serde_cbor/ciborium both fail to deserialize the gstreamer::Buffer object because the encoded struct size is incorrect. There are 7 elements in the struct, but the serializer only specifies 6.

It is curious that serde_json worked correctly, but binary serialization did not.

@sdroege
Copy link
Owner

sdroege commented Jan 10, 2025

Thanks. This is only a mirror of the official repository. Can you create a MR here please: https://gitlab.freedesktop.org/gstreamer/gstreamer-rs ?

@sdroege sdroege closed this Jan 10, 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