Menu

#101 Intercom_Control_PDU Decoding

3.0.0
closed-works-for-me
nobody
None
5
2022-01-25
2021-04-15
Anonymous
No

While testing the DIS7 interface I noticed that the Intercom_Control_PDU was not getting decoded correctly. Upon further inspection I saw that it was trying to pull the header off twice.
On line 337 of Intercom_Control_PDU.cpp
I replaced
Header::Decode( stream );
with
Header::Decode( stream, ignoreHeader);

and it works now.

Discussion

  • Karl  Jones

    Karl Jones - 2021-04-16

    Hmm I just checked and this is correct. Are you using an old version of KDIS?
    The latest master branch already has this fix. I fixed it in 2017 7963d9bd

     
  • Karl  Jones

    Karl Jones - 2021-04-16
    • status: open --> closed-works-for-me
     
  • Anonymous

    Anonymous - 2021-04-16

    I was using the 2_9_0 release.... I jsut downloaded the latest adn I can see its fixed. Thanks for the response

     
  • Karl  Jones

    Karl Jones - 2022-01-25

    Ah yes I do need to put together a new release :)

     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB