Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 5929] New protocol dissector for "CIP Motion"

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5929

--- Comment #23 from Bill Meier <wmeier@xxxxxxxxxxx> 2011-10-12 21:34:51 EDT ---
Additional compiler warnings fixed:

packet-cipmotion.c: In function 'dissect_cipmotion':
packet-cipmotion.c:1795:16: warning: variable 'seq_number' set but not used
[-Wunused-but-set-variable]

>> I commented out the 'seq_number' defintion and usage.

-------------
packet-cipmotion.c: At top level:
packet-cipmotion.c:60:12: error: 'hf_cip_time_data_update' defined but not used
[-Werror=unused-variable]
cc1: all warnings being treated as errors

>> I deleted 'hf_cip_time_data_update' definition.

--------------
packet-cipmotion.c: In function 'dissect_cipmotion':
packet-cipmotion.c:1836: warning: 'inst_count' may be used uninitialized in
this function

>> (Not actually valid); I initialized 'inst_count=0' to prevent the warning.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.