- Some minor asn1 corrections

- Dissect Enumerated correctly
- Add message to info col.

svn path=/trunk/; revision=18117

3 files changed, 72 insertions, 19 deletions

diff --git a/asn1/ulp/ULP.asn b/asn1/ulp/ULP.asn
index 93ce0f6582..12bbbf7828 100644
--- a/asn1/ulp/ULP.asn
+++ b/asn1/ulp/ULP.asn
@@ -103,8 +103,12 @@ LocationId ::= SEQUENCE {
 	...
 }
 
-Status ::= ENUMERATED {stale(0), current(1), unknown(2), ...
-                       }
+Status ::= ENUMERATED {
+	stale(0),
+	current(1),
+	unknown(2), 
+	...
+	}
 
 CellInfo ::= CHOICE {
   gsmCell    GsmCellInformation,
@@ -180,8 +184,10 @@ FrequencyInfoFDD ::= SEQUENCE {
   uarfcn-DL  UARFCN,
   ...}
 
-FrequencyInfoTDD ::= SEQUENCE {uarfcn-Nt  UARFCN,
-                               ...}
+FrequencyInfoTDD ::= SEQUENCE {
+	uarfcn-Nt  UARFCN,
+	...
+	}
 
 UARFCN ::= INTEGER(0..16383)
 
@@ -255,13 +261,27 @@ maxFreq INTEGER ::= 8
 maxTS INTEGER ::= 14
 
 StatusCode ::= ENUMERATED {
-  unspecified(0), systemFailure(1), unexpectedMessage(2), protocolError(3),
-  dataMissing(4), unexpectedDataValue(5), posMethodFailure(6),
-  posMethodMismatch(7), posProtocolMismatch(8), targetSETnotReachable(9),
-  versionNotSupported(10), resourceShortage(11), invalidSessionId(12),
-  nonProxyModeNotSupported(13), proxyModeNotSupported(14),
-  positioningNotPermitted(15), authNetFailure(16), authSuplinitFailure(17), consentDeniedByUser(100),
-  consentGrantedByUser(101), ...
+  unspecified(0), 
+  systemFailure(1), 
+  unexpectedMessage(2), 
+  protocolError(3),
+  dataMissing(4), 
+  unexpectedDataValue(5), 
+  posMethodFailure(6),
+  posMethodMismatch(7), 
+  posProtocolMismatch(8), 
+  targetSETnotReachable(9),
+  versionNotSupported(10), 
+  resourceShortage(11), 
+  invalidSessionId(12),
+  nonProxyModeNotSupported(13), 
+  proxyModeNotSupported(14),
+  positioningNotPermitted(15), 
+  authNetFailure(16), 
+  authSuplinitFailure(17), 
+  consentDeniedByUser(100),
+  consentGrantedByUser(101), 
+  ...
   }
 
 QoP ::= SEQUENCE {
@@ -393,24 +413,37 @@ Notification ::= SEQUENCE {
   ...}
 
 NotificationType ::= ENUMERATED {
-  noNotificationNoVerification(0), notificationOnly(1),
+  noNotificationNoVerification(0), 
+  notificationOnly(1),
   notificationAndVerficationAllowedNA(2),
-  notificationAndVerficationDeniedNA(3), privacyOverride(4), ...
+  notificationAndVerficationDeniedNA(3), 
+  privacyOverride(4), 
+  ...
   }
 
-EncodingType ::= ENUMERATED {ucs2(0), gsmDefault(1), utf8(2), ...
-                             }
+EncodingType ::= ENUMERATED {
+	ucs2(0), 
+	gsmDefault(1), 
+	utf8(2), 
+	...
+	}
 
 maxReqLength INTEGER ::= 50
 
 maxClientLength INTEGER ::= 50
 
--- IMSPublicidentity is incorrect as identifiers must start with lower case char
 FormatIndicator ::= ENUMERATED {
-  logicalName(0), e-mailAddress(1), msisdn(2), url(3), sipUrl(4), min(5),
-  mdn(6), iMSPublicidentity(7), ...
+  logicalName(0), 
+  e-mailAddress(1), 
+  msisdn(2), 
+  url(3), 
+  sipUrl(4), 
+  min(5),
+  mdn(6), 
+  ...
   }
 
+
 SLPMode ::= ENUMERATED {proxy(0), nonProxy(1)}
 
 MAC ::= BIT STRING(SIZE (64))
diff --git a/asn1/ulp/packet-ulp-template.c b/asn1/ulp/packet-ulp-template.c
index 2bb2d4c07b..e2fc1dfa2e 100644
--- a/asn1/ulp/packet-ulp-template.c
+++ b/asn1/ulp/packet-ulp-template.c
@@ -64,6 +64,8 @@ static int proto_ulp = -1;
 static gint ett_ulp = -1;
 #include "packet-ulp-ett.c"
 
+guint32 StatusCode_value_map[20] = {0, 1, 2, 3, 4, 5, 6, 7, 8 ,9 ,10 ,11, 12, 13, 14, 15, 16, 17, 100 , 101};
+
 /* Include constants */
 #include "packet-ulp-val.h"
 
diff --git a/asn1/ulp/ulp.cnf b/asn1/ulp/ulp.cnf
index 52738e6f8d..2b854ee0c1 100644
--- a/asn1/ulp/ulp.cnf
+++ b/asn1/ulp/ulp.cnf
@@ -15,4 +15,22 @@ ULP-PDU
 	if (check_col(pinfo->cinfo, COL_INFO))
 		col_clear(pinfo->cinfo, COL_INFO);
 
-  %(DEFAULT_BODY)s
\ No newline at end of file
+  %(DEFAULT_BODY)s
+
+#.FN_BODY StatusCode
+# Remove once asn2eth is fixed to handle PER Enummeration extensions
+  offset = dissect_per_enumerated(tvb, offset, pinfo, tree, hf_index,
+									19, NULL, NULL, TRUE, 0, StatusCode_value_map);
+
+#.FN_PARS UlpMessage 	VAL_PTR = &UlpMessage
+
+#.FN_BODY UlpMessage
+
+guint32 UlpMessage;
+
+  %(DEFAULT_BODY)s
+
+	if (check_col(pinfo->cinfo, COL_INFO))
+	{
+	    col_append_fstr(pinfo->cinfo, COL_INFO, "%%s ", val_to_str(UlpMessage,ulp_UlpMessage_vals,"Unknown"));
+	}