Wireshark · Ethereal-dev: [Ethereal-dev] Patch for inap dissector.

Ethereal-dev: [Ethereal-dev] Patch for inap dissector.

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date: Fri, 1 Apr 2005 15:18:48 +0100

Title: Patch for inap dissector.

This is a considerable upgrade on previous.

1) Return errors.
2) Return rejects.

Numerous other bugs fixed.

Index: packet-inap-template.c
===================================================================
--- packet-inap-template.c      (revision 13952)
+++ packet-inap-template.c      (working copy)
@@ -52,17 +52,26 @@
static int hf_inap_absent = -1;                /* NULL */
static int hf_inap_invokeId = -1;              /* InvokeId */
static int hf_inap_invoke = -1;                /* InvokePDU */
+static int hf_inap_ReturnError = -1;                /* InvokePDU */
static int hf_inap_returnResult = -1;                /* InvokePDU */
static int hf_inap_returnResult_result = -1;
static int hf_inap_getPassword = -1;
static int hf_inap_currentPassword = -1;
+static int hf_inap_genproblem = -1;
#include "packet-inap-hf.c"
-static guint global_tcap_itu_ssn = 241;

+static guint tcap_itu_ssn = 106;
+static guint tcap_itu_ssn1 = 241;
+
+
+static guint global_tcap_itu_ssn = 1;
+static guint global_tcap_itu_ssn1 = 1;
+
/* Initialize the subtree pointers */
static gint ett_inap = -1;
static gint ett_inap_InvokeId = -1;
static gint ett_inap_InvokePDU = -1;
+static gint ett_inap_ReturnErrorPDU = -1;
static gint ett_inap_ReturnResultPDU = -1;
static gint ett_inap_ReturnResult_result = -1;
static gint ett_inap_INAPPDU = -1;
@@ -81,6 +90,7 @@
{45, "CallInformationRequest"},
{53, "Cancel"},
{20, "Connect"},
+{18, "DisconnectForwardConnection"},
        {19,"ConnectToResource"},
        {17,"EstablishTemporaryConnection"},
        {24,"EventReportBCSM"},
@@ -91,10 +101,37 @@
        {99,"ReceivedInformation"}, /*???????*/
        {33,"ResetTimer"},
        {23,"RequestReportBCSMEvent"},
-       {49,"SpecializedResourceReport"}
+       {49,"SpecializedResourceReport"},
+       {22,"ReleaseCall"}
};

+const value_string inap_error_code_strings[] = {

+{0,"cancelled"},
+{1,"cancelFailed"},
+{3,"etcFailed"},
+{4,"improperCallerResponse"},
+{6,"missingCustomerRecord"},
+{7,"missingParameter"},
+{8,"parameterOutOfRange"},
+{10,"RequestedInfoError"},
+{11,"SystemFailure"},
+{12,"TaskRefused"},
+{13,"UnavailableResource"},
+{14,"UnexpectedComponentSequence"},
+{15,"UnexpectedDataValue"},
+{16,"UnexpectedParameter"},
+{17,"UnknownLegID"}
+};
+
+const value_string inap_general_problem_strings[] = {
+{0,"General Problem Unrecognized Component"},
+{1,"General Problem Mistyped Component"},
+{3,"General Problem Badly Structured Component"},
+};
+
+
+
static guint32 opcode=0;

static int
@@ -108,6 +145,20 @@
   return offset;
}

+
+
+static int
+dissect_inap_errorCode(gboolean implicit_tag _U_, tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, int hf_index) {

+ offset = dissect_ber_integer(FALSE, pinfo, tree, tvb, offset, hf_index, &opcode);
+
+ if (check_col(pinfo->cinfo, COL_INFO)){
+    col_set_str(pinfo->cinfo, COL_INFO, val_to_str(opcode, inap_error_code_strings, "Unknown Inap (%u)"));
+ }
+
+ return offset;
+}
+
+
static int dissect_invokeData(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
   switch(opcode){
   case 16: /*AssistRequestInstructions*/
@@ -125,7 +176,7 @@
   case 20: /*Connect*/
     offset=dissect_inap_Connectarg(FALSE, tvb, offset, pinfo, tree, -1);
     break;
-    case 18: /*DisconnectForwardConnections*/
+ case 18: /*DisconnectForwardConnections*/
     proto_tree_add_text(tree, tvb, offset, -1, "Disconnect Forward Connection");
     break;
   case 19: /*ConnectToResource*/
@@ -157,6 +208,9 @@
   case 33: /*ResetTimer*/
     offset=dissect_inap_ResetTimer(FALSE, tvb, offset, pinfo, tree, -1);
     break;
+   case 22: /*ResetTimer*/
+    offset=dissect_inap_ReleaseCallArg(FALSE, tvb, offset, pinfo, tree, -1);
+    break;
    default:
     proto_tree_add_text(tree, tvb, offset, -1, "Unknown invokeData blob");
     /* todo call the asn.1 dissector */
@@ -181,6 +235,12 @@
   return dissect_inap_Opcode(FALSE, tvb, offset, pinfo, tree, hf_inap_invokeCmd);
}

+
+static int
+dissect_errorCode(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
+ return dissect_inap_errorCode(FALSE, tvb, offset, pinfo, tree, hf_inap_ReturnError);
+}
+
static int dissect_invokeid(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
   return dissect_ber_integer(FALSE, pinfo, tree, tvb, offset, hf_inap_invokeid, NULL);
}
@@ -228,10 +288,31 @@

   return offset;
}
+
+
+static const ber_sequence_t returnErrorPDU_sequence[] = {
+ { BER_CLASS_UNI, -1/*choice*/, BER_FLAGS_NOOWNTAG|BER_FLAGS_NOTCHKTAG, dissect_invokeId },
+ { BER_CLASS_UNI, BER_UNI_TAG_INTEGER, BER_FLAGS_NOOWNTAG, dissect_errorCode },
+ { 0, 0, 0, NULL }
+};
+
+static int
+dissect_inap_returnErrorPDU(gboolean implicit_tag _U_, tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, int hf_index) {

+ offset = dissect_ber_sequence(implicit_tag, pinfo, tree, tvb, offset,
+                                returnErrorPDU_sequence, hf_index, ett_inap_ReturnErrorPDU);
+
+ return offset;
+}
+
+
static int dissect_invoke_impl(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
   return dissect_inap_InvokePDU(TRUE, tvb, offset, pinfo, tree, hf_inap_invoke);
}

+static int dissect_returnError_impl(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
+ return dissect_inap_returnErrorPDU(TRUE, tvb, offset, pinfo, tree, hf_inap_invoke);
+}
+
static const ber_sequence_t ReturnResult_result_sequence[] = {
   { BER_CLASS_UNI, BER_UNI_TAG_INTEGER, BER_FLAGS_NOOWNTAG, dissect_invokeCmd },
   { BER_CLASS_UNI, -1/*depends on Cmd*/, BER_FLAGS_NOOWNTAG|BER_FLAGS_NOTCHKTAG, dissect_returnResultData },
@@ -262,6 +343,11 @@
   return dissect_inap_returnResultPDU(TRUE, tvb, offset, pinfo, tree, hf_inap_returnResult);
}

+
+static int dissect_reject_impl(packet_info *pinfo, proto_tree *tree, tvbuff_t *tvb, int offset) {
+ return dissect_inap_RejectPDU(TRUE, tvb, offset, pinfo, tree, -1);
+}
+
static const value_string INAPPDU_vals[] = {
   {   1, "invoke" },
   {   2, "returnResult" },
@@ -273,10 +359,8 @@
static const ber_choice_t INAPPDU_choice[] = {
   {   1, BER_CLASS_CON, 1, BER_FLAGS_IMPLTAG, dissect_invoke_impl },
   {   2, BER_CLASS_CON, 2, BER_FLAGS_IMPLTAG, dissect_returnResult_impl },
-#ifdef REMOVED
   {   3, BER_CLASS_CON, 3, BER_FLAGS_IMPLTAG, dissect_returnError_impl },
   {   4, BER_CLASS_CON, 4, BER_FLAGS_IMPLTAG, dissect_reject_impl },
-#endif
   { 0, 0, 0, 0, NULL }
};

@@ -329,9 +413,21 @@
/*--- proto_reg_handoff_inap ---------------------------------------*/
void proto_reg_handoff_inap(void) {
     dissector_handle_t inap_handle;
-
+       static int inap_prefs_initialized = FALSE;
+
     inap_handle = create_dissector_handle(dissect_inap, proto_inap);
+
+       if (!inap_prefs_initialized) {
+               inap_prefs_initialized = TRUE;
+       }
+       else {
+               dissector_delete("tcap.itu_ssn", tcap_itu_ssn, inap_handle);
+               dissector_delete("tcap.itu_ssn", tcap_itu_ssn1, inap_handle);
+       }
+       tcap_itu_ssn = global_tcap_itu_ssn;
+       tcap_itu_ssn1 = global_tcap_itu_ssn1;
     dissector_add("tcap.itu_ssn", global_tcap_itu_ssn, inap_handle);
+    dissector_add("tcap.itu_ssn", global_tcap_itu_ssn1, inap_handle);

}

@@ -344,7 +440,11 @@
       { "invokeCmd", "inap.invokeCmd",
         FT_UINT32, BASE_DEC, VALS(inap_opr_code_strings), 0,
         "InvokePDU/invokeCmd", HFILL }},
-    { &hf_inap_invokeid,
+   { &hf_inap_ReturnError,
+      { "ReturnError", "inap.ReturnError",
+        FT_UINT32, BASE_DEC, VALS(inap_error_code_strings), 0,
+        "InvokePDU/ReturnError", HFILL }},
+     { &hf_inap_invokeid,
       { "invokeid", "inap.invokeid",
         FT_INT32, BASE_DEC, NULL, 0,
         "InvokeId/invokeid", HFILL }},
@@ -379,6 +479,7 @@
     &ett_inap,
     &ett_inap_InvokeId,
     &ett_inap_InvokePDU,
+    &ett_inap_ReturnErrorPDU,
     &ett_inap_ReturnResultPDU,
     &ett_inap_ReturnResult_result,
     &ett_inap_INAPPDU,
@@ -393,6 +494,7 @@
   proto_register_field_array(proto_inap, hf, array_length(hf));
   proto_register_subtree_array(ett, array_length(ett));

+
   inap_module = prefs_register_protocol(proto_inap, proto_reg_handoff_inap);

   prefs_register_uint_preference(inap_module, "tcap.itu_ssn",
@@ -400,6 +502,10 @@
                "Set Subsystem number used for INAP",
                10, &global_tcap_itu_ssn);

+ prefs_register_uint_preference(inap_module, "tcap.itu_ssn1",
+               "Subsystem number used for INAP",
+               "Set Subsystem number used for INAP",
+               10, &global_tcap_itu_ssn1);

}

Index: inap.asn
===================================================================
--- inap.asn    (revision 13952)
+++ inap.asn    (working copy)
@@ -344,7 +344,8 @@
        calledAddress (3),
        releaseCause (30)},
        correlationID [1] IMPLICIT OCTET STRING OPTIONAL,
-       extensions [2] IMPLICIT Extensions OPTIONAL}
+       extensions [2] IMPLICIT Extensions OPTIONAL,
+       legID                         [3] LegID OPTIONAL }

Cancelarg ::= CHOICE {
        invokeID [0] IMPLICIT INTEGER (-128..127),
@@ -642,7 +643,15 @@
ResourceStatus ::= ENUMERATED {
        busy (0),
        idle (1)}
+
ReleaseCall ::=        OCTET STRING (SIZE (2..10))
+
+ReleaseCallArg ::= CHOICE {
+       initialCallSegment Cause ,
+       allCallSegments [2] SEQUENCE {
+               releaseCause [0] Cause OPTIONAL }
+}
+
RequestCurrentStatusReportarg ::=      CHOICE {
        lineID [0] IMPLICIT OCTET STRING ,
        facilityGroupID [1] FacilityGroupID,
@@ -858,4 +867,44 @@
--unknownResource ERROR
-- ::= localValue 18

+-- additional pdus for reject
+
+InvokeIDType ::= INTEGER
+
+RejectPDU ::= SEQUENCE {
+               rinvokeID CHOICE {
+                               invidtype InvokeIDType,
+                               null NULL},
+               rproblem CHOICE {
+                               gp [0] IMPLICIT GeneralProblem,
+                               ip [1] IMPLICIT InvokeProblem,
+                               rrp [2] IMPLICIT ReturnResultProblem,
+                               rep [3] IMPLICIT ReturnErrorProblem}}
+
+GeneralProblem ::= INTEGER { -- ROSE-provider detected
+               unrecognisedAPDU(0),
+               mistypedAPDU(1),
+               badlyStructuredAPDU(2)}
+InvokeProblem ::= INTEGER { -- ROSE-user detected
+               duplicateInvocation(0),
+               unrecognisedOperation(1),
+               mistypedArgument(2),
+               resourceLimitation(3),
+               initiatorReleasing(4),
+               unrecognisedLinkedID(5),
+               linkedResponseUnexpected(6),
+               unexpectedChildOperation(7)}
+ReturnResultProblem ::= INTEGER { -- ROSE-user detected
+               unrecognisedInvocation(0),
+               resultResponseUnexpected(1),
+               mistypedResult(2)}
+ReturnErrorProblem ::= INTEGER { -- ROSE-user detected
+               unrecognisedInvocation(0),
+               errorResponseUnexpected(1),
+               unrecognisedError(2),
+               unexpectedError(3),
+               mistypedParameter(4)}
+
+
+
END
\ No newline at end of file

Follow-Ups:
- [Ethereal-dev] Re: Patch for inap dissector.
  - From: ronnie sahlberg

Prev by Date: Re: R: [Ethereal-dev] New columns with "display filter contents"?
Next by Date: [Ethereal-dev] Radius tunnel password decryption patch
Previous by thread: Re: R: [Ethereal-dev] New columns with "display filter contents"?
Next by thread: [Ethereal-dev] Re: Patch for inap dissector.
Index(es):
- Date
- Thread