[LU-10590] Building tshark lustre disectors fails with a compile error under : lustre_dlm_flags_wshark.c:333:21: error: expected '}' before numeric constant

• Type: Bug
• Resolution: Not a Bug
• Priority: Minor
• Affects Version/s: Lustre 2.10.2
• Labels: wireshark
• Severity: 3
• Assignee: Arshad Hussain
• Reporter: Arshad Hussain

Description

Building tshark lustre disectors fails with a compile error under : lustre_dlm_flags_wshark.c:333:21: error: expected '}' before numeric constant. As it is missing the comma ',' under lustre/contrib/wireshark/lustre_dlm_flags_wshark.c

Make  failure output.

[root@mrpel7 wireshark]# make
gcc -DINET6 -D_U_=__attribute__\(\(unused\)\) -Wall -Wpointer-arith -g -DXTHREADS -D_REENTRANT -DXUSE_MTSAFE_API -fPIC -DPIC -I/root/mrp/wireshark/wireshark-1.9.2 -I/root/mrp/lustre-xx-3.10/lustre-release/lustre/include -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include  -lglib-2.0   -DHAVE_CONFIG_H   -c -o packet-lustre.o packet-lustre.c
In file included from packet-lustre.c:11423:0:
lustre_dlm_flags_wshark.c: In function 'proto_register_dcerpc_lustre':
lustre_dlm_flags_wshark.c:333:21: error: expected '}' before numeric constant
       /* id      */ HFILL

Fix is to add missing comma. After after the fix it compiles successfully.

# tshark -r ~/truncate.pcap -R "lustre" -T fields -e col.Info
LDLM_ENQUEUE request [Concurrent Read][ intent : lookup ]
LDLM_ENQUEUE reply [Protected Read]
LDLM_ENQUEUE request [Concurrent Write][ intent : open create ][REINT_OPEN] filename :
LDLM_ENQUEUE reply [Concurrent Read]
LDLM_CANCEL request [MINMODE]
LDLM_CANCEL reply
MDS_REINT reply
OST_PUNCH request
OST_PUNCH reply
MDS_CLOSE request [REINT_SETATTR]
MDS_CLOSE reply

Pushing patch shortly.