Wireshark · Wireshark-commits: [Wireshark-commits] master 020e25b: packet-gtp.c: remove unneeded named structures

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=020e25b660250c97c8085e64cf85074ccfcdd06b
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

020e25b by Fabrice Fontaine (fontaine.fabrice@xxxxxxxxx):

    packet-gtp.c: remove unneeded named structures
    
    In uclibc, _header is already in sigcontext.h:
    
    packet-gtp.c:2361:16: error: redefinition of 'struct _header'
     typedef struct _header {
                    ^~~~~~~
    In file included from /home/dawncrow/buildroot-test/scripts/instance-0/output/host/i686-buildroot-linux-uclibc/sysroot/usr/include/bits/sigcontext.h:30:0,
                     from /home/dawncrow/buildroot-test/scripts/instance-0/output/host/i686-buildroot-linux-uclibc/sysroot/usr/include/signal.h:311,
                     from /home/dawncrow/buildroot-test/scripts/instance-0/output/host/i686-buildroot-linux-uclibc/sysroot/usr/include/glib-2.0/glib/gbacktrace.h:36,
                     from /home/dawncrow/buildroot-test/scripts/instance-0/output/host/i686-buildroot-linux-uclibc/sysroot/usr/include/glib-2.0/glib.h:34,
                     from ../../epan/proto.h:28,
                     from ../../epan/packet.h:14,
                     from packet-gtp.c:43:
    /home/dawncrow/buildroot-test/scripts/instance-0/output/host/i686-buildroot-linux-uclibc/sysroot/usr/include/asm/sigcontext.h:173:8: note: originally defined here
     struct _header {
    
    To fix this issue, transform _header and other named structures (with
    the exception of gtp_conv_info_t) into unnamed structures
    
    Fixes:
     - http://autobuild.buildroot.org/results/c41d42fe3489bc63c42e7ce7a9eccb1b4ca7b9b2
    
    Change-Id: I78116233c2a8dd7c54723b7cb558254bd5143bd2
    Signed-off-by: Fabrice Fontaine <fontaine.fabrice@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/32335
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  5220367   couchbase: check shift value before use.
     add  020e25b   packet-gtp.c: remove unneeded named structures


Summary of changes:
 epan/dissectors/packet-gtp.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)