Menu

#281 TRDP_NOSESSION_ERR should be returned from tlm_reply() and tlm_replyQuery() in case of incorrect session (id)

2.1.0.0
closed
None
2021-08-11
2019-10-10
No

It is expected that tlm_reply() and tlm_replyQuery() return TRDP_NO_SESSION_ERR when the session of the given session id does not exist. Currently, it does not return that but instead returns TRDP_NOLIST_ERR .

Analysis / Solution Proposal:

trdp_mdReply() returns TRDP_NOLIST_ERR if a corresponding MD element (with expected Session Id and State) is not found in appHandle->pMDRcvQueue which then goes to the user level as a return error from tlm_reply() or tlm_replyQuery().

It makes sense to return TRDP_NOSESSION_ERR in this case from trdp_mdLookupElement() itself, as TRDP_NOLIST_ERR has a different meaning that a listener is not added.

Discussion

  • Bernd Löhr

    Bernd Löhr - 2020-09-11
    • status: open --> pending
    • assigned_to: Bernd Löhr
     
  • Stefan Bender

    Stefan Bender - 2021-08-11

    Implementation added to trunk

     
  • Iris Bosse

    Iris Bosse - 2021-08-11
    • status: pending --> closed
     
  • Iris Bosse

    Iris Bosse - 2021-08-11

    error name changed (tested by inspection)

     

Log in to post a comment.

MongoDB Logo MongoDB