Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 3096] Ability to annotate packet captures

Date: Mon, 20 Feb 2012 12:44:30 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3096

Jose Pedro Oliveira <jpo@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #7861|                            |review_for_checkin?
              Flags|                            |

--- Comment #58 from Jose Pedro Oliveira <jpo@xxxxxxxxxxxx> 2012-02-20 12:44:29 PST ---
Created attachment 7861
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=7861
version_info: new function get_os_version_info() and shb_os option string
length

Summary:
The attached patch adds a new function - get_os_version_info() - to the
version_info module and uses it in dumpcap to fill the shb_os option value.

Current status:
Rev 41026 added support for the option shb_os in dumpcap and started filling
its value using get_runtime_version_info(). The problem is that the
runtime_version_info string is usually longer than 100 bytes and it exposed a
"nasty" limitation in previous versions of the Wiretap API that rendered the
newly created pcapng files unreadable with older versions of Wireshark (see bug
#6846). A patch for the problem has already been committed in branch-1.6 and
queued for branch-1.4 (revision 41037).

The function get_os_version_info() would allow the shb_os string length to stay
below the 100 bytes threshold (thus minimizing the exposure to bug #6846) and
would also make the its value a little more compliant to its description in the
spec. Note: The code of this function was previously in
get_runtime_version_info().

I think the runtime version information should still be added as an option,
maybe in a opt_comment, or in a new one (would require the registration oa a
new option for that purpose).

Comments and/or suggestions are welcome?
jpo



shb_os value from get_runtime_version_info()
----------
I: Section 1 options retrieved
  O: Option code  3   = 'on Linux 3.2.5-3.fc16.x86_64, with locale en_US.UTF-8,
with libpcap version
1.2.1, with libz 1.2.5.

Built using gcc 4.6.2 20111027 (Red Hat 4.6.2-1).
'
  O: Option code  4   = 'Dumpcap 1.7.1-SVN-41080 (SVN Rev Unknown from
unknown)'
----------


shb_os value from get_os_version_info()
----------
I: Section 1 options retrieved
  O: Option code  3   = 'Linux 3.2.5-3.fc16.x86_64'
  O: Option code  4   = 'Dumpcap 1.7.1-SVN-41080 (SVN Rev Unknown from
unknown)'
----------

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.