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

Restrict payload bulk transfer size to be exactly equal to the image frame size #197

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dmikushin
Copy link

fix #196

I have no idea what are those payload_final2_size1 and payload_final2_size2 are. But it seems that if we set the payload size to a larger value than the image frame size, the trailer packet is also appended to the payload transfer. The trailer is expected to be a separate transfer though. We set the maximum transfer size to the frame size (which is effectively equal to the value of payload_final2_size1) - this guarantees that the trailer will be always received in another separate bulk transfer.

This problem could be specific to MV-A5031MU815 camera.

Changelog

  • Restrict payload bulk transfer size to be exactly equal to the image frame size, otherwise the trailer buffer is appended to the end of payload buffer

…_size2 are.

But it seems that if we set the payload size to a larger value than the frames size,
the trailer packet is also appended to the payload transfer. The trailer is expected
to be a separate transfer though. We set the maximum transfer size to the frame size
(which is effectively equal to the value of payload_final2_size1) - this guarantees
that the trailer will be always received in another separate bulk transfer.
@strohel strohel changed the title I have no idea what are those payload_final2_size1 and payload_final2… Restrict payload bulk transfer size to be exactly equal to the image frame size Nov 5, 2024
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.

Crash due to TRAILER is unexpectedly appended to the end of PAYLOAD buffer
1 participant