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

add OBEX EndOfBody in Final OK packet for fix assert of unused headers #24

Merged
merged 1 commit into from Apr 14, 2018

Conversation

Projects
None yet
2 participants
@FREEWING-JP
Contributor

FREEWING-JP commented Mar 29, 2018

add OBEX EndOfBody in Final OK packet for fix assert of unused headers

I tested OBEX send file from PC to Android SmartPhone .
In CheckResponse function received (Final | OK) header packet .
They are including EndOfBody header packet .
len = 8
receivePacket2[length = 8] = { 0xCB, 0x00, 0x00, 0x00, 0x01, 0x49, 0x00, 0X03 }

  • 0x49 = EndOfBody header (end of body length = 3)

So I add End Of Body packet in Final OK packet block .

By the way I was guessing Debug.Fail or return false when (eobLen != (len - 5)) .
Tentatively I choice Debug.Fail .

@peterfoot peterfoot merged commit a3cb8b3 into inthehand:master Apr 14, 2018

@FREEWING-JP FREEWING-JP deleted the FREEWING-JP:feature/fix_obex_web_request_final_ok_packet branch Apr 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment