aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2016-02-08Revert "Add free_address_wmem() and other extensions to address API"João Valverde58-395/+400
This reverts commit 13ec77a9fc3af3b0b502820d0b55796c89997896. This commit introduces a segmentation fault for Lua code (uncovered by the test suite). Change-Id: Ibc273d1915cda9632697b9f138f0ae104d3fb65e Reviewed-on: https://code.wireshark.org/review/13813 Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07TCAP: Fix warning [-Wunused-function]João Valverde2-23/+48
Change-Id: Ie30f39cf1d79f571824cb02583130e9ec828ede4 Reviewed-on: https://code.wireshark.org/review/13788 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07SNMP: Fix warning [-Wunused-function]João Valverde4-13/+4
Add SNMP to clean dissectors also. Change-Id: Ifc4d5a03c13f864bd9219294028323761cd19368 Reviewed-on: https://code.wireshark.org/review/13786 Petri-Dish: João Valverde <j@v6e.pt> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07autotools: Move SBC dependency to libwscodecsJoão Valverde5-22/+27
Change-Id: I931961033798613b78f846c9176dffdb8385bf43 Reviewed-on: https://code.wireshark.org/review/13782 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07Add free_address_wmem() and other extensions to address APIJoão Valverde58-400/+395
Try to improve 'address' API (to be easier/safer) and also avoid some constness warnings by not overloading the 'data' pointer to store malloc'ed buffers (use private pointer for that instead). Change-Id: I7456516b12c67620ceadac447907c12f5905bd49 Reviewed-on: https://code.wireshark.org/review/13463 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07GTK: Check if lrint() is usable, fix warnings [-Wbad-function-cast]João Valverde5-62/+76
Change-Id: I4714ba6cfbd18847d99650f5c5cdc60dd2e7af26 Reviewed-on: https://code.wireshark.org/review/13808 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07proto_data: Remove superfluous castsJoão Valverde1-5/+5
The casts should be to gpointer but are unnecessary. Change-Id: I27229750ec53138820cce30b6fcc083d3ae5bea1 Reviewed-on: https://code.wireshark.org/review/13810 Reviewed-by: João Valverde <j@v6e.pt>
2016-02-07Check whether Ness is valid as well.Guy Harris1-1/+5
It's also used as an array index, and it also is no guaranteed to be in the proper range in all capture files, so bounds-check it. Make the variable's type match that of the field in the radio metadata. (It's never negative, so it's unsigned.) This appears to be the last of the unchecked array references from bug 12085. Bug: 12085 Change-Id: I3e5b821a9b70dfd520d01036f6f7d02008aca200 Reviewed-on: https://code.wireshark.org/review/13806 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2016-02-07MAX_MCS_INDEX is a valid array index.Guy Harris1-1/+1
The arrays have MAX_MCS_INDEX+1 entries; valid indices to from 0 to MAX_MCX_INDEX. Change-Id: Ibbd30d36cc16143482b34212b1c6ce8df9e555ab Ping-Bug: 12085 Reviewed-on: https://code.wireshark.org/review/13805 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2016-02-07MAX_MCS_INDEX is a valid array index.Guy Harris1-1/+1
The arrays have MAX_MCS_INDEX+1 entries; valid indices to from 0 to MAX_MCX_INDEX. Change-Id: Ib8b939e687a00505cab5e5f9b3b6dce34e9d0b47 Reviewed-on: https://code.wireshark.org/review/13803 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2016-02-07[Automatic update for 2016-02-07]Gerald Combs16-350/+461
Update manuf, services enterprise-numbers, translations, and other items. Change-Id: If7d44f7c40a98454261f9f71ce7e084a16b5add7 Reviewed-on: https://code.wireshark.org/review/13799 Reviewed-by: Gerald Combs <gerald@wireshark.org>
2016-02-07ndps: don't THROW() an exception from a dissectorMartin Kaiser1-21/+3
if we just return the offset, there's no need to check for overflows the caller continues dissection from the returned offset if the offset is invalid, the core routines will throw an exception if we call proto_item_set_end() with the calculated offset, we can do the range check using tvb_ensure_bytes_exist() Change-Id: I0e2a7b2a866596eda0d7ed4a948fbea8ed084845 Reviewed-on: https://code.wireshark.org/review/13792 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-06TCAP: fix indent and add modelines (use 2 spaces)Alexis La Goutte3-243/+282
Change-Id: I3ac35f3dd6514469bdcabd0acd67970077d5c364 Reviewed-on: https://code.wireshark.org/review/13793 Reviewed-by: João Valverde <j@v6e.pt>
2016-02-06llrp: limit recursion depthEvan Huus1-32/+35
LLRP defines fairly complex parameter nesting which we handle via recursion, however this means a large crafted packet could cause very deep stacks and potentially stack overflows. Limit our recursive depth to an arbitrary, which should be more than enough for any legitimate packet (I hope). Bug: 12048 Change-Id: I9ac31bddfa4ffd1a79809387d10d2261749b95e7 Reviewed-on: https://code.wireshark.org/review/13794 Reviewed-by: Evan Huus <eapache@gmail.com>
2016-02-06asn1: Cleanup space usageStig Bjørlykke12-111/+99
Fix space issues in some ASN.1 dissectors. Change-Id: I4ceccfbe9a13c93fc91821d1bfe4b7d6bb39c435 Reviewed-on: https://code.wireshark.org/review/13791 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2016-02-06TCAP: Fix dissect_tcap_ERROR() unused warning [-Wunused-function]João Valverde2-19/+1
Change-Id: I0d9511ed9cb93446766d510b97fdefe56a86a826 Reviewed-on: https://code.wireshark.org/review/13787 Petri-Dish: João Valverde <j@v6e.pt> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-06Bluetooth: GATT: Add ability to filtering by Service and Characteristic ↵Michal Labedzki1-81/+127
level UUIDs There are three level of characteristics: Services, Characteristics and Configuration Characteristics. To properly analyzing packets there is a need to display all three why dissecting UUID. Change-Id: I3121338942c990b52ac2b3a45ced0529f80d4114 Reviewed-on: https://code.wireshark.org/review/13742 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2016-02-06Bluetooth: GATT: Add support for OTS/OTPMichal Labedzki6-39/+728
OTS/OTP is the newset "profile" announced by Bluetooth SIG. It name describe what it doing: Object Transfer Service/Profile, so it is something like OPP. While we are at this move some HID attributes to get right order in switch-case. Change-Id: I460963a422c7292b2cabf7e88f32dbd6e8d7051f Reviewed-on: https://code.wireshark.org/review/13735 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2016-02-06Bluetooth: BTLE: Fix Random Rx/Tx BdAddr bit presentationMichal Labedzki1-3/+3
Random Rx/Tx bit are properly decoded but incorrect present in parent tree. Change-Id: I4c31d8e77b1adb5f821da6074bde5dff400d6c04 Reviewed-on: https://code.wireshark.org/review/13738 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2016-02-06Bluetooth: HCI: Vendor: Add expert info if not all is dissectedMichal Labedzki2-9/+35
If vendor dissector does not decode all bytes - it seems to be unexpected parameter. Also move vendor dissection tree under root as it is done for HCI CMD. >>> CID 1247678: Error handling issues (CHECKED_RETURN) >>> No check of the return value of "dissector_try_uint_new(hci_vendor_table, hci_vendor_data->manufacturer, tvb, pinfo, tree, 1, bluetooth_data)". >>> CID 1247679: Error handling issues (CHECKED_RETURN) >>> No check of the return value of "dissector_try_uint_new(hci_vendor_table, hci_vendor_data->manufacturer, tvb, pinfo, main_tree, 1, bluetooth_data)". Change-Id: Icdb8c1f166d5bc33cfc79c62d384ae416dfbf0cf Reviewed-on: https://code.wireshark.org/review/13737 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2016-02-06Bluetooth/6lowpan: Add missing handle for L2CAP CIDMichal Labedzki1-0/+1
Change-Id: I0a9e405069250cf37e38a0e4b1e9645997f80409 Reviewed-on: https://code.wireshark.org/review/13736 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michal Labedzki <michal.labedzki@tieto.com>
2016-02-06CMake: Avoid unnecessary link dependency with SBC and fix debian warningsJoão Valverde2-1/+2
Fixes dpkg warnings: dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/wireshark-qt/usr/bin/wireshark was not linked against libsbc.so.1 (it uses none of the library's symbols) dpkg-shlibdeps: warning: package could avoid a useless dependency if debian/wireshark-gtk/usr/bin/wireshark-gtk was not linked against libsbc.so.1 (it uses none of the library's symbols) dpkg-shlibdeps: warning: symbol sbc_decode used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries dpkg-shlibdeps: warning: symbol sbc_init used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries dpkg-shlibdeps: warning: symbol sin used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries dpkg-shlibdeps: warning: symbol floorf used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries dpkg-shlibdeps: warning: symbol floor used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries dpkg-shlibdeps: warning: symbol sbc_finish used by debian/libwscodecs0/usr/lib/x86_64-linux-gnu/libwscodecs.so.0.0.0 found in none of the libraries Change-Id: I71911513c348edd336cdc82ea358b6a05760b4b9 Reviewed-on: https://code.wireshark.org/review/13784 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Balint Reczey <balint@balintreczey.hu>
2016-02-06Add ARP protocol preference to register MAC to IP bindings for name resolutionJoão Valverde1-2/+14
Applies to the resolve Physical Addresses to names preference. Change-Id: Ib1f484afc940eb6a022e03a1766c18449b2dfed3 Reviewed-on: https://code.wireshark.org/review/13400 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt>
2016-02-06Check whether mcs_index is sane, not just whether it's set.Guy Harris1-1/+3
Don't just check whether we *have* the MCS index, check whether it's a valid MCS index, before we use it in calculations. Otherwise, we'll make out-of-bounds array accesses. (May or may not fix bug 12085, so just Ping-Bug for now. It's necessary in any case.) Change-Id: I7119366397b260089aea35ae9fcd5ad9ec6b06f2 Ping-Bug: 12085 Reviewed-on: https://code.wireshark.org/review/13790 Reviewed-by: Guy Harris <guy@alum.mit.edu>
2016-02-06GlusterFS: Add support for the SEEK procedureNiels de Vos2-0/+52
Gluster added support for a SEEK operation, supporting SEEK_DATA and SEEK_HOLE. The actual protocol modifications can be found in commit 9b71092f3 (http://review.gluster.org/11482). Bug:12088 Change-Id: I298b4a5023fa748e9c443ae5a24a1b58d76a5453 Signed-off-by: Niels de Vos <ndevos@redhat.com> Reviewed-on: https://code.wireshark.org/review/13780 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2016-02-06CMake: Minor ConfigureChecks.cmake cleanupJoão Valverde1-4/+2
Change-Id: Ib58aaf218bf1440e6c02665c8274fe2bee73978f Reviewed-on: https://code.wireshark.org/review/13783 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
2016-02-06Remove unbalanced comment markerJoão Valverde1-1/+1
Change-Id: I21faa3f454974ee4a167ff8709a8f7ecead8a7cd Reviewed-on: https://code.wireshark.org/review/13785 Reviewed-by: João Valverde <j@v6e.pt>
2016-02-05couchbase: Fix Dead Store (Dead assignement/Dead increment) Warning found by ↵Alexis La Goutte1-2/+2
Clang Change-Id: Ifee6ba30761cd22a7437f099cbef3764759fe985 Reviewed-on: https://code.wireshark.org/review/13724 Reviewed-by: Dave Rigby <daver@couchbase.com> Reviewed-by: Dario Lombardo <lomato@gmail.com> Petri-Dish: Dario Lombardo <lomato@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05RTSE and PRES relies on the TCP conversation to do reassembly, switch toAndersBroman4-15/+41
the new interface. Change-Id: I4f818d55416d3b1d09b46015d83f3acc5a9e71cc Reviewed-on: https://code.wireshark.org/review/13744 Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org> Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05CAPWAP: fix cannot optimize loop, the loop counter may overflow ↵Alexis La Goutte1-9/+7
[-Wunsafe-loop-optimizations] Change-Id: I99c556950007957c09809dc477a94d410cca4cc8 Reviewed-on: https://code.wireshark.org/review/13728 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: João Valverde <j@v6e.pt> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05pktc: don't THROW() an exception from a dissectorMartin Kaiser1-6/+0
in the switch-case statements, we already show an expert info it should be sufficient to exit and report to the caller how many bytes we dissected as for the string, we can just calculate the length and let proto_tree_add_item() throw an exception if that length is invalid Change-Id: I310a4011cb112f3ed70e804c5b44d58f275fab6b Reviewed-on: https://code.wireshark.org/review/13745 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-05ENIP: Add dissection of UCMM CPF Item over UDPMichael Mann3-2/+76
This includes request/response tracking Change-Id: I12ac4c198929aa6a75f3f839f9ee52ebf00b8059 Reviewed-on: https://code.wireshark.org/review/13743 Petri-Dish: Michael Mann <mmann78@netscape.net> Reviewed-by: D. Ulis <daulis0@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05Fix some Coverity issueMichal Labedzki4-53/+29
>>> CID 1349726: Incorrect expression (COPY_PASTE_ERROR) >>> "tvb_reported_length_remaining" in "tvb_reported_length_remaining(tvb, offset)" looks like a copy-paste error. >>> CID 1349727: Incorrect expression (COPY_PASTE_ERROR) >>> "tree" in "dissector_try_uint_new(bluetooth_eir_ad_manufacturer_company_id, company_id, new_tvb, pinfo, >>> CID 1349728: (DEADCODE) >>> Execution cannot reach this expression "0" inside statement "(0 && ((__s1_len = __builti...". >>> CID 1306904: Logically dead code (DEADCODE) >>> Execution cannot reach this statement "tap_hci_summary->interface_...". Change-Id: I5f99e37e6d5fac39da7082649caa8373d06bb808 Reviewed-on: https://code.wireshark.org/review/13739 Petri-Dish: Michal Labedzki <michal.labedzki@tieto.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05Add CIP Security objectsMichael Mann3-80/+654
These objects are defined in Volume 8, but it doesn't make sense to create a new dissector file for them, so just distribute them where it makes the most sense in the existing CIP dissectors. Also do some conversions to proto_tree_add_bitmask while in the neighborhood. Add support for EtherNet/IP over DTLS/TLS. Change-Id: I4e658e8871eebb222816229de7594ff766264308 Reviewed-on: https://code.wireshark.org/review/13710 Reviewed-by: D. Ulis <daulis0@gmail.com> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05couchbase: Use some unused hf (and remove unused hf...)Alexis La Goutte1-4/+2
Change-Id: I767a334c82c27b06be7e72461b7f3e3d961784b4 Reviewed-on: https://code.wireshark.org/review/13725 Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-by: Dave Rigby <daver@couchbase.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-05Multipath: use directly value of sizeof(guint32)Alexis La Goutte1-2/+2
it will be also make happy OS X x64 buildbot Change-Id: Ib718d717719739314170632f04b3ec68c2917ed6 Reviewed-on: https://code.wireshark.org/review/13730 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-05iso14443: add items for the actual length fieldsMartin Kaiser1-5/+64
the messages contain "length codes" instead of the actual lengths use a simple conversion table to covert length codes into lengths add generated items for the actual lengths Change-Id: Ic10aed0d20cfca30524cf767798df4eec2330592 Reviewed-on: https://code.wireshark.org/review/13734 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-05iso14443: move the dissection of the attrib command to a separate functionMartin Kaiser1-55/+68
Change-Id: I929a9e5ce22d87a169d94c5ff660f48512cb5134 Reviewed-on: https://code.wireshark.org/review/13733 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-05iso14443: make max_frame_size_code more genericMartin Kaiser1-7/+21
it's used by two messages, the relative position inside the current byte is different don't use a static bitmask for the hf Change-Id: I6a145cad46bab1afd22f66f144e7e4e9909f0b15 Reviewed-on: https://code.wireshark.org/review/13732 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-05iso14443: dissect some more fieldsMartin Kaiser1-3/+45
Change-Id: Ic1f20b94d9930150b60af4802bf8653713562a33 Reviewed-on: https://code.wireshark.org/review/13731 Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
2016-02-05SLL: add SLL protocol type to 'Decode As' dialogPascal Quantin1-1/+23
Bug: 12081 Change-Id: I2f2e2f85684f711c500bce83ce7a6d11676eabda Reviewed-on: https://code.wireshark.org/review/13713 Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2016-02-05Don’t adjust time column widths in cf_openStig Bjørlykke1-3/+0
The time column widths should not be adjusted in cf_open() because we don’t have any packets yet and Qt resizeColumnToContents() will not adjust any widths but emits a sectionResized() with invalid or default values (new_width seems to always be 32). This will in some cases (when start capturing packets) give wrong width values which is later stored in the recent file, and the time columns may end up narrow the next time the recent file is read. This fix is related to the column with issues previously compensated for in PacketList::sectionResized() (g4980d505). Change-Id: Id3b49069fe5d2b55d608cc7a6d32fe7851369bf9 Reviewed-on: https://code.wireshark.org/review/13712 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-05Qt: Add check for field extractorsStig Bjørlykke9-7/+44
The proto tree is needed in several cases when using Lua field extractors, because they fetch values from the tree. Without a valid field extractor a Lua plugin may misbehave and display wrong column info. This fixes column issues when: - Calling resetColumns() in Qt. This involves adding a display filter, change time display format, change name resolution and other changes in UI which requires column updates. - Print summary lines. - Export as CSV and PSML. Change-Id: Ieed6f8578cdf2759f1f836cd8413a4529b7bbd80 Reviewed-on: https://code.wireshark.org/review/13708 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-05Qt: Reset columns before auto resize time columnsStig Bjørlykke1-9/+11
When changing timestamp format, timestamp precision and display seconds with hours and minutes we must reset columns before auto resizing the time columns to get the size of the new column values. Without this we will resize to the length of the preference we are changing from, which is not what we want. Change-Id: If7081bf0b9b6f6974232cea0b3fe0186c904f2a2 Reviewed-on: https://code.wireshark.org/review/13711 Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-05Network Service Header Dissector for Ethernet & GRE encapsulationChidambaram Arunachalam6-0/+390
draft-ietf-sfc-nsh-01.txt Bug: 11490 Change-Id: I95adb1e0b1e42ba8c75e82145a756e2836a9a989 Reviewed-on: https://code.wireshark.org/review/13633 Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-05fuzz-test: Move the ASan message outside our loops.Gerald Combs1-3/+7
Change-Id: I109a3e222358523de8f43c887bf7551e92fce135 Reviewed-on: https://code.wireshark.org/review/13714 Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-04RTPS: Improve Status Info dissection.Juanjo Martin1-46/+89
Change-Id: Ia645e9a1bb61d9e4bf6a36ccbe83c54e7bfa0dd9 Reviewed-on: https://code.wireshark.org/review/12782 Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
2016-02-04Have CIP dissectors find the CIP Class dissector table, rather than have a ↵Michael Mann4-2/+5
externed global variable. Change-Id: Id8156680d67d65d87c156df05e8a66e2531728d2 Reviewed-on: https://code.wireshark.org/review/13709 Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
2016-02-04Multipart: add dissection of multipart/encrypted and also fix problems to ↵DiablosOffens1-38/+197
find the right boundaries Bug: 11978 Change-Id: Ia31e1b451bfae268e8ede84bddffd5dae8d97d8e Signed-off-by: Alexis La Goutte <alexis.lagoutte@gmail.com> Reviewed-on: https://code.wireshark.org/review/12281 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
2016-02-04Fix asn2wrs warnings [-Wredundant-decls]João Valverde9-15/+16
Change-Id: Iced801f17c56618eaaf1bb7c85d68ecdebd633a1 Reviewed-on: https://code.wireshark.org/review/13705 Petri-Dish: João Valverde <j@v6e.pt> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-03 05:24:45 +0000