[Roland Knall <rknall@xxxxxxxxx>]

That is an early design flaw - still going strong to that very day. It stems from the fact, that the version query was introduced later on, and the interface list query basically covered that as well. Additionally, parsed extcap interfaces where not parsed on a tool (meaning executable) level, but rather on a per-interface level. That resulted in the fact, that there was no storage in place for saving the version after parsing, except in extcap_interface.

extcap_tools is rather new to this concert (merged last week) and so the version moved there. But I am in the process of restructuring extcap_interface, and in that process I clean up a bit more. In the meantime, when in doubt, extcap_info->version is the one read by the overall system.

regards

Roland

On Mon, Feb 27, 2017 at 11:19 AM, Dario Lombardo <dario.lombardo.ml@xxxxxxxxx> wrote:

Hi

In extcap structs I've found 2 entries for "version". In

typedef struct _extcap_interface {

    gchar * call;

    gchar * display;

    gchar * version;

    gchar * help;

    gchar * extcap_path;

    extcap_sentence_type if_type;

} extcap_interface;

and

typedef struct _extcap_info {

    gchar * basename;

    gchar * full_path;

    gchar * version;

    GList * interfaces;

} extcap_info;

Is that intentional? While I get the reason for version in extcap info, I don't get it for interface. Moreover, extcap tools show global version but not version in interface

extcap {version=1.0.0}{help=file:///usr/local/share/wireshark/ciscodump.html}

interface {value=cisco}{display=Cisco remote capture}

extcap {version=0.1.0}{help=file:///usr/local/share/wireshark/randpktdump.html}

interface {value=randpkt}{display=Random packet generator}

extcap {version=1.0.0}{help=file:///usr/local/share/wireshark/sshdump.html}

interface {value=ssh}{display=SSH remote capture}

extcap {version=0.1.0}{help=file:///usr/local/share/wireshark/udpdump.html}

interface {value=udpdump}{display=UDP Listener remote capture}

What is it for?

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@wireshark.org?subject=unsubscribe