summaryrefslogtreecommitdiff
path: root/print-ppp.c
Commit message (Collapse)AuthorAgeFilesLines
* NDOize 8 more small decodersDenis Ovsienko2014-03-131-4/+4
| | | | | This change converts ZeroMQ, IPX, MPLS, IPv6 options, PPPoE, RIPng, PFLOG and Sun RPC decoders.
* NDOize more small decodersDenis Ovsienko2014-03-121-1/+1
| | | | | This change converts decoders for: DLT_RAW, IPv6 routing header, USB, Symantec firewall and cHDLC.
* VJC: NDOizeDenis Ovsienko2014-03-121-2/+2
|
* print-ppp: variable 'length' is updated and the new value is never usedFrancois-Xavier Le Bail2014-03-071-1/+0
|
* Use a common style for all option parsers.Guy Harris2014-03-041-113/+143
| | | | | | | | | | | | | | | Report "length bogus, should be ..." for invalid lengths for all options. That's not a fatal error - the next option can be parsed - so don't return 0 in that case. Don't put a colon after the option length unless there's something following it. Do the TCHECKs for particular CCP options in the code to handle the option, as is done elsewhere. Print the values for the CCP options on the same line, as is done elsewhere.
* Print some PPP CCP options; taken from issue #214, with changes.Guy Harris2014-03-041-5/+63
| | | | | | | | | Report the CCP protocol type as CCP. Decode options for BSD, MRVCA, and DEFLATE compression. Add some explanatory comments, and add some length checks, while we're at it.
* don't include pcap.h needlesslyDenis Ovsienko2014-02-281-1/+0
| | | | | | | | | | Both interface.h and netdissect.h include <pcap.h>, thus most files should not include it regardless if these need it or not. The only exceptions so far remain: * addrtoname.c * missing/datalinks.c * missing/dlnames.c * tcpdump.c
* Another length check, also found by the Clang Static Analyzer.Guy Harris2014-01-161-0/+3
|
* Add some length checks for byte-escaped PPP.Guy Harris2014-01-161-4/+17
| | | | | Thanks and a tip of the Hatlo hat to the Clang Static Analyzer for catching these.
* remove tcpdump's own CVS keywordsDenis Ovsienko2014-01-031-5/+0
| | | | | | Remove lots of $Header's and a few $Id's that all belong to the former CVS repository of tcpdump itself. These keywords have been frozen since the migration to git in late 2008.
* ndo-ize print-ascii: hex_print_with_offset()Michael Richardson2014-01-011-16/+16
|
* whitespace changesMichael Richardson2014-01-011-6/+6
|
* justify declarations of struct tok arraysDenis Ovsienko2013-09-241-13/+13
| | | | | | Make sure all of them are declared const and most of them -- static. Proper declaration of token arrays is a common review point for new code that is based on existing decoders. Thus fix the issue at its root.
* Convert the IPv6 printer to use netdissect.Guy Harris2010-11-071-10/+10
|
* Don't directly fetch multi-byte integers from packets.Guy Harris2010-02-211-2/+2
| | | | | | | | Use the EXTRACT_ macros to extract multi-byte integral values from packets, rather than just dereferencing pointers into the packet; there is no guarantee that the packet data will be aligned on the right boundary, and there is no guarantee that, if they're not, a direct access will work correctly.
* add support for further dissection of the IPCP Compression Optionhannes2005-12-051-5/+71
|
* calculate correct length of LCP echo/echo-reply Magic-Num trailing datahannes2005-11-031-3/+4
|
* The data in an Echo-Request, Echo-Reply, or Discard-Request packet areguy2005-06-181-4/+14
| | | | | | | | | | | | just uninterpreted data; calling it "Unknown Data" makes it sound as if it's data that shouldn't be there or for which we don't have any code to interpret it. Just call it "Data". The data in a Protocol-Reject packet is the rejected packet; label it as such. The data in an Identification packet is a message intended to be human-readable; print it with "fn_printn()".
* Check to make sure PPP options have a minimum length of 2, as the lengthguy2005-06-161-1/+36
| | | | | | includes the lengths of the 1-byte type and length fields. Thanks and a tip of the hat to Gerald Combs for catching this one with his fuzz-testing stuff.
* -make the PPP printer multiline (behind vflag)hannes2005-05-081-105/+133
| | | | | -make more use of tok2str() -improve unknown option hexprinting
* Check, and use, the length from the PAP header.guy2005-04-201-1/+11
|
* refactored ip_print() so that chained header parser (ESP/AH) canmcr2005-04-061-4/+12
| | | | more easily call the inner parts.
* cleanup: use (PPP_ADDRESS << 8 | PPP_CONTROL) rather than a private 0xff03hannes2005-01-251-2/+2
|
* clean up the LCP Auth option printer, remove printf() chains and make use of ↵hannes2005-01-251-26/+17
| | | | tok2str() instead
* bugfix for MLPPP:hannes2004-10-281-4/+5
| | | | | | inserted the printer at the end of a fall-through chain which caused all packets to first get passed through the ML-PPP printer and causing bogus display output
* - add a brief ML-PPP printerhannes2004-10-201-4/+30
| | | | | | - add PPP_IPV6 in the PPP name resolution table - change the cryptic SSNHF LCP options to "12 Bit seq #" for better clarity to human devices
* -add basic support for IP6CPhannes2004-09-111-37/+87
| | | | | -make more use of tok2str() in the IPCP options printer -print length field for empty conf-req messages
* make use of tok2str() in the CHAP and PAP printers, misc. display cosmeticshannes2004-09-091-30/+20
|
* add ppp support for DLT_PPP_WITHDIRECTION; print direction (hidden under eflag)hannes2004-08-181-7/+26
|
* Fix a comment (that's *not* a "can't happen" situation), and clean upguy2004-07-151-6/+5
| | | | indentation.
* from Darren Reed <darrenr@reed.wattle.id.au>:hannes2004-07-131-1/+5
| | | | | | catch the self-set null pointer if handle_ctrl_proto() stumbles across an unknown control-proto for which we have no specific options decoder;
* From Darren Reed: just show protocol 0x00fb as "compressed PPP data".guy2004-07-061-4/+7
| | | | | Also, show protocols we don't dissect by name, if we have the name in "ppptype2str[]".
* per suggestion from darrenr@reed.wattle.id.au: resolve the PPP_COMP proto-idhannes2004-07-051-1/+2
|
* from Darren Reed <darrenr@reed.wattle.id.au>:hannes2004-07-021-1/+74
| | | | support for HDLC inside PPP
* call into the ip printer for non-compressed VJ IP packets, indicate the its ↵hannes2004-07-021-3/+4
| | | | IP inside the Van Jacobson encaps.
* Add bounds checks.guy2004-03-241-14/+111
|
* display cosmetics: remove the PPP- string from proto outputhannes2004-03-181-2/+2
|
* add Vendor OUI resulutionhannes2003-11-301-7/+12
|
* Have the configure script arrange that the Makefile define _U_guy2003-11-161-4/+4
| | | | | | | | | | appropriately, and that GNUmakefile and the MSVC++ project file define it apppriately, as we do with libpcap, rather than defining it in "interface.h". Undo the rcsid-shuffling and addition of extra #includes, as we no longer need to arrange that "interface.h" be included before using _U_ in an RCS ID or copyright.
* From Neil Spring:guy2003-11-151-4/+4
| | | | | | | | | | | use "_U_" in the definitions of "rcsid[]", to eliminate complaints about those variables being unused; move the definitions after the include of "interface.h", or add an include of "interface.h", so that "_U_" is defined. Include "config.h" before including "tcpdump-stdinc.h" in "missing/datalinks.c".
* print opcode number alwayshannes2003-10-201-3/+4
|
* don't pass on src & dst MAC adresses to the isoclns decoder as MAC adresseshannes2003-05-221-2/+2
| | | | should be really printed in ether_print() using the eflag
* print length after LCP optionshannes2003-05-221-2/+3
|
* cleanup Magic-Num fieldshannes2003-05-221-5/+5
|
* remove some EXTRACT_ lookalikes plus misc. display cleanuphannes2003-05-221-54/+60
|
* - be more verbose about the rejected protocol in handle_ctrl_proto()hannes2003-05-221-2/+7
| | | | - hook in MPLSCP, OSICP and IPV6CP handle_ctrl_proto()
* minor display cosmetics:hannes2003-05-151-2/+2
| | | | | print 3-digit ttl field in print-ip.c unbracketize the LCP id in print-ppp.c
* misc. display cosmetics for e/v flagshannes2003-05-111-5/+5
|
* hide link-layer information correct under the eflaghannes2003-05-081-2/+2
|
* Hoist a bunch of stuff that should be done by all if_print routines intoguy2002-12-191-83/+22
| | | | | | tcpdump.c. Have if_print routines return the length of the link-layer header, so that the common code knows how to skip the link-layer header when printing the packet in hex/ASCII.
View Lorry Controller Status