Hi Daniel,
thank you for your contribution!
The "SDO over UDP" and the improvement of the "Info column" were both on
my ToDo-list for a long time ;-)
Based on your improvements I would propose the following additional
changes (see attached patch) to the info column:
- don't show EPL src- and dst-address of SoC frame (same as SoA)
- show SoA requested service only if it's not "NO_SERVICE"
- NMT state in StatusResponse in words, not numbers
- don't show MC and PS flags in SoC (it's now configurable via
"Preferences")
Furthermore I extended the value_string struct for the NMT-Command-IDs
(asnd_cid_vals). This change is used to fully decode the NMTRequest frames.
As we have now the support of "SDO over UDP" we should propably change
the display filter of the SDO fields from "epl.asnd.sdo." to a more
generic "epl.sdo.".
Best regards,
David
--
David BuechiZurich University of Applied Sciences Winterthur
Dipl. Ing. FH Institute of Embedded Systems InES
Realtime CommunicationTechnikumstrasse 9
Tel: +41 52 267 70 60P.O.Box 805
Fax: +41 52 268 70 60 CH-8401 Winterthur
Daniel Krüger schrieb:
> Hello,
>
> thanks to David Buechi for his EPL v2 dissector.
>
> I attached a patch to this dissector, which includes some corrections,
> updates and SDO by UDP support. I will upload a sample capture of SDO by
> UDP to the wiki. The patch is fuzzy tested against the current SVN rev
> under Linux and it builds also under MSVC2005. Maybe I will provide
> further updates in the future.
>
> I have a feature request: is it possible to add a shortcut for "Start
> capture". I start and stop capturing very often and a shortcut is very
> handy. I know that capturing must be configured at first.
>
> cu
> Daniel
Index: epan/dissectors/packet-epl.c
===
--- epan/dissectors/packet-epl.c(revision 20956)
+++ epan/dissectors/packet-epl.c(working copy)
@@ -1,14 +1,14 @@
/* packet-epl.c
- * Routines for "Ethernet Powerlink 2.0" dissection
+ * Routines for "Ethernet Powerlink 2.0" dissection
* (ETHERNET Powerlink V2.0 Communication Profile Specification Draft Standard
Version 1.0.0)
*
* Copyright (c) 2006: Zurich University of Applied Sciences Winterthur (ZHW)
* Institute of Embedded Systems (InES)
* http://ines.zhwin.ch
- *
- * - Dominic B'chaz
+ *
+ * - Dominic Bechaz
* - Damir Bursic
- * - David B_chi
+ * - David Buechi
*
* Copyright (c) 2007: SYS TEC electronic GmbH
* http://www.systec-electronic.com
@@ -45,6 +45,7 @@
#include
#include
#include
+#include
#include "packet-epl.h"
@@ -222,6 +223,10 @@
static gint ett_epl_el_entry_type = -1;
static gint ett_epl_sdo_entry_type = -1;
+/* preference whether or not display the SoC flags in info column */
+gboolean show_soc_flags = FALSE;
+
+
/* Define the tap for epl */
/*static gint epl_tap = -1;*/
@@ -287,7 +292,8 @@
switch (epl_mtyp)
{
case EPL_SOC:
-col_add_fstr(pinfo->cinfo, COL_INFO, "SoCsrc = %3d dst =
%3d ", epl_src, epl_dest);
+/* source and destination NodeID are fixed according to the
spec */
+col_add_str(pinfo->cinfo, COL_INFO, "SoC");
break;
case EPL_PREQ:
@@ -450,10 +456,12 @@
}
offset += 2;
-if (check_col(pinfo->cinfo, COL_INFO))
-{
-col_append_fstr(pinfo->cinfo, COL_INFO, "MC = %d PS = %d",
-((EPL_SOC_MC_MASK & flags) >> 7), ((EPL_SOC_PS_MASK &
flags) >> 6));
+if (show_soc_flags) {
+if (check_col(pinfo->cinfo, COL_INFO))
+{
+col_append_fstr(pinfo->cinfo, COL_INFO, "MC = %d PS = %d",
+((EPL_SOC_MC_MASK & flags) >> 7),
((EPL_SOC_PS_MASK & flags) >> 6));
+}
}
if (epl_tree)
@@ -634,10 +642,13 @@
}
offset += 1;
-if (check_col(pinfo->cinfo, COL_INFO))
+if (svid != EPL_SOA_NOSERVICE)
{
-col_append_fstr(pinfo->cinfo, COL_INFO, "tgt = %3d %s",
-target, match_strval(svid, soa_svid_vals));
+if (check_col(pinfo->cinfo, COL_INFO))
+{
+col_append_fstr(pinfo->cinfo, COL_INFO, "tgt = %3d %s",
+target, match_strval(svid, soa_svid_vals));
+}
}
if (epl_tree)
@@ -927,7 +938,7 @@
nmt_state = tvb_get_guint8(tvb, offset);
if (check_col(pinfo->cinfo, COL_INFO))
{
-col_append_fstr(pinfo->cinfo, COL_INFO, "NMTState = 0x%02X",
nmt_state);
+col_append_fstr(pinfo->cinfo, COL_INFO, "%s ",
match_strval(nmt_st
