Wireshark mailing list archives

Re: Replace TRUE/FALSE with proper ENC_* in proto_tree_add_item() using a script.


From: Bill Meier <wmeier () newsguy com>
Date: Mon, 10 Oct 2011 14:41:59 -0400

On 10/4/2011 2:45 PM, Guy Harris wrote:
Presumably all the uses of
proto_tree_add_item() and the like for FT_ABSOLUTE_TIME values
already have the encoding specified and already use
ENC_LITTLE_ENDIAN/ENC_BIG_ENDIAN.


Not quite all :)    See below.


=========================================================================

--> packet-gmhdr.c

   FT_ABSOLUTE_TIME:
proto_tree_add_item(gmhdr_tree, hf_gmhdr_timestamp, tvb, offset, fl, [[FALSE]-->[???]]);


-->packet-netflow.c

   FT_ABSOLUTE_TIME:
 proto_tree_add_item(pdutree, hf_pie_ntop_rtp_first_ts,
                 tvb, offset, length, [[FALSE]-->[???]]);

   FT_ABSOLUTE_TIME:
 proto_tree_add_item(pdutree, hf_pie_ntop_rtp_last_ts,
                         tvb, offset, length, [[FALSE]-->[???]]);

   FT_ABSOLUTE_TIME:
 proto_tree_add_item(pdutree, hf_pie_plixer_origination_time,
                         tvb, offset, length, [[FALSE]-->[???]]);

   FT_ABSOLUTE_TIME:
 proto_tree_add_item(pdutree, hf_pie_plixer_date_time,
                         tvb, offset, length, [[FALSE]-->[???]]);

-->packet-reload.c

   FT_ABSOLUTE_TIME:
proto_tree_add_item(local_tree, hf_reload_diagnostic_expiration, tvb, offset, 8, [[ENC_TIME_NTP]-->[???]]);

   FT_ABSOLUTE_TIME:
proto_tree_add_item(local_tree, hf_reload_diagnostic_expiration, tvb, offset, 8, [[ENC_TIME_NTP]-->[???]]);

   FT_ABSOLUTE_TIME:
proto_tree_add_item(local_tree, hf_reload_diagnosticrequest_timestampinitiated, tvb, offset+local_offset, 8, [[ENC_TIME_NTP]-->[???]]);

   FT_ABSOLUTE_TIME:
proto_tree_add_item(local_tree, hf_reload_diagnosticresponse_timestampreceived, tvb, offset+local_offset, 8, [[ENC_TIME_NTP]-->[???]]);
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request () wireshark org?subject=unsubscribe


Current thread: