Commit Graph

8 Commits

Author SHA1 Message Date
AndersBroman c68769765c Fix no previous prototype warnings.
Change-Id: I859597ebc957a58f659800ef63318f94e80feabf
Reviewed-on: https://code.wireshark.org/review/1104
Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-04-14 12:50:31 +00:00
Guy Harris ea7660a301 Fix two ett_ variables that weren't getting registered.
Change-Id: I0b37bf3d89a108978a6c917fc4d3a33cba456493
Reviewed-on: https://code.wireshark.org/review/1048
Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-04-10 01:06:50 +00:00
Guy Harris 7d458fa4a6 Make sure IPv4 address structures really have 4 bytes of data.
Change-Id: If89827677f7503fc67ba9a025f28447a68a98f07
Reviewed-on: https://code.wireshark.org/review/1025
Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-04-08 18:04:22 +00:00
Guy Harris beb4876cb4 Don't assume the data pointed to by an address structure is aligned.
And don't assume you can dereference the pointer nonetheless; that
doesn't work on SPARC, for example - you get an unaligned-access trap.

Instead, use pntoh32() to fetch IPv4 address values from the address
structures.

While we're at it, just use guint32 for those addresses; we don't need
in_addr_t.

Change-Id: I84e6c653fe33b1bc6e67d9097ce423b82f1eb0c8
Reviewed-on: https://code.wireshark.org/review/1024
Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-04-08 17:58:30 +00:00
Graham Bloice 7077aaa54f Fixes for Windows build post change 1006
Windows still needs the definition of in_addr_t

Change-Id: I43c417de8e8199cfa58b9d494be5e828f959f1a9
Reviewed-on: https://code.wireshark.org/review/1009
Reviewed-by: Graham Bloice <graham.bloice@trihedral.com>
Tested-by: Graham Bloice <graham.bloice@trihedral.com>
2014-04-08 11:31:37 +00:00
Guy Harris d3df2e588b On UN*X, <arpa/inet.h> suffices for struct in_addr and in_addr_t.
So don't pull in <netinet/in.h>.  Also, avoid <sys/types.h> in
packet-dcom.c.

While we're at it, do *not* assume that pinfo->src or pinfo->dst are
IPv4 addresses.

Change-Id: I5fc8e859780a8d863aaf6e90a21a7039cabae0e6
Reviewed-on: https://code.wireshark.org/review/1006
Reviewed-by: Guy Harris <guy@alum.mit.edu>
2014-04-08 08:23:34 +00:00
Alexis La Goutte abbdcd8750 Fix warning found by -Wunused-const-variable (Clang 3.5)
Change-Id: I4506be6ae8deda5e8d1e038b089620a15b650b03
Reviewed-on: https://code.wireshark.org/review/1000
Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
2014-04-08 08:15:11 +00:00
Michael Mann 2f4ca9c8d9 Initial checkin of LBM aka 29West dissectors. See Bug 9718.
Change-Id: If8fcfe1971c8863f370e440f64c36eb7566f6852
Reviewed-on: https://code.wireshark.org/review/113
Reviewed-by: Michael Mann <mmann78@netscape.net>
2014-04-07 23:28:46 +00:00