Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 8333] New: Corrections to Recent DNP3 Control Code Changes

Date: Thu, 14 Feb 2013 19:11:18 +0000
Bug ID 8333
Summary Corrections to Recent DNP3 Control Code Changes in Bug ID 7865
Classification Unclassified
Product Wireshark
Version 1.9.x (Experimental)
Hardware x86
OS Windows 7
Status UNCONFIRMED
Severity Normal
Priority Low
Component Dissection engine (libwireshark)
Assignee [email protected]
Reporter [email protected]

Created attachment 10011 [details]
DNP3 Patch file against rev 56985

Build Information:
Version 1.9.0-SELFM (SVN Rev Unknown from unknown)

Copyright 1998-2013 Gerald Combs <[email protected]> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled (32-bit) with GTK+ 2.24.14, with Cairo 1.10.2, with Pango 1.30.1, with
GLib 2.34.1, with WinPcap (4_1_2), with libz 1.2.5, without POSIX capabilities,
without libnl, with SMI 0.4.8, with c-ares 1.7.1, with Lua 5.1, without Python,
with GnuTLS 2.12.18, with Gcrypt 1.4.6, with MIT Kerberos, with GeoIP, with
PortAudio V19-devel (built Jan  7 2013), with AirPcap.

Running on 32-bit Windows 7 Service Pack 1, build 7601, with WinPcap version
4.1.2 (packet.dll version 4.1.0.2001), based on libpcap version 1.0 branch
1_0_rel0b (20091008), GnuTLS 2.12.18, Gcrypt 1.4.6, without AirPcap.

Built using Microsoft Visual C++ 9.0 build 30729

Wireshark is Open Source Software released under the GNU General Public
License.

Check the man page and http://www.wireshark.org for more information.
--
Changes to the packet-dnp3 Control Code Dissection broke proper dissection of
additional control code bits (Queue/Clear and Qualifier Fields) as well as
dissection to remaining 10 bytes.  The main issue was that the "Control Status"
byte was relocated to before the Count,On-Time and Off-Time fields - not
correct as per the DNP3 specs.

I've attached a patch file (against rev 46985, current enough for DNP3) to
correct the issue.  

I've also attached a sample pcap file that clearly illustrates the issue; the
capture file must be configured using the DLT_USER dissector to skip the first
12 bytes as 'data' and process the remaining payload as 'dnp3.udp'.

Thanks, Chris


You are receiving this mail because:
  • You are watching all bug changes.