aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-lustre.c
AgeCommit message (Collapse)AuthorFilesLines
2018-09-11sizeof isn't useful when calculating the size of data in the packet.Guy Harris1-2/+2
Srsly, you're dealing with *four-byte* value, might as well say "4" - either sizeof(guint32) is guaranteed to be 4, in which case you might as well just use 4, or it's *not* guaranteed to be 4, in which case you *have* to use 4, because a value other than 4 is invalid. Change-Id: I6deb106f326e9402744a2e728468b3ce4c220b02 Reviewed-on: https://code.wireshark.org/review/29586 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2018-09-08Register some unregistered etts.Jeff Morriss1-0/+1
... Use of an unregistered ett leads to an abort. Inspired by I3ee2f557ace1643dfba5a978add66c3c7ba7d895. Some day I should get the ett_ registration checking code in checkAPIs ready for prime time... Change-Id: I69162d4bcec571e6a517a107ac365aa78bfe8d25 Reviewed-on: https://code.wireshark.org/review/29474 Petri-Dish: Jeff Morriss <jeff.morriss.ws@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-04-05Lustre: Add barrier processingNathaniel Clark1-22/+243
In lustre 2.10 the ability to freeze/thaw the filesystem for snapshotting was added. This processes the new wire commands used. Change-Id: Ie4d87b200078d2da1c983c9e8df2952c82c90d83 Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com> Reviewed-on: https://code.wireshark.org/review/26692 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-03-25Clean up REPORT_DISSECTOR_BUG().Guy Harris1-5/+4
Have it take a format and argument list as arguments, and have the formatting done inside the reporting code. That way, we're not relying on any particular wmem scope working. If WIRESHARK_ABORT_ON_DISSECTOR_BUG is set, try to add the message to the crash information (currently only supported in macOS), and print it to the standard error, before crashing. We won't necessarily have a usable crash dump to analyze, so we can't rely on that to find the cause of the crash. Ping-Bug: 14490 Change-Id: I2b39169c45c84f2ada31efa1d413bd28c140f8f4 Reviewed-on: https://code.wireshark.org/review/26643 Petri-Dish: Guy Harris <guy@alum.mit.edu> Reviewed-by: Guy Harris <guy@alum.mit.edu>
2018-02-08replace SPDX identifier GPL-2.0+ with GPL-2.0-or-later.Dario Lombardo1-1/+1
The first is deprecated, as per https://spdx.org/licenses/. Change-Id: I8e21e1d32d09b8b94b93a2dc9fbdde5ffeba6bed Reviewed-on: https://code.wireshark.org/review/25661 Petri-Dish: Anders Broman <a.broman58@gmail.com> Petri-Dish: Dario Lombardo <lomato@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-12Lustre: add missing NULL terminator to flags listsPascal Quantin1-0/+2
Bug: 14331 Change-Id: I439b1fe36ecb2e063146d7aebf517d842d4f8a4c Reviewed-on: https://code.wireshark.org/review/25277 Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2018-01-10Lustre: capitalize comment to please GCC7Pascal Quantin1-1/+1
Change-Id: I30c31dcdbfdfd99dfe204e2ba67a221f356a5088 Reviewed-on: https://code.wireshark.org/review/25242 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2018-01-09lustre: fix format 'long unsinged int' errorUli Heilmeier1-4/+4
Change-Id: I764aa32c5571e667917c202feb70612dd877fc40 Reviewed-on: https://code.wireshark.org/review/25231 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2018-01-09lustre: add missing breaks (fix gcc-7 compilation).Dario Lombardo1-0/+2
Change-Id: I00f0901853818df5f6bf67d7d75720067d66ee50 Reviewed-on: https://code.wireshark.org/review/25225 Petri-Dish: Dario Lombardo <lomato@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Nathaniel Clark <nathaniel.clark@misrule.us> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2018-01-09[lustre] Initial Lustre supportNathaniel Clark1-0/+8078
This interperates the main body of Lustre traffic. This dissects all current Lustre OPCODES (as of Lustre 2.10.2) This dissects MDS REINT sub-opcodes This dissects LDLM Intent opcodes This dissects LLOG EADATA Conversation matching is just IP based and not IP/port based. Only one lustre "instance" can be running on a given host at a given time, and request / reply pairs aren't don't always match by port numbers. Add exception for lustre_* structure names in PROTOABBREV. We have several lustre.lustre_* because the internal lustre structre is named lustre_ (i.e. lustre_handle or lustre_msg_v2) This is still a work in progress, as there are missing FLAG values and some LLOG EADATA structures that aren't fully decoded. Change-Id: If57085e2692565336e49f40fb475ca1035da7a35 Signed-off-by: Nathaniel Clark <nathaniel.l.clark@intel.com> Reviewed-on: https://code.wireshark.org/review/24800 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>