Axis2-C
  1. Axis2-C
  2. AXIS2C-1086

In function axis2_op_register_op_ctx() failure is logged and returned in a working path.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0
    • Component/s: None
    • Labels:
      None

      Description

      In the following code of axis2_op_register_op_ctx() function

      msg_id = axis2_msg_ctx_get_msg_id(msg_ctx, env);
      if (!msg_id)

      { AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, "Message id not found for message context while "\ "registering operation context for operation %s", opname); return AXIS2_FAILURE; }

      When message id is not found in the messages context it is correctly logged as error and returned failure. However every places from which this function called, status is not handled and although error is logged and failure returned still code work successfully. This is behaviour is not acceptable.

        Activity

        Dinesh Weerapurage made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 1.3.1 [ 12313091 ]
        Hide
        Dinesh Weerapurage added a comment -

        This was happened because of the incorrect sequence of the functions that we called. I fixed this issue and change the log level into warning.

        Show
        Dinesh Weerapurage added a comment - This was happened because of the incorrect sequence of the functions that we called. I fixed this issue and change the log level into warning.
        Dinesh Weerapurage made changes -
        Field Original Value New Value
        Assignee Dinesh Premalal [ xydinesh ]
        Damitha Kumarage created issue -

          People

          • Assignee:
            Dinesh Weerapurage
            Reporter:
            Damitha Kumarage
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development