aboutsummaryrefslogtreecommitdiffstats
path: root/asn1/p1
diff options
context:
space:
mode:
authorMichael Mann <mmann78@netscape.net>2013-06-14 12:44:50 +0000
committerMichael Mann <mmann78@netscape.net>2013-06-14 12:44:50 +0000
commit2dbf6c59a2f5d99d3dc8ea3d2ffa409ef2eb611a (patch)
treec021a26288d3d501a3dbb8de190cdc6a8e40ffa1 /asn1/p1
parent875040ec1e304110b2a7f1a6e9ead855d3be2e6f (diff)
Remove check_col() from ASN.1 dissectors
svn path=/trunk/; revision=49927
Diffstat (limited to 'asn1/p1')
-rw-r--r--asn1/p1/p1.cnf16
-rw-r--r--asn1/p1/packet-p1-template.c3
2 files changed, 7 insertions, 12 deletions
diff --git a/asn1/p1/p1.cnf b/asn1/p1/p1.cnf
index 1c9a6160f7..1c6b44a91e 100644
--- a/asn1/p1/p1.cnf
+++ b/asn1/p1/p1.cnf
@@ -689,7 +689,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
} else {
- if (check_col(actx->pinfo->cinfo, COL_INFO) && mtaname) {
+ if (mtaname) {
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " %%s", tvb_format_text(mtaname, 0, tvb_length(mtaname)));
}
@@ -1093,7 +1093,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
if(*oraddress) {
proto_item_append_text(address_item, " (%%s/", oraddress);
- if(doing_subjectid && check_col(actx->pinfo->cinfo, COL_INFO)) {
+ if(doing_subjectid) {
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " (%%s/", oraddress);
}
}
@@ -1111,7 +1111,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
if(doing_address)
proto_item_append_text(address_item, " $ %%s)", tvb_format_text(id, 0, tvb_length(id)));
- if(doing_subjectid && check_col(actx->pinfo->cinfo, COL_INFO))
+ if(doing_subjectid)
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " $ %%s)", tvb_format_text(id, 0, tvb_length(id)));
}
@@ -1206,7 +1206,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
#.FN_BODY MTABindError
int error = -1;
%(DEFAULT_BODY)s
- if((error != -1) && check_col(actx->pinfo->cinfo, COL_INFO))
+ if((error != -1))
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " (%%s)", val_to_str(error, p1_MTABindError_vals, "error(%%d)"));
#.FN_PARS TokenTypeIdentifier
@@ -1226,9 +1226,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
%(DEFAULT_BODY)s
if( (credentials!=-1) && p1_Credentials_vals[credentials].strptr ){
- if (check_col(actx->pinfo->cinfo, COL_INFO)) {
- col_append_fstr(actx->pinfo->cinfo, COL_INFO, " %%s", p1_Credentials_vals[credentials].strptr);
- }
+ col_append_fstr(actx->pinfo->cinfo, COL_INFO, " %%s", p1_Credentials_vals[credentials].strptr);
}
#.FN_PARS TokenDataType
@@ -1274,7 +1272,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
%(DEFAULT_BODY)s
if( (apdu!=-1) && p1_MTS_APDU_vals[apdu].strptr ){
- if(check_col(actx->pinfo->cinfo, COL_INFO) && (apdu != 0)) { /* we don't show "message" - sub-dissectors have better idea */
+ if(apdu != 0) { /* we don't show "message" - sub-dissectors have better idea */
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " %%s", p1_MTS_APDU_vals[apdu].strptr);
}
}
@@ -1288,9 +1286,7 @@ MessageToken B "2.6.1.7.36" "id-hat-forwarded-token"
%(DEFAULT_BODY)s
if( (report!=-1) && p1_ReportType_vals[report].strptr ){
- if(check_col(actx->pinfo->cinfo, COL_INFO)) {
col_append_fstr(actx->pinfo->cinfo, COL_INFO, " %%s", p1_ReportType_vals[report].strptr);
- }
}
#.FN_BODY MessageSubmissionArgument
diff --git a/asn1/p1/packet-p1-template.c b/asn1/p1/packet-p1-template.c
index bee6d586ca..cb6cb2a8c8 100644
--- a/asn1/p1/packet-p1-template.c
+++ b/asn1/p1/packet-p1-template.c
@@ -218,8 +218,7 @@ dissect_p1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree)
return;
}
- if (check_col(pinfo->cinfo, COL_INFO))
- col_set_str(pinfo->cinfo, COL_INFO, p1_op_name);
+ col_set_str(pinfo->cinfo, COL_INFO, p1_op_name);
while (tvb_reported_length_remaining(tvb, offset) > 0){
old_offset=offset;