[Wireshark-dev] wiki editor permissions

2016-09-14 Thread Dirk Jagdmann
Hello developers, I'd like to write a wiki page about using the XSLT program to convert a PDML file into a HTML file. I'm thus requesting wiki editor permissions. My wiki username is "doj". -- ---> Dirk Jagdmann > http://cubic.org/~doj -

Re: [Wireshark-dev] Rename TVB captured length vs reported length

2014-02-18 Thread Dirk Jagdmann
velopers will interpret it differently. Use the longer, but explicit function names. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archi

Re: [Wireshark-dev] Rename TVB captured length vs reported length

2014-02-18 Thread Dirk Jagdmann
the frame and the captured length. I suggest that we simply state these lengths in the function name and *not* have an unqualified (shorter) name which will get misinterpreted. So my suggestion for a rename would be: tvb_actual_length() tvb_captured_length() -- ---> Dirk Jagdmann > http://cub

Re: [Wireshark-dev] Decrypting SSL in dissector

2014-01-09 Thread Dirk Jagdmann
do you have a new_register_dissector("amp", ...) in the proto_register_amp() function? Otherwise the SSL dissector can not match the "amp" string to a dissector handle/function. -- ---> Dirk Jagdmann > http://cubic.org/~doj -

Re: [Wireshark-dev] OSX 10.9 build procedures for latest source

2013-12-17 Thread Dirk Jagdmann
sed OsX 10.9 yet, so I don't know if it requires any extra steps. You shouldn't use macports to install dependencies. What you need is Xcode + X11 packages, then use the macosx-setup.sh to install any dependencies into /usr/local/. -- ---> Dirk Jagdmann > http://cubic.org/~doj

Re: [Wireshark-dev] Val_to_str as a macro

2013-12-11 Thread Dirk Jagdmann
the original function. I can use a single temp global, but that's just ugly and won't work if we ever do multithreading. It may be ugly, but using thread local storage for that global variable will make it safe for multi-threading. -- ---> Dirk Jagdmann > http:

Re: [Wireshark-dev] Gerrit sandbox available for testing

2013-09-18 Thread Dirk Jagdmann
sting this system? -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wir

Re: [Wireshark-dev] Transport name resolution

2013-09-16 Thread Dirk Jagdmann
name just differ in capitalization and my brain typically wouldn't notice any difference, so it may not make any big difference. Side note: it may be worthwhile to add a recommendation to the check-api program that short name and filter name should case insensitive compare equal. -- -

Re: [Wireshark-dev] Transport name resolution

2013-09-16 Thread Dirk Jagdmann
ce the dissector short name is typically used as the filter prefix. It is just confusing if slightly different strings are shown, because they come from some other list/database. -- ---> Dirk Jagdmann > http://cubic.

Re: [Wireshark-dev] [Wireshark-commits] rev 51169: / /trunk/epan/: app_mem_usage.c

2013-08-06 Thread Dirk Jagdmann
old_offset = lseek(fd, 0, SEEK_CUR); // missing error handling lseek(fd, offset, SEEK_SET); // missing error handling const ssize_t ret = read(fd, buf, count); lseek(fd, old_offset, SEEK_SET); // missing error handling return ret; } -- ---> Dirk Jagdmann > http://cubic.or

Re: [Wireshark-dev] The field called Command Sequence Number in the SMB2 dissector is actually the Message ID

2013-08-01 Thread Dirk Jagdmann
> I took a half-educated whack at fixing those in r51080. Thanks for pointing > it > out (and feel free to point out any mistakes I may have made). It all looks reasonable, thenk you for making those changes. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> ht

Re: [Wireshark-dev] The field called Command Sequence Number in the SMB2 dissector is actually the Message ID

2013-07-31 Thread Dirk Jagdmann
while you are renaming the displayed name of this variable, the rest of the SMB2 dissector is using the term "sequence number" in many places. It's now confusing if source code comments talk about sequence numbers and use them in hash tables, while the dissection shows it as Message ID. To really w

Re: [Wireshark-dev] Code sharing between TLS and DTLS

2013-07-04 Thread Dirk Jagdmann
s to examine SSL/TLS or DTLS (maybe studying something generic about these kinds of encryption protocols) the user would like to have a filter prefix that will match both use cases. -- ---> Dirk Jagdmann ^ doj / cubic >

Re: [Wireshark-dev] Code sharing between TLS and DTLS

2013-07-04 Thread Dirk Jagdmann
packets may not be matched. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: h

Re: [Wireshark-dev] Fileshark (AKA Dissecting Files with Wireshark)

2013-06-21 Thread Dirk Jagdmann
packets. Also there are definately more file formats than network protocols. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://

[Wireshark-dev] using C++, was: Notes from Sharkfest '13

2013-06-21 Thread Dirk Jagdmann
re of the packet/file/application lifetime might work. We should research this, write guidelines how to use C++ objects in Wireshark and then make a decision if we want to allow C++ features everywhere. Another advantage would be that we can use real C++ exceptions. -- ---> Dirk Jagdmann > http:/

Re: [Wireshark-dev] "Type-punned pointer... breaks anti-aliasing rules" in dfilter-macro.c

2013-06-18 Thread Dirk Jagdmann
oid*)macros)" helps? -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscri

Re: [Wireshark-dev] [Wireshark-commits] rev 49995: /trunk/ /trunk/: macosx-setup.sh

2013-06-18 Thread Dirk Jagdmann
d just be too much effort, so I quit. I suggest it is not worth investing time to support those non-working old versions, as nobody seems to have complained yet (and I don't think anybody will in the future). -- ---> Dirk Jagdmann > http://cubic.

Re: [Wireshark-dev] [Wireshark-commits] rev 49995: /trunk/ /trunk/: macosx-setup.sh

2013-06-18 Thread Dirk Jagdmann
I've created https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8821 with my suggestion. We can continue to discuss there if/what/how we'd like to determine the optimum number of parallel make jobs. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> h

[Wireshark-dev] how does cmake know how to generate plugin.c

2013-06-18 Thread Dirk Jagdmann
I'm studying cmake rules in the plugin/ directory. I can not find out, how cmake knows how to build the plugin.c file for each of the plugins. Where is this defined? -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http:/

Re: [Wireshark-dev] [Wireshark-commits] rev 49995: /trunk/ /trunk/: macosx-setup.sh

2013-06-18 Thread Dirk Jagdmann
(personal note, my old MacBook Pro has two cores and no hyper-thread, so personally I'm fine with 3 make jobs [note that this was the default in the OsX build script for a long time]) -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org _

Re: [Wireshark-dev] [Wireshark-commits] rev 49995: /trunk/ /trunk/: macosx-setup.sh

2013-06-18 Thread Dirk Jagdmann
_OPT" ] ; then MAKE_BUILD_OPT="-j 3" fi This way the user can set custom make options, but the script has a default. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:

[Wireshark-dev] cast errors reported by a OsX cmake build

2013-06-17 Thread Dirk Jagdmann
m 'char *' to 'struct state *' increases required alignment from 1 to 8 [-Werror,-Wcast-align] newaction->x.stp = (struct state *)arg; and many more of this type. Is somebody looking into cmake builds on OsX? -- ---> Dirk Jagdmann --

Re: [Wireshark-dev] 1.10 branch + release schedule

2013-04-22 Thread Dirk Jagdmann
older Wireshark releases. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscrib

Re: [Wireshark-dev] 1.10 branch + release schedule

2013-04-18 Thread Dirk Jagdmann
Note that we would still be committed to supporting OS X PPC and U3 users until Wireshark 1.10 reaches end of life in 2015. I suggest remove OsX PPC and U3 even for the 1.10 release. Your changes for GTK/QT sound reasonable. -- ---> Dirk Jagdmann > http://cubic.org/~doj -

Re: [Wireshark-dev] Copying TVBs for Reassembly [Was: Filebacked-tvbuffs : GSoC'13]

2013-04-18 Thread Dirk Jagdmann
I dont think composite tvbs actually work. or at least they didnt work when we originally wrote the reassembly code. They have been fixed last year. They are working for me in 1.8.x code. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.

Re: [Wireshark-dev] Icon Poll - is this OK?

2013-04-11 Thread Dirk Jagdmann
le button, which probably has two advantages: - no confusion - one less button in the bar -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailin

Re: [Wireshark-dev] read/write capture comments from the command line

2013-04-08 Thread Dirk Jagdmann
g the glib functions. Otherwise we would just add more code to Wireshark and related tools for no real benefit. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wiresha

Re: [Wireshark-dev] [Wireshark-commits] buildbot failure in Wireshark (development) on Windows-XP-x86

2013-03-17 Thread Dirk Jagdmann
indows cmd.exe, run the vcvars32.bat script, then add the path to your cygwin bin directory to your PATH. That should result in a shell which can run the Microsoft tools and the cygwin tools. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___

Re: [Wireshark-dev] Simpifying exporting DLL symbols

2013-02-27 Thread Dirk Jagdmann
if it works for all platform we care about. However we should also add a section to the developer README to explain the new macros. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org __

Re: [Wireshark-dev] checkett.pl anyone?

2013-02-21 Thread Dirk Jagdmann
d ett value, that's probably a better approach. It is not a static code analysis, but if our users report on issues, it's easy to fix them. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org #!/usr/bin/env perl use strict; die "usage: checkett.

[Wireshark-dev] Google Summer of Code 2013 project

2013-02-17 Thread Dirk Jagdmann
thernet src/dst address + payload length - specific protocol dissectors can create their own hashes, for example DCE/RPC dissector can use protocol type (UUID) + Call ID; HTTP dissector could use header key/value pairs, etc. Via a context menu on the packet I can select which of these hashes to

Re: [Wireshark-dev] Idle Thought - Compiling with C++

2013-02-11 Thread Dirk Jagdmann
me mangling etc). -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https

Re: [Wireshark-dev] Get_cpu_info on platforms other than Windows?

2013-02-06 Thread Dirk Jagdmann
-- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailma

Re: [Wireshark-dev] Reducing Autotools verbosity

2013-02-04 Thread Dirk Jagdmann
maybe mention in the developer README how to bring back verbose build output. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives

Re: [Wireshark-dev] [Wireshark-commits] rev 47188: /trunk/ui/gtk/ /trunk/ui/gtk/: main.c

2013-01-21 Thread Dirk Jagdmann
(prefs.capture_device != NULL) && (prefs.capture_device[0] != '\0')) { -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list

Re: [Wireshark-dev] RFD: The Future of Memory Management in Wireshark

2012-10-27 Thread Dirk Jagdmann
cator); // or like this? -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscrib

Re: [Wireshark-dev] Asterisk AMI and FreeSWITCH ESL dissectors

2012-10-27 Thread Dirk Jagdmann
TTP, but it should give some results. If that doesn't work well, you'll have to look into your own custom dissector. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Se

Re: [Wireshark-dev] DND crash through all versions?

2012-09-23 Thread Dirk Jagdmann
g (for me) from the Mac finder to Wireshark. I don't know if DnD is supported with GTK2/X11 on Mac, but it may not be. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:

[Wireshark-dev] Wireshark Final Cleanup Function/Hook?

2012-08-28 Thread Dirk Jagdmann
Hello Wireshark developers, is there a way to register a cleanup function in a dissector to be called when a file closed? I have a couple of hash tables allocated using g_hash_table_new() that I want to free when the user closes the file. -- ---> Dirk Jagdmann > http://cubic.or

Re: [Wireshark-dev] Is Python Still Considered Optional for the Build Process? What Should the Minimum Version Be?

2012-07-31 Thread Dirk Jagdmann
make python a mandatory requirement for building. It's easy to install on every major platform we are supporting and not having to maintain the same program written is a step in the right direction. -- ---> Dirk Jagdmann > http://cubic.org/~doj --

[Wireshark-dev] remote control Wireshark

2012-07-05 Thread Dirk Jagdmann
ows, OsX and the Linux/BSD world. Did someone start such a project? Would we prefer a certain existing protocol/library? I could also think about simply using a multicast socket and send messages across. This would without further effort allow Wireshark control from different hosts. -- -

Re: [Wireshark-dev] [Wireshark-commits] rev 40352: /trunk/ /trunk/macosx-support-lib-patches/: glib-pkgconfig.patch /trunk/: macosx-setup.sh

2012-01-01 Thread Dirk Jagdmann
though it can be downloaded and build fast, I prefer to only install the minimum of support programs required to build a GTK on Mac. Since OsX tar can handle bzip2, I prefer to use that and not install the xz program. -- ---> Dirk Jagdmann > http://cubic.or

[Wireshark-dev] misleading description on Wireshark Download page

2011-11-02 Thread Dirk Jagdmann
can remove that old release candidate, or alternatively put a daily/weekly tarball of trunk on that page. ---> Dirk Jagdmann ^ doj / cubic > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wire

Re: [Wireshark-dev] Replace TRUE/FALSE with proper ENC_* in proto_tree_add_item() using a script.

2011-10-13 Thread Dirk Jagdmann
n fix them manually later. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: ht

Re: [Wireshark-dev] Replace TRUE/FALSE with proper ENC_* in proto_tree_add_item() using a script.

2011-10-04 Thread Dirk Jagdmann
ASCII *and* ENC_LITTLE_ENDIAN|ENC_BIG_ENDIAN ? Shouldn't this be ENC_ASCII | ENC_NA in this case ? Probably. Don't expect every developer of the various dissectors to have specified 100% correct parameters here, since a lot of combinations so far did just work. -- ---> Dirk Jagdmann

Re: [Wireshark-dev] [Wireshark-commits] rev 39143: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dvbci.c

2011-09-26 Thread Dirk Jagdmann
obably a lot of work, I vote for this approach. For the 1.8 release we should emit warnings if FT_ and ENC_ don't match, then for 1.10 emit error/exceptions, so that eventually all FT_ and ENC_ values express the real relationshi

Re: [Wireshark-dev] Remove support for libpcre?

2011-08-23 Thread Dirk Jagdmann
> Now that we require GLib 2.14 (which includes GRegex) we don't need libpcre. > Should we remove the support for libpcre completely? Away with it, less dependencies are always preferred. -- ---> Dirk Jagdmann > http://cubic.org/~doj ->

Re: [Wireshark-dev] Time shift patch causes compile error

2011-08-15 Thread Dirk Jagdmann
long double" format. And 64bit of double should be enough for a nanosecond precision. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:

Re: [Wireshark-dev] [Bug 6086] How does it work?

2011-08-02 Thread Dirk Jagdmann
ll frames captures today, you'll write a filter as "frame.time>='2011-08-02 00:00:00' && frame.time<='2011-08-02 23:59:59'" -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org

Re: [Wireshark-dev] How to build on MACOS (with revised instructions)

2011-07-16 Thread Dirk Jagdmann
lso document that these scripts exists and move them to a common directory (probably below the tools/ directory), so people can find them easily. At least for your mac script you should review the README.macos file, since it probably contains outdated content as well. ---> Dirk Jagdmann ^ doj /

Re: [Wireshark-dev] How to build on MACOS (with revised instructions)

2011-07-16 Thread Dirk Jagdmann
ntains a similar script to install gtk and dependencies, but is no longer working. ---> Dirk Jagdmann ^ doj / cubic > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev maili

Re: [Wireshark-dev] Presenting WireViz

2011-07-16 Thread Dirk Jagdmann
compilers and not some ad-hoc string match/replace. ---> Dirk Jagdmann ^ doj / cubic > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list Archives:http://www.wiresha

Re: [Wireshark-dev] Increase minimum GLib and GTK+ versions?

2011-07-06 Thread Dirk Jagdmann
Gerald Combs wrote: > Is there any reason we shouldn't make GLib 2.14.0 and GTK+ 2.12.0 the > minimum required versions to build Wireshark? This would let us get rid I don't see any reason. -- ---> Dirk Jagdmann > http://cubic.org/~doj -

Re: [Wireshark-dev] About Dead Store in clang Analysis

2011-03-24 Thread Dirk Jagdmann
that includes, as > already pointed out, leaving things in a clean state for when the next > field is added to the end of that tag.) If only I could have expressed this so good... -- ---> Dirk Jagdmann > http://cubic.org/~do

Re: [Wireshark-dev] About Dead Store in clang Analysis

2011-03-24 Thread Dirk Jagdmann
ting the offset is well proven throughout the code. Having that additional integer increment will not result in wasted CPU cycles. And if you have an incomplete dissector if the correct offset increment is used, you will not introduce a bug when adding additional proto_items. -- ---> Dirk J

Re: [Wireshark-dev] malformed packet for no reason

2011-03-21 Thread Dirk Jagdmann
, data_size); But tvb_get_ptr() can return NULL when out of bounds of the current packet. I guess this condition is reached on your 6th iteration. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___

Re: [Wireshark-dev] bitmask handling in wireshark

2010-12-29 Thread Dirk Jagdmann
> #define MYPROTO_FLAG_SAMPLING 0xf0 A better name would be #define MYPROTO_MASK_SAMPLING 0xf0 since it is a bitmask you're defining here. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> htt

Re: [Wireshark-dev] [Wireshark-commits] rev 35213: /trunk/docbook/wsdg_src/ /trunk/docbook/wsdg_src/: WSDG_chapter_build_intro.xml WSDG_preface.xml

2010-12-18 Thread Dirk Jagdmann
changes. I think having one concise document which shows the way how new development should be made is a good thing. The WSDG from the website looks like generated from docbook. So we could generate a plain text file from that and put it a README.developer into the sources. ---> Dirk Ja

Re: [Wireshark-dev] How to avoid dissection based on port defined by a different dissector?

2010-12-15 Thread Dirk Jagdmann
g systems have different default ranges and you can typically override those defaults. See http://en.wikipedia.org/wiki/Ephemeral_port -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org

Re: [Wireshark-dev] How to avoid dissection based on port defined by a different dissector?

2010-12-15 Thread Dirk Jagdmann
s which are not heuristic, but if a port clash occurs the developer who wants to submit a new dissector should convert the existing one to heuristic and make his new one heuristic as well. Or simply live with the fact, that some other dissector might "win the lottery" which one gets

Re: [Wireshark-dev] Enhancement-Request: RTP-MIDI dissector

2010-12-10 Thread Dirk Jagdmann
ot;non serial cable distribution". I guess your approach is fine then. Does it work with your earlier applemidi dissector? -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent

Re: [Wireshark-dev] Enhancement-Request: RTP-MIDI dissector

2010-12-09 Thread Dirk Jagdmann
that we don't need to have a new MIDI dissection implementation for other protocols. -- ---> Dirk Jagdmann > http://cubic.org/~doj -> http://llg.cubic.org ___ Sent via:Wireshark-dev mailing list A

Re: [Wireshark-dev] Problems creating the OsX App

2010-11-19 Thread Dirk Jagdmann
shark.app/Contents/Resources/bin/ results in: > > This is fixed in trunk in revision 34583, I'll add it to the roadmap > for released versions. > Try the attached patch. that patch is working for me. ---> Dirk Jagdmann ^ doj

[Wireshark-dev] Problems creating the OsX App

2010-11-18 Thread Dirk Jagdmann
et up a OsX compilation environment lately and maybe has hints how to resolve all dependencies for an Application (and eventually Package) creation? ---> Dirk Jagdmann ^ doj / cubic > http://cubic.org/~doj -> http://llg.cubic.org __