Skip to content

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

Open
dmikushin wants to merge 1 commit intocameleon-rs:mainfrom
dmikushin:restrict-payload-size
Open

Restrict payload bulk transfer size to be exactly equal to the image frame size?#197
dmikushin wants to merge 1 commit intocameleon-rs:mainfrom
dmikushin:restrict-payload-size

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
@strohel strohel changed the title Restrict payload bulk transfer size to be exactly equal to the image frame size Restrict payload bulk transfer size to be exactly equal to the image frame size? Mar 5, 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.

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

1 participant