From 850f5d3be0707dc84661877d8e22eeadfa6e1189 Mon Sep 17 00:00:00 2001 From: Michael Mann Date: Mon, 25 Jan 2016 23:01:46 -0500 Subject: [PATCH] Remove the deprecated column types so users don't think they still exist. Bug: 11559 Change-Id: I51836dc9a4fa399835c7bdabcba577ebd40327ad Reviewed-on: https://code.wireshark.org/review/13538 Reviewed-by: Michael Mann Petri-Dish: Michael Mann Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman --- doc/README.dissector | 8 +- epan/column-utils.c | 3 +- epan/column-utils.h | 110 ++++++++---------- epan/column.c | 260 ++++++++++++++++++------------------------- epan/prefs.c | 66 ++++++----- 5 files changed, 202 insertions(+), 245 deletions(-) diff --git a/doc/README.dissector b/doc/README.dissector index 59fced396d..5b4f02388e 100644 --- a/doc/README.dissector +++ b/doc/README.dissector @@ -584,8 +584,7 @@ based on the time-value. For example: - nstime_delta(&ts, &pinfo->fd->abs_ts, &tcpd->ts_first); - col_set_time(pinfo->cinfo, COL_REL_CONV_TIME, &ts, "tcp.time_relative"); + col_set_time(pinfo->cinfo, COL_REL_TIME, &ts, "s4607.ploc.time"); 1.5 Constructing the protocol tree. @@ -2553,10 +2552,9 @@ selected in random order once the whole file has been read. After calculating the conversation timestamps, it is time to put them in the appropriate columns with the function 'col_set_time' (described in -section 1.5.9). There are two columns for conversation timestamps: +section 1.5.9). The column used for relative timestamps is: -COL_REL_CONV_TIME, /* Relative time to beginning of conversation */ -COL_DELTA_CONV_TIME,/* Delta time to last frame in conversation */ +COL_REL_TIME, /* Delta time to last frame in conversation */ Last but not least, there MUST be a preference in each dissector that uses conversation timestamps that makes it possible to enable and diff --git a/epan/column-utils.c b/epan/column-utils.c index f7bb4ea36c..ea932034b9 100644 --- a/epan/column-utils.c +++ b/epan/column-utils.c @@ -1789,8 +1789,7 @@ col_set_fmt_time(const frame_data *fd, column_info *cinfo, const gint fmt, const /* --------------------------- */ /* Set the given (relative) time to a column element. * - * Used by multiple dissectors to set the time in the column - * COL_DELTA_CONV_TIME + * Used by dissectors to set the time in a column * * @param cinfo the current packet row * @param el the column to use, e.g. COL_INFO diff --git a/epan/column-utils.h b/epan/column-utils.h index 5c5a83cd7b..9e2bf3cbc2 100644 --- a/epan/column-utils.h +++ b/epan/column-utils.h @@ -55,66 +55,53 @@ enum { COL_ABS_YMD_TIME, /**< 1) Absolute date, as YYYY-MM-DD, and time */ COL_ABS_YDOY_TIME, /**< 2) Absolute date, as YYYY/DOY, and time */ COL_ABS_TIME, /**< 3) Absolute time */ - COL_CIRCUIT_ID, /**< 4) Circuit ID */ - COL_DSTIDX, /**< 5) !! DEPRECATED !! - Dst port idx - Cisco MDS-specific */ - COL_SRCIDX, /**< 6) !! DEPRECATED !! - Src port idx - Cisco MDS-specific */ - COL_VSAN, /**< 7) VSAN - Cisco MDS-specific */ - COL_CUMULATIVE_BYTES, /**< 8) Cumulative number of bytes */ - COL_CUSTOM, /**< 9) Custom column (any filter name's contents) */ - COL_DCE_CALL, /**< 10) DCE/RPC connection oriented call id OR datagram sequence number */ - COL_DCE_CTX, /**< 11) !! DEPRECATED !! - DCE/RPC connection oriented context id */ - COL_DELTA_TIME, /**< 12) Delta time */ - COL_DELTA_CONV_TIME,/**< 13) Delta time to last frame in conversation */ - COL_DELTA_TIME_DIS, /**< 14) Delta time displayed*/ - COL_RES_DST, /**< 15) Resolved dest */ - COL_UNRES_DST, /**< 16) Unresolved dest */ - COL_RES_DST_PORT, /**< 17) Resolved dest port */ - COL_UNRES_DST_PORT, /**< 18) Unresolved dest port */ - COL_DEF_DST, /**< 19) Destination address */ - COL_DEF_DST_PORT, /**< 20) Destination port */ - COL_EXPERT, /**< 21) Expert Info */ - COL_IF_DIR, /**< 22) FW-1 monitor interface/direction */ - COL_OXID, /**< 23) !! DEPRECATED !! - Fibre Channel OXID */ - COL_RXID, /**< 24) !! DEPRECATED !! - Fibre Channel RXID */ - COL_FR_DLCI, /**< 25) !! DEPRECATED !! - Frame Relay DLCI */ - COL_FREQ_CHAN, /**< 26) IEEE 802.11 (and WiMax?) - Channel */ - COL_BSSGP_TLLI, /**< 27) !! DEPRECATED !! - GPRS BSSGP IE TLLI */ - COL_HPUX_DEVID, /**< 28) !! DEPRECATED !! - HP-UX Nettl Device ID */ - COL_HPUX_SUBSYS, /**< 29) !! DEPRECATED !! - HP-UX Nettl Subsystem */ - COL_DEF_DL_DST, /**< 30) Data link layer dest address */ - COL_DEF_DL_SRC, /**< 31) Data link layer source address */ - COL_RES_DL_DST, /**< 32) Resolved DL dest */ - COL_UNRES_DL_DST, /**< 33) Unresolved DL dest */ - COL_RES_DL_SRC, /**< 34) Resolved DL source */ - COL_UNRES_DL_SRC, /**< 35) Unresolved DL source */ - COL_RSSI, /**< 36) IEEE 802.11 - received signal strength */ - COL_TX_RATE, /**< 37) IEEE 802.11 - TX rate in Mbps */ - COL_DSCP_VALUE, /**< 38) IP DSCP Value */ - COL_INFO, /**< 39) Description */ - COL_COS_VALUE, /**< 40) !! DEPRECATED !! - L2 COS Value */ - COL_RES_NET_DST, /**< 41) Resolved net dest */ - COL_UNRES_NET_DST, /**< 42) Unresolved net dest */ - COL_RES_NET_SRC, /**< 43) Resolved net source */ - COL_UNRES_NET_SRC, /**< 44) Unresolved net source */ - COL_DEF_NET_DST, /**< 45) Network layer dest address */ - COL_DEF_NET_SRC, /**< 46) Network layer source address */ - COL_NUMBER, /**< 47) Packet list item number */ - COL_PACKET_LENGTH, /**< 48) Packet length in bytes */ - COL_PROTOCOL, /**< 49) Protocol */ - COL_REL_TIME, /**< 50) Relative time */ - COL_REL_CONV_TIME, /**< 51) !! DEPRECATED !! - Relative time to beginning of conversation */ - COL_DEF_SRC, /**< 52) Source address */ - COL_DEF_SRC_PORT, /**< 53) Source port */ - COL_RES_SRC, /**< 54) Resolved source */ - COL_UNRES_SRC, /**< 55) Unresolved source */ - COL_RES_SRC_PORT, /**< 56) Resolved source port */ - COL_UNRES_SRC_PORT, /**< 57) Unresolved source port */ - COL_TEI, /**< 58) Q.921 TEI */ - COL_UTC_YMD_TIME, /**< 59) UTC date, as YYYY-MM-DD, and time */ - COL_UTC_YDOY_TIME, /**< 60) UTC date, as YYYY/DOY, and time */ - COL_UTC_TIME, /**< 61) UTC time */ - COL_CLS_TIME, /**< 62) Command line-specified time (default relative) */ - NUM_COL_FMTS /**< 63) Should always be last */ + COL_VSAN, /**< 4) VSAN - Cisco MDS-specific */ + COL_CUMULATIVE_BYTES, /**< 5) Cumulative number of bytes */ + COL_CUSTOM, /**< 6) Custom column (any filter name's contents) */ + COL_DCE_CALL, /**< 7) DCE/RPC connection oriented call id OR datagram sequence number */ + COL_DELTA_TIME, /**< 8) Delta time */ + COL_DELTA_TIME_DIS, /**< 9) Delta time displayed*/ + COL_RES_DST, /**< 10) Resolved dest */ + COL_UNRES_DST, /**< 11) Unresolved dest */ + COL_RES_DST_PORT, /**< 12) Resolved dest port */ + COL_UNRES_DST_PORT, /**< 13) Unresolved dest port */ + COL_DEF_DST, /**< 14) Destination address */ + COL_DEF_DST_PORT, /**< 15) Destination port */ + COL_EXPERT, /**< 16) Expert Info */ + COL_IF_DIR, /**< 17) FW-1 monitor interface/direction */ + COL_FREQ_CHAN, /**< 18) IEEE 802.11 (and WiMax?) - Channel */ + COL_DEF_DL_DST, /**< 19) Data link layer dest address */ + COL_DEF_DL_SRC, /**< 20) Data link layer source address */ + COL_RES_DL_DST, /**< 21) Resolved DL dest */ + COL_UNRES_DL_DST, /**< 22) Unresolved DL dest */ + COL_RES_DL_SRC, /**< 23) Resolved DL source */ + COL_UNRES_DL_SRC, /**< 24) Unresolved DL source */ + COL_RSSI, /**< 25) IEEE 802.11 - received signal strength */ + COL_TX_RATE, /**< 26) IEEE 802.11 - TX rate in Mbps */ + COL_DSCP_VALUE, /**< 27) IP DSCP Value */ + COL_INFO, /**< 28) Description */ + COL_RES_NET_DST, /**< 29) Resolved net dest */ + COL_UNRES_NET_DST, /**< 30) Unresolved net dest */ + COL_RES_NET_SRC, /**< 31) Resolved net source */ + COL_UNRES_NET_SRC, /**< 32) Unresolved net source */ + COL_DEF_NET_DST, /**< 33) Network layer dest address */ + COL_DEF_NET_SRC, /**< 34) Network layer source address */ + COL_NUMBER, /**< 35) Packet list item number */ + COL_PACKET_LENGTH, /**< 36) Packet length in bytes */ + COL_PROTOCOL, /**< 37) Protocol */ + COL_REL_TIME, /**< 38) Relative time */ + COL_DEF_SRC, /**< 39) Source address */ + COL_DEF_SRC_PORT, /**< 40) Source port */ + COL_RES_SRC, /**< 41) Resolved source */ + COL_UNRES_SRC, /**< 42) Unresolved source */ + COL_RES_SRC_PORT, /**< 43) Resolved source port */ + COL_UNRES_SRC_PORT, /**< 44) Unresolved source port */ + COL_TEI, /**< 45) Q.921 TEI */ + COL_UTC_YMD_TIME, /**< 46) UTC date, as YYYY-MM-DD, and time */ + COL_UTC_YDOY_TIME, /**< 47) UTC date, as YYYY/DOY, and time */ + COL_UTC_TIME, /**< 48) UTC time */ + COL_CLS_TIME, /**< 49) Command line-specified time (default relative) */ + NUM_COL_FMTS /**< 50) Should always be last */ }; /** Allocate all the data structures for constructing column data, given @@ -356,8 +343,7 @@ WS_DLL_PUBLIC void col_append_sep_fstr(column_info *cinfo, const gint col, const /** Set the given (relative) time to a column element. * - * Used by multiple dissectors to set the time in the column - * COL_DELTA_CONV_TIME + * Used by dissectors to set the time in a column * * @param cinfo the current packet row * @param col the column to use, e.g. COL_INFO diff --git a/epan/column.c b/epan/column.c index b6ffd1fd3b..f6f97a746f 100644 --- a/epan/column.c +++ b/epan/column.c @@ -45,67 +45,71 @@ col_format_to_string(const gint fmt) { "%Yt", /* 1) COL_ABS_YMD_TIME */ "%YDOYt", /* 2) COL_ABS_YDOY_TIME */ "%At", /* 3) COL_ABS_TIME */ - "%c", /* 4) COL_CIRCUIT_ID */ - "%Xd", /* 5) COL_DSTIDX - !! DEPRECATED !!*/ - "%Xs", /* 6) COL_SRCIDX - !! DEPRECATED !!*/ - "%V", /* 7) COL_VSAN - !! DEPRECATED !!*/ - "%B", /* 8) COL_CUMULATIVE_BYTES */ - "%Cus", /* 9) COL_CUSTOM */ - "%y", /* 10) COL_DCE_CALL */ - "%z", /* 11) COL_DCE_CTX */ - "%Tt", /* 12) COL_DELTA_TIME */ - "%dct", /* 13) COL_DELTA_CONV_TIME */ - "%Gt", /* 14) COL_DELTA_TIME_DIS */ - "%rd", /* 15) COL_RES_DST */ - "%ud", /* 16) COL_UNRES_DST */ - "%rD", /* 17) COL_RES_DST_PORT */ - "%uD", /* 18) COL_UNRES_DST_PORT */ - "%d", /* 19) COL_DEF_DST */ - "%D", /* 20) COL_DEF_DST_PORT */ - "%a", /* 21) COL_EXPERT */ - "%I", /* 22) COL_IF_DIR */ - "%XO", /* 23) COL_OXID */ - "%XR", /* 24) COL_RXID */ - "%C", /* 25) !! DEPRECATED !! - COL_FR_DLCI */ - "%F", /* 26) COL_FREQ_CHAN */ - "%l", /* 27) !! DEPRECATED !! - COL_BSSGP_TLLI */ - "%P", /* 28) !! DEPRECATED !! - COL_HPUX_DEVID */ - "%H", /* 29) !! DEPRECATED !! - COL_HPUX_SUBSYS */ - "%hd", /* 30) COL_DEF_DL_DST */ - "%hs", /* 31) COL_DEF_DL_SRC */ - "%rhd", /* 32) COL_RES_DL_DST */ - "%uhd", /* 33) COL_UNRES_DL_DST */ - "%rhs", /* 34) COL_RES_DL_SRC*/ - "%uhs", /* 35) COL_UNRES_DL_SRC */ - "%e", /* 36) COL_RSSI */ - "%x", /* 37) COL_TX_RATE */ - "%f", /* 38) COL_DSCP_VALUE */ - "%i", /* 39) COL_INFO */ - "%U", /* 40) !! DEPRECATED !! - COL_COS_VALUE */ - "%rnd", /* 41) COL_RES_NET_DST */ - "%und", /* 42) COL_UNRES_NET_DST */ - "%rns", /* 43) COL_RES_NET_SRC */ - "%uns", /* 44) COL_UNRES_NET_SRC */ - "%nd", /* 45) COL_DEF_NET_DST */ - "%ns", /* 46) COL_DEF_NET_SRC */ - "%m", /* 47) COL_NUMBER */ - "%L", /* 48) COL_PACKET_LENGTH */ - "%p", /* 49) COL_PROTOCOL */ - "%Rt", /* 50) COL_REL_TIME */ - "%rct", /* 51) !! DEPRECATED !! - COL_REL_CONV_TIME */ - "%s", /* 52) COL_DEF_SRC */ - "%S", /* 53) COL_DEF_SRC_PORT */ - "%rs", /* 54) COL_RES_SRC */ - "%us", /* 55) COL_UNRES_SRC */ - "%rS", /* 56) COL_RES_SRC_PORT */ - "%uS", /* 57) COL_UNRES_SRC_PORT */ - "%E", /* 58) COL_TEI */ - "%Yut", /* 59) COL_UTC_YMD_TIME */ - "%YDOYut", /* 60) COL_UTC_YDOY_TIME */ - "%Aut", /* 61) COL_UTC_TIME */ - "%t" /* 62) COL_CLS_TIME */ + "%V", /* 4) COL_VSAN - !! DEPRECATED !!*/ + "%B", /* 5) COL_CUMULATIVE_BYTES */ + "%Cus", /* 6 COL_CUSTOM */ + "%y", /* 7) COL_DCE_CALL */ + "%Tt", /* 8) COL_DELTA_TIME */ + "%Gt", /* 9) COL_DELTA_TIME_DIS */ + "%rd", /* 10) COL_RES_DST */ + "%ud", /* 11) COL_UNRES_DST */ + "%rD", /* 12) COL_RES_DST_PORT */ + "%uD", /* 13) COL_UNRES_DST_PORT */ + "%d", /* 14) COL_DEF_DST */ + "%D", /* 15) COL_DEF_DST_PORT */ + "%a", /* 16) COL_EXPERT */ + "%I", /* 17) COL_IF_DIR */ + "%F", /* 18) COL_FREQ_CHAN */ + "%hd", /* 19) COL_DEF_DL_DST */ + "%hs", /* 20) COL_DEF_DL_SRC */ + "%rhd", /* 21) COL_RES_DL_DST */ + "%uhd", /* 22) COL_UNRES_DL_DST */ + "%rhs", /* 23) COL_RES_DL_SRC*/ + "%uhs", /* 24) COL_UNRES_DL_SRC */ + "%e", /* 25) COL_RSSI */ + "%x", /* 26) COL_TX_RATE */ + "%f", /* 27) COL_DSCP_VALUE */ + "%i", /* 28) COL_INFO */ + "%rnd", /* 29) COL_RES_NET_DST */ + "%und", /* 30) COL_UNRES_NET_DST */ + "%rns", /* 31) COL_RES_NET_SRC */ + "%uns", /* 32) COL_UNRES_NET_SRC */ + "%nd", /* 33) COL_DEF_NET_DST */ + "%ns", /* 34) COL_DEF_NET_SRC */ + "%m", /* 35) COL_NUMBER */ + "%L", /* 36) COL_PACKET_LENGTH */ + "%p", /* 37) COL_PROTOCOL */ + "%Rt", /* 38) COL_REL_TIME */ + "%s", /* 39) COL_DEF_SRC */ + "%S", /* 40) COL_DEF_SRC_PORT */ + "%rs", /* 41) COL_RES_SRC */ + "%us", /* 42) COL_UNRES_SRC */ + "%rS", /* 43) COL_RES_SRC_PORT */ + "%uS", /* 44) COL_UNRES_SRC_PORT */ + "%E", /* 45) COL_TEI */ + "%Yut", /* 46) COL_UTC_YMD_TIME */ + "%YDOYut", /* 47) COL_UTC_YDOY_TIME */ + "%Aut", /* 48) COL_UTC_TIME */ + "%t" /* 49) COL_CLS_TIME */ }; + /* The following formats have been used in deprecated columns. Noted here + * so they aren't reused + * + * "%U", COL_COS_VALUE + * "%c", COL_CIRCUIT_ID + * "%l", COL_BSSGP_TLLI + * "%H", COL_HPUX_SUBSYS + * "%P", COL_HPUX_DEVID + * "%C", COL_FR_DLCI + * "%rct", COL_REL_CONV_TIME + * "%dct", COL_DELTA_CONV_TIME + * "%XO", COL_OXID + * "%XR", COL_RXID + * "%Xd", COL_SRCIDX + * "%Xs", COL_DSTIDX + * "%z", COL_DCE_CTX + */ if (fmt < 0 || fmt >= NUM_COL_FMTS) return NULL; @@ -121,65 +125,52 @@ col_format_desc(const gint fmt) { "Absolute date, as YYYY-MM-DD, and time", /* 1) COL_ABS_YMD_TIME */ "Absolute date, as YYYY/DOY, and time", /* 2) COL_ABS_YDOY_TIME */ "Absolute time", /* 3) COL_ABS_TIME */ - "Circuit ID", /* 4) COL_CIRCUIT_ID */ - "Cisco Dst PortIdx", /* 5) COL_DSTIDX */ - "Cisco Src PortIdx", /* 6) COL_SRCIDX */ - "Cisco VSAN", /* 7) COL_VSAN */ - "Cumulative Bytes" , /* 8) COL_CUMULATIVE_BYTES */ - "Custom", /* 9) COL_CUSTOM */ - "DCE/RPC call (cn_call_id / dg_seqnum)", /* 10) COL_DCE_CALL */ - "DCE/RPC context ID (cn_ctx_id)", /* 11) COL_DCE_CTX */ - "Delta time", /* 12) COL_DELTA_TIME */ - "Delta time (conversation)", /* 13) COL_DELTA_CONV_TIME */ - "Delta time displayed", /* 14) COL_DELTA_TIME_DIS */ - "Dest addr (resolved)", /* 15) COL_RES_DST */ - "Dest addr (unresolved)", /* 16) COL_UNRES_DST */ - "Dest port (resolved)", /* 17) COL_RES_DST_PORT */ - "Dest port (unresolved)", /* 18) COL_UNRES_DST_PORT */ - "Destination address", /* 19) COL_DEF_DST */ - "Destination port", /* 20) COL_DEF_DST_PORT */ - "Expert Info Severity", /* 21) COL_EXPERT */ - "FW-1 monitor if/direction", /* 22) COL_IF_DIR */ - "Fibre Channel OXID", /* 23) COL_OXID */ - "Fibre Channel RXID", /* 24) COL_RXID */ - "Frame Relay DLCI", /* 25) !! DEPRECATED !! - COL_FR_DLCI */ - "Frequency/Channel", /* 26) COL_FREQ_CHAN */ - "GPRS BSSGP TLLI", /* 27) !! DEPRECATED !! - COL_BSSGP_TLLI */ - "HP-UX Device ID", /* 28) !! DEPRECATED !! - COL_HPUX_DEVID */ - "HP-UX Subsystem", /* 29) !! DEPRECATED !! - COL_HPUX_SUBSYS */ - "Hardware dest addr", /* 30) COL_DEF_DL_DST */ - "Hardware src addr", /* 31) COL_DEF_DL_SRC */ - "Hw dest addr (resolved)", /* 32) COL_RES_DL_DST */ - "Hw dest addr (unresolved)", /* 33) COL_UNRES_DL_DST */ - "Hw src addr (resolved)", /* 34) COL_RES_DL_SRC*/ - "Hw src addr (unresolved)", /* 35) COL_UNRES_DL_SRC */ - "IEEE 802.11 RSSI", /* 36) COL_RSSI */ - "IEEE 802.11 TX rate", /* 37) COL_TX_RATE */ - "IP DSCP Value", /* 38) COL_DSCP_VALUE */ - "Information", /* 39) COL_INFO */ - "L2 COS Value (802.1p)", /* 40) !! DEPRECATED !! - COL_COS_VALUE */ - "Net dest addr (resolved)", /* 41) COL_RES_NET_DST */ - "Net dest addr (unresolved)", /* 42) COL_UNRES_NET_DST */ - "Net src addr (resolved)", /* 43) COL_RES_NET_SRC */ - "Net src addr (unresolved)", /* 44) COL_UNRES_NET_SRC */ - "Network dest addr", /* 45) COL_DEF_NET_DST */ - "Network src addr", /* 46) COL_DEF_NET_SRC */ - "Number", /* 47) COL_NUMBER */ - "Packet length (bytes)" , /* 48) COL_PACKET_LENGTH */ - "Protocol", /* 49) COL_PROTOCOL */ - "Relative time", /* 50) COL_REL_TIME */ - "Relative time (conversation)", /* 51) !! DEPRECATED !! - COL_REL_CONV_TIME */ - "Source address", /* 52) COL_DEF_SRC */ - "Source port", /* 53) COL_DEF_SRC_PORT */ - "Src addr (resolved)", /* 54) COL_RES_SRC */ - "Src addr (unresolved)", /* 55) COL_UNRES_SRC */ - "Src port (resolved)", /* 56) COL_RES_SRC_PORT */ - "Src port (unresolved)", /* 57) COL_UNRES_SRC_PORT */ - "TEI", /* 58) COL_TEI */ - "UTC date, as YYYY-MM-DD, and time", /* 59) COL_UTC_YMD_TIME */ - "UTC date, as YYYY/DOY, and time", /* 60) COL_UTC_YDOY_TIME */ - "UTC time", /* 61) COL_UTC_TIME */ - "Time (format as specified)" /* 62) COL_CLS_TIME */ + "Cisco VSAN", /* 4) COL_VSAN */ + "Cumulative Bytes" , /* 5) COL_CUMULATIVE_BYTES */ + "Custom", /* 6) COL_CUSTOM */ + "DCE/RPC call (cn_call_id / dg_seqnum)", /* 7) COL_DCE_CALL */ + "Delta time", /* 8) COL_DELTA_TIME */ + "Delta time displayed", /* 9) COL_DELTA_TIME_DIS */ + "Dest addr (resolved)", /* 10) COL_RES_DST */ + "Dest addr (unresolved)", /* 11) COL_UNRES_DST */ + "Dest port (resolved)", /* 12) COL_RES_DST_PORT */ + "Dest port (unresolved)", /* 13) COL_UNRES_DST_PORT */ + "Destination address", /* 14) COL_DEF_DST */ + "Destination port", /* 15) COL_DEF_DST_PORT */ + "Expert Info Severity", /* 16) COL_EXPERT */ + "FW-1 monitor if/direction", /* 17) COL_IF_DIR */ + "Frequency/Channel", /* 18) COL_FREQ_CHAN */ + "Hardware dest addr", /* 19) COL_DEF_DL_DST */ + "Hardware src addr", /* 20) COL_DEF_DL_SRC */ + "Hw dest addr (resolved)", /* 21) COL_RES_DL_DST */ + "Hw dest addr (unresolved)", /* 22) COL_UNRES_DL_DST */ + "Hw src addr (resolved)", /* 23) COL_RES_DL_SRC*/ + "Hw src addr (unresolved)", /* 24) COL_UNRES_DL_SRC */ + "IEEE 802.11 RSSI", /* 25) COL_RSSI */ + "IEEE 802.11 TX rate", /* 26) COL_TX_RATE */ + "IP DSCP Value", /* 27) COL_DSCP_VALUE */ + "Information", /* 28) COL_INFO */ + "Net dest addr (resolved)", /* 29) COL_RES_NET_DST */ + "Net dest addr (unresolved)", /* 30) COL_UNRES_NET_DST */ + "Net src addr (resolved)", /* 31) COL_RES_NET_SRC */ + "Net src addr (unresolved)", /* 32) COL_UNRES_NET_SRC */ + "Network dest addr", /* 33) COL_DEF_NET_DST */ + "Network src addr", /* 34) COL_DEF_NET_SRC */ + "Number", /* 35) COL_NUMBER */ + "Packet length (bytes)" , /* 36) COL_PACKET_LENGTH */ + "Protocol", /* 37) COL_PROTOCOL */ + "Relative time", /* 38) COL_REL_TIME */ + "Source address", /* 39) COL_DEF_SRC */ + "Source port", /* 40) COL_DEF_SRC_PORT */ + "Src addr (resolved)", /* 41) COL_RES_SRC */ + "Src addr (unresolved)", /* 42) COL_UNRES_SRC */ + "Src port (resolved)", /* 43) COL_RES_SRC_PORT */ + "Src port (unresolved)", /* 44) COL_UNRES_SRC_PORT */ + "TEI", /* 45) COL_TEI */ + "UTC date, as YYYY-MM-DD, and time", /* 46) COL_UTC_YMD_TIME */ + "UTC date, as YYYY/DOY, and time", /* 47) COL_UTC_YDOY_TIME */ + "UTC time", /* 48) COL_UTC_TIME */ + "Time (format as specified)" /* 49) COL_CLS_TIME */ }; g_assert((fmt >= 0) && (fmt < NUM_COL_FMTS)); @@ -473,10 +464,6 @@ get_column_longest_string(const gint format) case COL_DELTA_TIME_DIS: return get_timestamp_column_longest_string(TS_DELTA_DIS, timestamp_get_precision()); break; - case COL_REL_CONV_TIME: /* 'abuse' TS_RELATIVE to set the time format */ - case COL_DELTA_CONV_TIME: /* for the conversation related time columns */ - return get_timestamp_column_longest_string(TS_RELATIVE, timestamp_get_precision()); - break; case COL_DEF_SRC: case COL_RES_SRC: case COL_UNRES_SRC: @@ -514,20 +501,9 @@ get_column_longest_string(const gint format) case COL_CUMULATIVE_BYTES: return "00000000"; break; - case COL_RXID: - case COL_OXID: - return "000000"; - break; case COL_IF_DIR: return "i 00000000 I"; break; - case COL_CIRCUIT_ID: - return "000000"; - break; - case COL_SRCIDX: - case COL_DSTIDX: - return "00000000"; - break; case COL_VSAN: return "000000"; break; @@ -537,36 +513,18 @@ get_column_longest_string(const gint format) case COL_RSSI: return "100"; break; - case COL_HPUX_SUBSYS: - return "OTS9000-TRANSPORT"; - break; - case COL_HPUX_DEVID: - return "0000"; - break; case COL_DCE_CALL: return "0000"; break; - case COL_DCE_CTX: - return "0000"; - break; case COL_8021Q_VLAN_ID: return "0000"; break; case COL_DSCP_VALUE: return "00"; break; - case COL_COS_VALUE: - return "0"; - break; case COL_TEI: return "127"; break; - case COL_FR_DLCI: - return "8388608"; - break; - case COL_BSSGP_TLLI: - return "0xffffffff"; - break; case COL_EXPERT: return "ERROR"; break; diff --git a/epan/prefs.c b/epan/prefs.c index 8e711ba340..2fb1b0d567 100644 --- a/epan/prefs.c +++ b/epan/prefs.c @@ -2840,6 +2840,45 @@ find_val_for_string(const char *needle, const enum_val_t *haystack, return default_value; } + +/* Array of columns that have been migrated to custom columns */ +struct deprecated_columns { + const gchar *col_fmt; + const gchar *col_expr; +}; +static struct deprecated_columns migrated_columns[] = { + { /* COL_COS_VALUE */ "%U", "vlan.priority" }, + { /* COL_CIRCUIT_ID */ "%c", "iax2.call" }, + { /* COL_BSSGP_TLLI */ "%l", "bssgp.tlli" }, + { /* COL_HPUX_SUBSYS */ "%H", "nettl.subsys" }, + { /* COL_HPUX_DEVID */ "%P", "nettl.devid" }, + { /* COL_FR_DLCI */ "%C", "fr.dlci" }, + { /* COL_REL_CONV_TIME */ "%rct", "tcp.time_relative" }, + { /* COL_DELTA_CONV_TIME */ "%dct", "tcp.time_delta" }, + { /* COL_OXID */ "%XO", "fc.ox_id" }, + { /* COL_RXID */ "%XR", "fc.rx_id" }, + { /* COL_SRCIDX */ "%Xd", "mdshdr.srcidx" }, + { /* COL_DSTIDX */ "%Xs", "mdshdr.dstidx" }, + { /* COL_DCE_CTX */ "%z", "dcerpc.cn_ctx_id" } +}; + +static gboolean +is_deprecated_column_format(const gchar* fmt) +{ + guint haystack_idx; + + for (haystack_idx = 0; + haystack_idx < G_N_ELEMENTS(migrated_columns); + ++haystack_idx) { + + if (strcmp(migrated_columns[haystack_idx].col_fmt, fmt) == 0) { + return TRUE; + } + } + + return FALSE; +} + /* Preferences file format: * - Configuration directives start at the beginning of the line, and * are terminated with a colon. @@ -2896,7 +2935,7 @@ parse_column_format(fmt_data *cfmt, const char *fmt) g_strfreev(cust_format_info); } else { col_fmt = get_column_format_from_str(fmt); - if (col_fmt == -1) + if ((col_fmt == -1) && (!is_deprecated_column_format(fmt))) return FALSE; } @@ -3852,32 +3891,10 @@ string_to_name_resolve(const char *string, e_addr_resolve *name_resolve) return '\0'; } - static void try_convert_to_custom_column(gpointer *el_data) { - /* Array of columns that have been migrated to custom columns */ - struct { - gint el; - const gchar *col_expr; - } migrated_columns[] = { - { COL_COS_VALUE, "vlan.priority" }, - { COL_CIRCUIT_ID, "iax2.call" }, - { COL_BSSGP_TLLI, "bssgp.tlli" }, - { COL_HPUX_SUBSYS, "nettl.subsys" }, - { COL_HPUX_DEVID, "nettl.devid" }, - { COL_FR_DLCI, "fr.dlci" }, - { COL_REL_CONV_TIME, "tcp.time_relative" }, - { COL_DELTA_CONV_TIME, "tcp.time_delta" }, - { COL_OXID, "fc.ox_id" }, - { COL_RXID, "fc.rx_id" }, - { COL_SRCIDX, "mdshdr.srcidx" }, - { COL_DSTIDX, "mdshdr.dstidx" }, - { COL_DCE_CTX, "dcerpc.cn_ctx_id" } - }; - guint haystack_idx; - const gchar *haystack_fmt; gchar **fmt = (gchar **) el_data; @@ -3885,8 +3902,7 @@ try_convert_to_custom_column(gpointer *el_data) haystack_idx < G_N_ELEMENTS(migrated_columns); ++haystack_idx) { - haystack_fmt = col_format_to_string(migrated_columns[haystack_idx].el); - if (strcmp(haystack_fmt, *fmt) == 0) { + if (strcmp(migrated_columns[haystack_idx].col_fmt, *fmt) == 0) { gchar *cust_col = g_strdup_printf("%%Cus:%s:0", migrated_columns[haystack_idx].col_expr);