Commit Graph

47566 Commits

Author SHA1 Message Date
Anders Broman e9872cf077 Update dissection.
svn path=/trunk/; revision=50604
2013-07-15 04:50:42 +00:00
Anders Broman 17c0371414 From Javier Godoy:
fix partitions being inadvertently set as removed.

In dissect_osd_get_attributes and dissect_osd_set_attributes there were calls to dissect_osd_partition_id with the last parameter (is_removed) set to TRUE instead of FALSE. That caused dissect_osd_partition_id reporting the partitions as "removed". 

The only service action where is_removed must be TRUE is dissect_osd_remove_partition.

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8927

svn path=/trunk/; revision=50603
2013-07-15 04:50:03 +00:00
Jeff Morriss 58b3567ccb From Cal Turney via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8920 :
Changed FATTR4_DISSECT_VALUES arg to FATTR4_BITMAP_ONLY in calls to 
dissect_nfs4_fattrs() in cases NFS4_OP_OPEN and NFS4_OP_CREATE of
dissect_nfs4_response_op().

svn path=/trunk/; revision=50602
2013-07-15 04:00:20 +00:00
Jeff Morriss 3c76662814 Try 2 at fixing the Debian package.
svn path=/trunk/; revision=50601
2013-07-15 03:19:32 +00:00
Jeff Morriss a8ad045a77 Add report_err.c to cmake build.
svn path=/trunk/; revision=50600
2013-07-15 03:12:02 +00:00
Jeff Morriss 16ef3ce2f1 Restore inclusion of report_err.h
svn path=/trunk/; revision=50599
2013-07-15 03:06:00 +00:00
Jeff Morriss 54bb2e7a5c Move report_err.{h,c} from epan into wsutil: there's nothing epan-specific there and moving it avoids having to recompile the file for use in editcap and capinfos (which don't link against libwireshark).
svn path=/trunk/; revision=50598
2013-07-15 02:48:26 +00:00
Jeff Morriss 27059c2804 Fix up some white space.
svn path=/trunk/; revision=50597
2013-07-15 02:05:54 +00:00
Jeff Morriss a05830e8b2 Try to fix the "make debian-package" buildbot step: update patch 03-preferences
so it applies more cleanly to the modern prefs.c .

Note: I built this patch partially by hand from "svn diff"; hopefully the
Debian stuff likes it.

svn path=/trunk/; revision=50596
2013-07-15 02:03:36 +00:00
Jeff Morriss 021175097a One more try at fixing all the issues reported in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8908 :

Rename check_expand_children() to check_expand_trees() since the function
doesn't just do the children of the current path/iter.

Add a parameter to that function which controls whether the parent (or just
the children) get expanded.

As suggest by Jakub in the bug: block further calls to expand_tree() when
we're expanding all these trees.  Add a comment telling callers of
check_expand_trees() to do that.

svn path=/trunk/; revision=50595
2013-07-15 01:46:47 +00:00
Jakub Zawadzki 6d7a3a8cee Rename frame_invalidate to frame_cache.
svn path=/trunk/; revision=50594
2013-07-14 23:32:08 +00:00
Jakub Zawadzki c3c6580d18 Fix assertion failed: ERROR:print.c:823:const guint8 *get_field_data(GSList *, field_info *): code should not be reached
svn path=/trunk/; revision=50593
2013-07-14 23:01:03 +00:00
Jakub Zawadzki a25c186eaf Yet another place where + offset was missing.
svn path=/trunk/; revision=50592
2013-07-14 22:43:52 +00:00
Guy Harris d296ebc525 All routines taking an offset and length that want to access the data in
the backing tvbuff must add the subset tvbuff's offset to the offset
argument, to skip to the subset.

svn path=/trunk/; revision=50591
2013-07-14 22:38:03 +00:00
Guy Harris b68e6dcc42 A bit more comment updating.
svn path=/trunk/; revision=50590
2013-07-14 20:26:56 +00:00
Guy Harris 221f39988b Fix incorrect use of dlsw.frame_direction for capability exchange
frames; that should be a separate dlsw.capex_type field.

dlsw.error_cause is a numeric field, so it needs a base; the values in
the value_string table are in hex, so show it in hex.

svn path=/trunk/; revision=50589
2013-07-14 19:34:45 +00:00
Guy Harris cc4197b1fe Fix a comment.
svn path=/trunk/; revision=50588
2013-07-14 19:07:09 +00:00
Guy Harris d3be9807c4 Get rid of duplicate "Old message type" field.
svn path=/trunk/; revision=50587
2013-07-14 19:02:19 +00:00
Guy Harris 0e40234217 Make pointers to raw packet data pointers to guint8, not pointers to
char; they're chunks of raw octets, not characters.

svn path=/trunk/; revision=50586
2013-07-14 17:39:36 +00:00
Guy Harris b161d15f0e Use tvb_new_chain with the tvbuffs from a reassembly.
svn path=/trunk/; revision=50585
2013-07-14 17:34:16 +00:00
Guy Harris 976cca3f98 A fragment_data structure now contains a pointer to a tvbuff to the
fragment or reassembled data, not a pointer to the data itself; instead
of having to construct a tvbuffs for the reassembled data, we can now
just use the existing tvbuff.

Spell out "fragment" and "fragmentation" in comments while we're at it.

svn path=/trunk/; revision=50584
2013-07-14 17:27:57 +00:00
Guy Harris a1ca4be5ef Cast the result of g_malloc(), to avoid warnings about trying to compile
this code as C++.

Make pointers to raw packet data pointers to guint8, not pointers to
char, as they're octets, not characters.

svn path=/trunk/; revision=50583
2013-07-14 17:16:56 +00:00
Jakub Zawadzki 543ba2746b fix: ../../asn1/t38/packet-t38-template.c:346: warning: request for implicit conversion from 'gpointer' to 'char *' not permitted in C++
svn path=/trunk/; revision=50582
2013-07-14 15:51:15 +00:00
Jakub Zawadzki 2671b8a398 fix: packet-mp2t.c:489: warning: implicit declaration of function 'tvb_new_proxy'
svn path=/trunk/; revision=50581
2013-07-14 15:20:09 +00:00
Jakub Zawadzki c766e78cfa Rewrite reassemble API to use TVBs instead of raw data.
(it seems to be working for TCP ^^)

svn path=/trunk/; revision=50580
2013-07-14 14:42:05 +00:00
Gerald Combs d924325d37 [Automatic manuf, services and enterprise-numbers update for 2013-07-14]
svn path=/trunk/; revision=50577
2013-07-14 14:03:38 +00:00
Michael Tüxen 43e20ff200 Don't use compiler options not supported by clang in XCode 4.6.3
on Apple platforms.

svn path=/trunk/; revision=50576
2013-07-14 12:39:25 +00:00
Guy Harris 1f32615d99 Export tvb_free_chain() so that tvbtest can use it.
As on Windows with MSVC, and as with reassemble_test, link tvbtest with
libwireshark and the libraries it requires.

svn path=/trunk/; revision=50575
2013-07-14 10:30:49 +00:00
Guy Harris f99694bb11 Squelch a "not permitted in C++" warning.
svn path=/trunk/; revision=50574
2013-07-14 10:23:59 +00:00
Guy Harris 0ca2dd85ae Build tvbtest the same way we build reassemble_test; see whether that
fixes the build errors we're seeing, with complaints about

	tvbtest.obj : warning LNK4217: locally defined symbol tvb_XX imported in function test
	tvbtest.obj : error LNK2019: unresolved external symbol __imp_tvb_XXX referenced in function run_tests
	tvbuff.obj : error LNK2019: unresolved external symbol tvb_XXX referenced in function tvb_new_YYY

svn path=/trunk/; revision=50573
2013-07-14 10:19:12 +00:00
Guy Harris eedc0601d7 Fix whitespace.
svn path=/trunk/; revision=50572
2013-07-14 09:17:31 +00:00
Jakub Zawadzki 2131f22508 Rewrite packet-mime-encap to use composite tvbs + cloning
svn path=/trunk/; revision=50571
2013-07-14 08:41:04 +00:00
Jakub Zawadzki b3e0623e5c Fix tvbtest linking, move private structures to tvbuff_subset
svn path=/trunk/; revision=50570
2013-07-14 07:59:42 +00:00
Jakub Zawadzki 0d85b75305 Move tvb real and subset implementations to seperate files.
svn path=/trunk/; revision=50569
2013-07-14 07:42:19 +00:00
Guy Harris c089afca5c OK, with Qt 5.1, the pkg-config modules are all called Qt5Xxx rather
than QtXxx, so look for those as well.

svn path=/trunk/; revision=50568
2013-07-14 02:35:26 +00:00
Michael Mann 79aea4b295 Generate better hf_ name
Generate initial filter name
Apply consistent whitespace

svn path=/trunk/; revision=50567
2013-07-14 02:13:07 +00:00
Michael Mann a32513416b First real test of convert_proto_tree_add_text.pl.
svn path=/trunk/; revision=50566
2013-07-14 02:01:49 +00:00
Evan Huus b54d5b7d54 Fix the OID registration leaks.
Only 1,625 bytes to go...

svn path=/trunk/; revision=50565
2013-07-14 01:02:44 +00:00
Evan Huus c7909c4195 Fix one small leak in xml dissector registration.
svn path=/trunk/; revision=50564
2013-07-14 00:54:02 +00:00
Evan Huus afa96c3f33 Fix some of the preferences leaks. Don't strdup the defaults, as they will be
strduped again immediately when they are registered. To make this safe, tweak
the flow a bit so that they are guaranteed to be registered immediately after
being set here.

svn path=/trunk/; revision=50563
2013-07-14 00:45:19 +00:00
Guy Harris bab8c3fdb9 Note that Qt builds are supported by autotools.
svn path=/trunk/; revision=50562
2013-07-14 00:10:17 +00:00
Martin Kaiser fb177c2fe9 set svn properties
svn path=/trunk/; revision=50561
2013-07-13 22:51:55 +00:00
Guy Harris bc239a7286 On OS X, set the rpath for executables to include
@executable_path/../lib as well as /usr/local/lib, so we can use @rpath
in the install names in the executables and libraries in the application
bundle.

Have the osx-app.sh script tweak all references to libraries from
/usr/local/lib in all executables, libraries, and plugins in the app
bundle to use @rpath.  (The "all" is important; it fixes the GTK+ crash
mentioned in the comment in osx-app.sh.  The notion of doing all of them
came from the osx-app.sh script in a newer version of Inkscape.)

This renders the setting of DYLD_LIBRARY_PATH in the wrapper scripts in
the bundle unnecessary; remove it.  (Ideally, we should try to get rid
of the wrapper scripts entirely, but that might have to wait for us to
switch to using Qt.)

svn path=/trunk/; revision=50560
2013-07-13 22:43:35 +00:00
Michael Mann 94f44f7601 Here's a tool to help in the conversion of proto_tree_add_text to proto_tree_add_item. It's still rough around the edges, but it's a lot better than doing it all manually. It targets proto_tree_add_text functions with formatted parameters. It "guesses" the necessary parameters for proto_tree_add_item and the associated hf_ strictly through proto_tree_add_text parameters, so it does better for cases where the tvb_ function (ie tvb_get_guint8) is one of the printf style parameters in the proto_tree_add_text.
If there are cleaner ways (better regex) to represent the logic, please feel free to update.

The key is to improve the "defaults" detected so less manual editting of the proto_tree_input file (and subsequent dissector file) is necessary.

I'll be testing it more (and probably updating) as I try to work through the checkAPIs.pl "naughty list" (now that the list has expanded)

svn path=/trunk/; revision=50559
2013-07-13 21:28:53 +00:00
Jakub Zawadzki ef93945811 Add support for cloning TVBs, move tvb_new() prototype to wtap-int.h
svn path=/trunk/; revision=50558
2013-07-13 17:53:33 +00:00
Evan Huus 9bdc911c29 From Dominic Spill via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8921
Update the README to reflect the value_string name changes in r48645.

From me: reorganize a bit to promote the use of val_to_str over try_val_to_str
in most cases.

svn path=/trunk/; revision=50557
2013-07-13 15:07:15 +00:00
Jakub Zawadzki af61f188e0 Add new function: validate_offset() which checks if offset is within bounds of tvb.
svn path=/trunk/; revision=50556
2013-07-13 14:51:25 +00:00
Evan Huus 611eaf8069 From Chris Botje via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8917
Minor Enhancement to add further to the Fast Message conversation structure to
provide Data Region Address/Name lookup.

From me: use tree for lookups, not slist.

svn path=/trunk/; revision=50555
2013-07-13 14:37:06 +00:00
Martin Kaiser 8bd14d8dba cygwin's dos2unix belongs to the Base category
svn path=/trunk/; revision=50554
2013-07-13 11:41:34 +00:00
Martin Kaiser 19de5ccfa1 From Vaibhav Katkade
Add Netflow Record TrustSec fields
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8928

svn path=/trunk/; revision=50553
2013-07-13 11:33:44 +00:00