aboutsummaryrefslogtreecommitdiffstats
path: root/epan
AgeCommit message (Collapse)AuthorFilesLines
2020-03-26ASTERIX: Bugfix in CAT010Prince Paul1-4/+4
1. Add missing sparebits in I010_020 and I010_070 2. Correct scaling factor for I010_042 to 1 m instead of 0.5 Thanks to Marko Hrastovec <marko.hrastovec@gmail.com> for the suggestions. Change-Id: I22f360f9b687c4b38b36bb2d5dd81db36536497c Reviewed-on: https://code.wireshark.org/review/36586 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
2020-03-26TPNCP: Fix dissection of acTOpenOrUpdateSBCSessionOrgad Shaneh1-2/+17
Struct Layout: struct acTOpenOrUpdateSBCSession { int CidA; int CidB; int ChannelACommandSeqNumber; int ChannelBCommandSeqNumber; acMediationLevel AudioMediationLevel; int SourceDirection; int DestinationDirection; acAudioTranscodingMode AudioTranscodingMode; acSBCConnectionType SBCConnectionType; acTSBCChannelConfiguration ChannelAConfiguration; acTSBCChannelConfiguration ChannelBConfiguration; }; acTSBCChannelConfiguration size can vary, so Channel B might be incorrectly dissected. Fix by recognizing the first field of acTSBCChannelConfiguration and dividing the remaining size by 2, giving each channel the same byte count. Change-Id: I961394523fc307fa214d36c3eae5589f788fc6df Reviewed-on: https://code.wireshark.org/review/36584 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-26Add a dissector for the USB printer class.Martin Kaiser2-0/+191
Start dissecting the class-specific control messages. Change-Id: I21e97777c9fc0396a8c0c575ba21909f58bbb577 Reviewed-on: https://code.wireshark.org/review/36539 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx> Petri-Dish: Martin Kaiser <wireshark@kaiser.cx> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25rlcmac: Introduce MS Radio Access Capabilities 2 to fix related spare bitsPau Espin Pedrol2-10/+20
There's two variants for the Ms Radio Access Capabilities. * The usual encoding with spare bits (usually to fill up to octet boundary) as defined in TS 24.008 Table 10.5.146 And there's too: * MS Radio Access Capabilities 2 IE from TS44.060 section 12.30, which is the same but removing all spare bits, and which is used in messages like Packet Resource Request and Additional MS RAC messages. The later is used basically for messages having extra IEs after the MS Radio Access capabilities IE, since they are encoded immediatelly afterwards. So this patch does: * Adds the expected spare bits (M_PADDING) to MS_Radio_Access_capability_t * Creates a new MS_Radio_Access_capability2_t without padding * Updates code to use the new "2" version where needed. The new MS_Radio_Access_capability_t with padding is disabled since it's not used in wirehsark because it's using an ad-hoc decoder for that TS 24.008 IE: see packet-gsm_a_gm.c de_gmm_ms_radio_acc_cap() Port from osmo-pcu.git e50ce6e45c4509805807d599cadf1a1b23d37f63 Related: https://osmocom.org/issues/4463 Change-Id: Ie2844bd94437c8d7a50787cc7c8c4d81a4fc48ba Reviewed-on: https://code.wireshark.org/review/36574 Petri-Dish: Pascal Quantin <pascal@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-25rlcmac: Fix bug receiving RA capPau Espin Pedrol1-13/+5
It seems the assumptions regarding maximum number of RA capabilitites in one message were wrong. Doing some rough calculations, each RA capabilitiy value (without extensions) can take around 20ish bits, which means for a message containing up to 52 bytes that quite a lot of different values could be theoretically fed in. Let's be safe and increase the array size to be able to handle all different access technologies listed in See TS 24.008 table 10.5.146 following restrictions: * "The MS Radio Access capability is a type 4 information element, with a maximum length of 52 octets." * "Among the three Access Type Technologies GSM 900-P, GSM 900-E and GSM 900-R only one shall be present." * "the mobile station should provide the relevant radio access capability for either GSM 1800 band OR GSM 1900 band, not both". Port from osmo-pcu.git 7faa5da209d0ef48fe593603c217615f09fb61fb. Related: https://osmocom.org/issues/4463 Change-Id: Ief5189f88ba0e4970847567c9a15b1ada8b9df4b Reviewed-on: https://code.wireshark.org/review/36573 Reviewed-by: Harald Welte <laforge@gnumonks.org> Petri-Dish: Pascal Quantin <pascal@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-25rlcmac: Don't pass array element to CSN1 descriptorsPau Espin Pedrol1-12/+11
This way the macros can be used to access the arrays themselves and calculate its static size to enable validation lateri on. In the case of Packet_Access_Reject_t, modify the description to use a M_REC_TARRAY_1 object to get rid of access to 2nd element. The new description is the correct one, since the first element is mandatory according to TS 44.060 Table 11.2.1. Port from osmo-pcu.git 81b40cbaf3070f70954663f68375100128bdc77e. Change-Id: I48fc257305af799beb6a48ffc6d11b73026bff51 Reviewed-on: https://code.wireshark.org/review/36571 Reviewed-by: Harald Welte <laforge@gnumonks.org> Reviewed-by: Vadim Yanitskiy <axilirator@gmail.com> Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-25csn1: Validate recursive array max size during decodingPau Espin Pedrol2-5/+19
This way if CSN1 encoded bitstream contains more elements than what the defintion expects it will fail instead of overflowing the decoded buffer. Example: RA Capabilities struct (recursive array) sent by a real android phone when attaching to the network. Then SGSN sends it back and osmo-pcu would crash similar to this: *** stack smashing detected ***: terminated Process terminating with default action of signal 6 (SIGABRT): dumping core at 0x4C62CE5: raise (in /usr/lib/libc-2.31.so) by 0x4C4C856: abort (in /usr/lib/libc-2.31.so) by 0x4CA62AF: __libc_message (in /usr/lib/libc-2.31.so) by 0x4D36069: __fortify_fail (in /usr/lib/libc-2.31.so) by 0x4D36033: __stack_chk_fail (in /usr/lib/libc-2.31.so) by 0x124706: testRAcap2(void*) (RLCMACTest.cpp:468) Port from osmo-pcu.git efad80bfbffb2a35d2516e56dc40979f19c6c370 Related: https://osmocom.org/issues/4463 Change-Id: I6bdd6960141829491aebbfdaab548c41d4a3bc9f Reviewed-on: https://code.wireshark.org/review/36572 Reviewed-by: Harald Welte <laforge@gnumonks.org> Petri-Dish: Pascal Quantin <pascal@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-25lltd: fix typo found by lintian (Phyiscal => Physical)Alexis La Goutte1-1/+1
Change-Id: I697ea60c2a31b18175f2e57fd7721888d5f99668 Reviewed-on: https://code.wireshark.org/review/36568 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25fix typo found by lintian (Uknown => Unknown)Alexis La Goutte3-3/+3
Change-Id: I50ae0d1595cc42b3432f4611009c9c48b4cdde70 Reviewed-on: https://code.wireshark.org/review/36567 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25ASTERIX: Add support to dissect CAT010Prince Paul1-1/+604
Add support to dissect ASTERIX CAT010 Version 1.10 Change-Id: I82f59555e75c03fc4bd2819b2455f7fd6fa08384 Reviewed-on: https://code.wireshark.org/review/36560 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
2020-03-25TPNCP: Use security_cmd_offset valueOrgad Shaneh1-21/+64
The layout of OpenSecureChannel command is: struct acTOpenSecureChannelSendCmd { int CID; struct acTOpenChannelCmd OpenChannelCmd; struct acTOpenSecureChannelCmd SecureChannelCmd; }; OpenChannelCmd contains a field that stores the offset of SecureChannelCmd (rtp_authentication_algorithm) from the beginning of OpenChannelCmd (cmd_rev_lsb). Support both forward and backward compatibility (added or missing fields). Change-Id: If277996aa7fb94fef55ae34fed5a307728ebdf7c Reviewed-on: https://code.wireshark.org/review/36566 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25TPNCP: Simplify names for data field infoOrgad Shaneh1-86/+74
Change-Id: Id02c6bd553251ed1c318af2687f4c9d2d3b6db8f Reviewed-on: https://code.wireshark.org/review/36565 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25fix typo found by lintian (mesage => message)Alexis La Goutte6-7/+7
Change-Id: I51654f51b2affd2363966900be4d57824d1c9385 Reviewed-on: https://code.wireshark.org/review/36563 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25GTPv2: Dissect IE 8.106 Node NumberAnders Broman1-1/+20
Change-Id: I910910f9447048bf1a5645ad01f07da8fa0bfb59 Reviewed-on: https://code.wireshark.org/review/36561 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-25RDMnetJim Schaettle1-0/+9
Fixing infinite loop bugs in multiple places. This fix is not in bugzilla, it was found by inspecting the code. Change-Id: I7c58242d298a73fd929c327d254b34b3e60cf383 Reviewed-on: https://code.wireshark.org/review/36557 Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl> Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-24wslua: WSLUA_BIT_OP_FUNC macro already does return .Martin Mathieson1-6/+0
epan/wslua/wslua_int64.c:445: style: Consecutive return, break, continue, goto or throw statements are unnecessary. epan/wslua/wslua_int64.c:454: style: Consecutive return, break, continue, goto or throw statements are unnecessary. epan/wslua/wslua_int64.c:463: style: Consecutive return, break, continue, goto or throw statements are unnecessary. epan/wslua/wslua_int64.c:966: style: Consecutive return, break, continue, goto or throw statements are unnecessary. epan/wslua/wslua_int64.c:975: style: Consecutive return, break, continue, goto or throw statements are unnecessary. epan/wslua/wslua_int64.c:984: style: Consecutive return, break, continue, goto or throw statements are unnecessary. Change-Id: I98d8c07cb13c523ba21469b7ad84ff4738764d49 Reviewed-on: https://code.wireshark.org/review/36556 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
2020-03-24NAS-5GS: shift supi_fmt field before testingMartin Mathieson1-2/+1
From cppcheck: epan/dissectors/packet-nas_5gs.c:749: style: Mismatching assignment and comparison, comparison 'supi_fmt==1' is always false. Change-Id: I702f0cae24fd95dc925a87ab3766f2995f3eb759 Reviewed-on: https://code.wireshark.org/review/36554 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-24USBLL: Handle addresses for all packetsAmeya Deshpande1-8/+137
This is based on patch created by Krzysztof Opasiak. USB is a stateful protocol. So we store the link to the previous packets. Ping-Bug: 15908 Change-Id: I5d844041b68375a5b3ffc46f1e8ffca2804ed5b6 Reviewed-on: https://code.wireshark.org/review/34309 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23ACDR: Support LawfulIntercept bitOrgad Shaneh2-11/+23
Change-Id: I28613791ac9262277ab1636f4ec476427c4f1db9 Reviewed-on: https://code.wireshark.org/review/36549 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23kerberos: fix compilation when kerberos is unavailable.Dario Lombardo3-79/+135
Small indentation fixes. Change-Id: I62ba59f71dd530f82b5f0982fc0600737c5778c0 Reviewed-on: https://code.wireshark.org/review/36547 Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Petri-Dish: Dario Lombardo <lomato@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23kerberos: remove redundant returns.Dario Lombardo2-52/+28
It's already incluided in the template. Change-Id: I534b0b116f232e2b0168e141ad865ec6c96612d3 Reviewed-on: https://code.wireshark.org/review/36546 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23kerberos: change indentation from tabs to spaces.Dario Lombardo2-563/+563
The asn1 dissector generator creates file with 2 spaces indentation. The right way to indent a cnf file is the same, then. Change-Id: I6a210ba17b332234678698d111863e27830dcece Reviewed-on: https://code.wireshark.org/review/36545 Petri-Dish: Anders Broman <a.broman58@gmail.com> Petri-Dish: Dario Lombardo <lomato@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23wassp: fix typo found by lintian (Unknown => Unknow)Alexis La Goutte1-1/+1
Change-Id: I3dcc71c590e6fb29f1d476fb73625746413fdb9f Reviewed-on: https://code.wireshark.org/review/36548 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23btmesh: fix typo (Accesss => Access)Alexis La Goutte1-1/+1
Change-Id: I17b94100139d3262315cb42b66fc245ade82224b Reviewed-on: https://code.wireshark.org/review/36544 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23systemd-journal: fix lintian (Unkown => Unknown)Alexis La Goutte1-14/+14
Change-Id: I83070917b76c0d730148b29cc38151648a1a88de Reviewed-on: https://code.wireshark.org/review/36542 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23fix typo found by lintian (Unkown => Unknown)Alexis La Goutte3-3/+3
Change-Id: I32f636c5a8de652303fbe03668bf937004f2e6a8 Reviewed-on: https://code.wireshark.org/review/36543 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Tomasz Moń <desowin@gmail.com>
2020-03-23dot11crypt_system(.h): fix WdocumentationAlexis La Goutte1-8/+0
dot11decrypt_system.h:274:11: warning: parameter 'decrypt_data' not found in the function declaration [-Wdocumentation] dot11decrypt_system.h:277:11: warning: parameter 'decrypt_len' not found in the function declaration [-Wdocumentation] dot11decrypt_system.h:279:11: warning: parameter 'key' not found in the function declaration [-Wdocumentation] Change-Id: I9a0d1dd29ddd130b9425a38645777e4eb05e9fa3 Reviewed-on: https://code.wireshark.org/review/36540 Reviewed-by: Mikael Kanstrup <mikael.kanstrup@sony.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-23dot11decrypt: Fix decryption of MFP enabled connectionsMikael Kanstrup1-9/+10
MFP enabled connections with SHA-256 key management (IEEE 802.11w) use EAPOL key version == 3. This case was missing making decryption of such connections fail. Allow key version 3 to handle these too. Change-Id: If9e3fcc5c3bbfb46e82b39dfed5b2a74787a4f16 Reviewed-on: https://code.wireshark.org/review/36534 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-22RTCP: contain BYE packet dissection to its stated lengthJaap Keuter1-3/+3
The Goodbye RTCP packet dissection has to derive whether the optional "Reason for leaving" string is present or not. This has to be derived from the length. When put into a compound RTCP packet, the length derivation from the TVB length does not work, because another RTCP packet may follow in this compound RTCP packet. With this change the stated length of this RTCP packet is passed to the RTCP BYE packet dissection function in order to make a proper length determination and not overrun into the next RTCP packet in the compound RTCP packet. Bug: 16434 Change-Id: Iab0fdd52c745028a9928bbef6c731ff649213277 Reviewed-on: https://code.wireshark.org/review/36532 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-22[Automatic update for 2020-03-22]Gerald Combs1-5/+53
Update manuf, services enterprise numbers, translations, and other items. Change-Id: Ia5e7f2ad5422ba028b3205b8c2a5fe568578d828 Reviewed-on: https://code.wireshark.org/review/36528 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2020-03-21Some cppcheck fixes.Martin Mathieson5-5/+5
- UDPCP: fix test for Sync frame detection - Avoid a couple of redundant assignemnts Change-Id: Ib69a3ff4f0e802ddbbbd29eb2a803372c41a7ce4 Reviewed-on: https://code.wireshark.org/review/36526 Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
2020-03-21Fix the nanosecond-resolution pcap magic numbers.Guy Harris1-2/+2
See NSEC_TCPDUMP_MAGIC in https://github.com/the-tcpdump-group/libpcap/blob/master/sf-pcap.c The last octet is 0x4d, not 0xd4. Change-Id: Ie49a7a9c33e59fe119746bec910522a46b46c2f6 Reviewed-on: https://code.wireshark.org/review/36519 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2020-03-20NAS 5GS: improve dissection of SUCIPascal Quantin1-8/+42
- a SUCI with null scheme is always a MSIN, name it accordingly - add breakout of scheme output for profiles A and B Change-Id: I3453336291cbd515809054057a6e53902590e896 Reviewed-on: https://code.wireshark.org/review/36514 Reviewed-by: Pascal Quantin <pascal@wireshark.org> Petri-Dish: Pascal Quantin <pascal@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-20couchbase: Add new DCP commandsJim Walker1-0/+50
Add support for: * DCP seqno advanced * DCP oso snapshot Change-Id: I6461ac3d1a16829e36e5c259a50943ef254b197d Reviewed-on: https://code.wireshark.org/review/36506 Reviewed-by: Dave Rigby <daver@couchbase.com> Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-20gsmtap: Support dissection of non CS-1 (E)GPRS data blocksPau Espin Pedrol1-1/+217
Work based on current code from epan/dissectors/packet-gsm_abis_pgsl.c, as well as on initial patch from Holger Hans Peter Freyther [1]. Tested with one downlink MCS1 data block generated by osmo-pcu. [1] https://osmocom.org/issues/1542 Change-Id: I01a8bd1cdb78d1c236a451fbee37854eb688fa14 Reviewed-on: https://code.wireshark.org/review/36489 Reviewed-by: Harald Welte <laforge@gnumonks.org> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-19SOMEIP: SD did not register port for IPv6 correctly (bugfix)Dr. Lars Völker1-3/+4
SOME/IP-SD did not register a port learned correctly, so that packets with this port number were not dissected. Bug: 16448 Change-Id: Ic648a50bf08dcab65c49182e08257cf6660261a5 Signed-off-by: Dr. Lars Völker <lars.voelker@technica-engineering.de> Reviewed-on: https://code.wireshark.org/review/36503 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-19s_addr -> src_addr, d_addr -> dst_addr.Guy Harris1-37/+37
On at least some platforms (at least some versions of the Windows SDK, and netinet/in.h on Illumos and thus possibly Solaris, and possibly others), struct in_addr's only member is a union, with some members of the union being substructures with fields corresponding to the pre-CIDR parts of an IPv4 address, with s_addr #defined to be the member of the union corresponding to the IPv4 address as a 32-bit integral value. That's presumably maintained for source compatibility with really old programs written before CIDR, on platforms where they worry about that. The *BSDs don't care, and macOS came out post-CIDR and never supportede the pre-CIDR code in the first place, so they don't do that; the same probably applies to Linux. This causes problems when s_addr is used as a variable name. In addition, inside Wireshark we tend to use "src" rather than just "s" as a prefix to refer to source addresses and ports - and "dst" or "dest" rather than just "d" as a prefix to refer to destination addresses and ports. So just go with that. Change-Id: I6010d5ea9697d2333597c06756fa393a5898364e Reviewed-on: https://code.wireshark.org/review/36504 Petri-Dish: Guy Harris <guy@alum.mit.edu> Tested-by: Petri Dish Buildbot Reviewed-by: Guy Harris <guy@alum.mit.edu>
2020-03-19NR RRC: fix dissection of q-RxLevMin parameterPascal Quantin2-6/+10
Change-Id: Ia8aa965e19d7ad55d7197cb636b203140abfba5b Reviewed-on: https://code.wireshark.org/review/36501 Petri-Dish: Pascal Quantin <pascal@wireshark.org> Tested-by: Petri Dish Buildbot Reviewed-by: Pascal Quantin <pascal@wireshark.org>
2020-03-19RTPS: hf_rtps_nokey_data_frag_number is FT_UINT32John Thacker1-3/+3
In (deprecated) NOKEY_DATA_FRAG, the frag_number field is a uint32 so ret_uint should be used, etc. Fixing to be like DATA_FRAG. Change-Id: I75848dfbde114cf6e4bdc6864d0690a75ca58d1d Reviewed-on: https://code.wireshark.org/review/36500 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2020-03-19packet-kerberos: split out a decrypt_krb5_data_asn1() helper functionStefan Metzmacher2-43/+63
If decrypt_krb5_data_private() is available we use it directly in order to pass down kerberos_private_data_t. Change-Id: I864d3e7d9adda454c96384d7b146ec774cd05d5f Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36491 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-19packet-kerberos: split out a decrypt_krb5_data_private() functionStefan Metzmacher2-19/+61
This allows passing 'kerberos_private_data_t' down to used_encryption_key(). This will be used in order to implement Kerberos FAST decryption. For now we'll pass a zeroed kerberos_private_data_t, but in future code can use decrypt_krb5_data_private() directly and pass in the result of kerberos_get_private_data(actx). Change-Id: Iffdd3c3168eca3ed90cfa0a924248df9fac98a0c Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36490 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: rename some generated fields to avoid name conflictsStefan Metzmacher2-41/+44
The following changes would otherwise create conflicts. Change-Id: I80bf9270aa1bdc1b92602ef68c9bfbf2b2b7ee37 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36471 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: dissect PA_REQ_ENC_PA_REPStefan Metzmacher3-24/+33
Change-Id: Ib0adf5a74e0872c6fc84dbd1dae27d0622f797ef Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36470 Petri-Dish: Anders Broman <a.broman58@gmail.com> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: dissect new RFC6113 errors messagesStefan Metzmacher3-30/+56
Change-Id: I2b95bc257762cc8734386171da1bf574220c4530 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36469 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: remove unused line '#EncryptedData/etype encryptedData_etype'Stefan Metzmacher2-33/+32
Change-Id: I544e22bfd7977021bd40b88df8bd549201f97a36 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36468 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: dissect names of new PAC_* typesStefan Metzmacher2-18/+120
For now we don't dissect the details, but it's already useful, if they are not unknown elements. Change-Id: I38b521262b688ba0afbbb9c58b99c3b50dbd2b24 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36467 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: Encrypted{AuthorizationData,Authenticator} are not the sameStefan Metzmacher4-42/+173
We need to dissect them differently... Change-Id: Idb6d65800b1787b9cb6fca2630373547b9b7b1bc Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36466 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: we only need PA-PAC-REQUEST onceStefan Metzmacher4-74/+68
Change-Id: I6c3d3810456d30877802fbd5d8d8ee64df2417f7 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36465 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: decrypt PAC_CREDENTIAL_INFO from [MS-PAC]Stefan Metzmacher2-32/+154
In future we could further dissect the details, but seeing the decrypted bytes is already very useful. Change-Id: Id03664513e66b089e2815140a061ec90b9a49232 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36464 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
2020-03-18packet-kerberos: dissect AD_GSS_API_ETYPE_NEGOTIATIONStefan Metzmacher3-13/+21
Change-Id: Ic4f0e92847151c467f7cc5fca3495e51586d8fd8 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-on: https://code.wireshark.org/review/36463 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-10-05 00:19:35 +0000