Menu

#55 Check for error after _SetHandleSize

open
Other (8)
5
2008-01-28
2008-01-28
No

AddToHandle and AddToQueue within I.QLIB call _SetHandleSize without checking for errors.
In conditions when the queue buffers are filled, these calls returned without error which had the effect of increasing the length recorded inside the handle without actually extending the handle size.
This subsequently caused problems with attempts by PtrToPtr to copy data into/from the handle expecting that the length was ok.
PtrToPtr checks the length and calls _SysFail if there is a problem.
This is probably the cause of a few other issues.

Discussion

  • Andrew Roughan

    Andrew Roughan - 2008-01-28

    Logged In: YES
    user_id=257705
    Originator: YES

    Calls to AddToHandle and AddToQueue are made from I.IP, I.ICMP, I.TCP. All calls should be followed by a check for carry set and appropriate action (e.g. purge the packet)

     
  • Andrew Roughan

    Andrew Roughan - 2008-01-28

    Logged In: YES
    user_id=257705
    Originator: YES

    Other calls with similar problem are AddToICMPQueue, InsertQueue and InsertDataIntoHandle.

    I.DNR is also affected.

    Some points in the code of I.DNR and I.IP need more investigation as to what to do. Search for "not sure where to go".

     
  • Andrew Roughan

    Andrew Roughan - 2008-01-28
    • assigned_to: roughana --> kashum
     
  • Andrew Roughan

    Andrew Roughan - 2008-01-28

    Logged In: YES
    user_id=257705
    Originator: YES

    I.DNR resolved.

    Some points in the code of I.IP need more investigation as to
    what to do. Search for "not sure where to go".

    Source code can be viewed within CVS:
    http://marinetti.cvs.sourceforge.net/marinetti/MOSP/Marinetti/Init/

     
  • Andrew Roughan

    Andrew Roughan - 2008-01-29

    Logged In: YES
    user_id=257705
    Originator: YES

    When this issue has been resolved, retest this issue:
    45 Casper causes PtrToPtr failure

     

    Last edit: Andrew Roughan 2017-02-14
  • Andrew Roughan

    Andrew Roughan - 2008-02-10

    Logged In: YES
    user_id=257705
    Originator: YES

    Code tested and minor changes made to clear carry before returning (doh!) when successful.

     

Log in to post a comment.

MongoDB Logo MongoDB