Menu

#74 TCPIPSendIPDatagram bad

3.0b10
pending-fixed
None
2
2019-04-25
2018-10-22
Ewen Wannop
No

I cannot get TCPIPSendIPDatagram to work. It does not raise any error codes, and apparently returns normally from the call, but with a correctly formatted Datagram it does not actually send anything out to the Link layer, and causes memory corruption afterwards resulting in random hangs and crashes.

According to the Manual, the TCPIPSendIPDatagram call should be calculating the source IP Address, and the checksum for the main part of the datagram, but as nothing gets to the Link Layer, I can’t see whether it is doing that or not.

Discussion

  • Andrew Roughan

    Andrew Roughan - 2018-10-22
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1,3 @@
    -I cannot get TCPIPSendIPDatagram to work. It does not raise any error codes, and apparently returns normally from the call,  but does not actually send anything out to the Link layer, and causes memory corruption afterwards. 
    +I cannot get TCPIPSendIPDatagram to work. It does not raise any error codes, and apparently returns normally from the call,  but with a correctly formatted Datagram it does not actually send anything out to the Link layer, and causes memory corruption afterwards resulting in random hangs and crashes.
    +
    +According to the Manual, the TCPIPSendIPDatagram call should be calculating the source IP Address, and the checksum for the main part of the datagram, but as nothing gets to the Link Layer, I can’t see whether it is doing that or not.
    
    • assigned_to: Andrew Roughan
    • Group: 3.0b5 --> 3.0b10
    • Priority: 5 --> 2
     
  • Andrew Roughan

    Andrew Roughan - 2018-11-04

    Ticket moved from /p/marinetti/feature-requests/38/

     
  • Andrew Roughan

    Andrew Roughan - 2018-11-11

    IPSENDDATAGRAM Usage of OpenDatagram requires correct DP to be set and then the local variable usage requires the original DP to be restored. PHD and PLD added around OpenDatagram and another LDA >MyDP TCD inserted prior to WriteDatagram to be consistent with other usages of OpenDatagram and WriteDatagram.

    AFAICT This has never been correct. There was old code commented out which used SendDatagram directly. Looks like Richard was converting and never tested.

    Internal note: PC;GSport; NewDev2; testbuild; I.IP.B74.S

     
  • Andrew Roughan

    Andrew Roughan - 2019-04-25
    • status: open --> pending-fixed
     

Log in to post a comment.

MongoDB Logo MongoDB