ASNMP: serialize the pdu is failed (long type is 8 bytes on 64 bits machine)

[Maayan311]

### Version ACE VERSION: 6.5.5

### Host machine and operating system centOS Linux 7 (vmware)

### Target machine and operating system (if different from host) centOS Linux 7 (vmware)

### Compiler name and version (including patch level) gcc-8.3.0

### The $ACE_ROOT/ace/config.h file #include "ace/config-linux.h" #if 1 // WATERFALL UNICODE SUPPORT #define ACE_HAS_STANDARD_CPP_LIBRARY 1 #define ACE_HAS_WCHAR #define ACE_USES_WCHAR #define ACE_HAS_ICONV #define ACE_HAS_IPV6 #endif

### The $ACE_ROOT/include/makeinclude/platform_macros.GNU file include $(ACE_ROOT)/include/makeinclude/platform_linux.GNU no_hidden_visibility = 1 CCFLAGS:=$(filter-out -fvisibility=hidden,$(CCFLAGS)) CCFLAGS:=$(filter-out -fvisibility-inlines-hidden $(CCFLAGS)) CCFLAGS:= -I/usr/libiconv/include $(CCFLAGS) LDFLAGS += -L/usr/libiconv/lib64 -liconv

### Contents of $ACE_ROOT/bin/MakeProjectCreator/config/default.features // The global features file contains the default feature settings. // By default, if a feature isn't listed here then it is enabled. // If it is listed, then the value assigned to the feature is interpreted // as a boolean value. You really shouldn't have to edit this file except // to change the defaults. If you want to override these values you can do // one of two things. // // 1) Create a default.features in this directory with the features you // want enabled or disabled. // 2) Create a feature file anywhere you like with the features you want and // use the -feature_file option to specify where it is located. // // Feature definitions from this file are always overwritten by -features // command option of mwc.pl. // // This file is somewhat akin to the platform_macros.GNU file in that build // settings can be set here.

// Enable these by default. acexml = 1 ace_svcconf = 1 ace_token = 1 ace_idl_dependencies = 1

// By default we disable these. ace_for_tao = 0 cross_compile = 0 ssl = 1 openssl11 = 1 qos = 0 rapi = 0 repo = 0 minimum_corba = 0 java = 0 zlib = 0 zzip = 0 bzip2 = 0 lzo1 = 0 lzo2 = 0 ipv6 = 0 mfc = 0 rpc = 0 sctp = 0 boost = 0 dummy_label = 0 wince = 0 uses_wchar = 0 // i.e. unicode gen_ostream = 0 ziparchive = 0 negotiate_codesets = 0 notify_monitor_control = 0 versioned_namespace = 0 vcl = 0 stl = 0 xsc = 0 tao_idl_fe_gen = 0 ace_svcconf_gen = 0 xerces = 0 xerces2 = 0 xerces3 = 0 ofccm = 0 ndds = 0 nddsmonitor = 0 mcpp = 0 wireshark_any = 0 wireshark = 0 wireshark_cmake = 0 old_stdstream = 0 exclude_inet = 0 inet_ssl = 0 coredx = 0 dds4ccm_opendds = 0 dds4ccm_ndds = 0 dds4ccm_splice = 0 dds4ccm_coredx = 0 openmp = 0

// Features for various GUI libraries

// Some of features are switched on because they are also controlled // by higher level features eg. gl is 1 because it is necessary for fl, // but if someone turns on fl, we may assume gl is available. This way makefile // generation is simplified. Moreover all internal ACE features like ace_qtreactor // and tao_qtresource are also turn on, because we want to build every component // related with if qt is provided. It is proposed to not remove // switched on features from global.features just because features are available // by default. This file is also a kind of ACE capabilities documentation, hence // it is better to enumerate here all features recognized by ACE.

// 1 - wxWindows (wxWidgets) libraries are available wxWindows = 0 // 1 - Motif (Lestif) libraries are available motif = 0 // 1 - Athena widgets libraries are available athena = 0 // 1 - X11 libraries are available x11 = 1 // 1 - X Toolkit libraries are available xt = 0 // 1 - Tk libraries are available tk = 0 // 1 - Fast Light toolkit libraries are available (assume lowercase h in header files) fl = 0 // 1 - Qt3 libraries are available qt = 0 qt3 = 0 // 1 - Qt4 core libraries are available qt4 = 0 // Qt5 qt5 = 0 // 1 - Fox libraries are available fox = 0 // 1 - OpenGL libraries are available gl = 1

// Features for ACE support for various GUI libraries // Use QtReactor ace_qtreactor = 1 // Use XtReactor ace_xtreactor = 1 // Use TkReactor ace_tkreactor = 1 // Use FlReactor ace_flreactor = 1 // Use FoxReactor ace_foxreactor = 1

// Features for TAO support for various GUI libraries // Use QtResources for TAO tao_qtresource = 1 // Use XtResources for TAO tao_xtresource = 1 // Use TkResources for TAO tao_tkresource = 1 // Use FlResources for TAO tao_flresource = 1 // Use FoxResources for TAO tao_foxresource = 1

//Use the ZIP_Wrapper feature for when using the ZIP wrapper classes ZIP_Wrapper = 0

// Feature for CORBA subsets corba_e_compact = 0 corba_e_micro = 0

// Footprint feature optimize_collocated_invocations = 1

// Option to disable iiop tao_no_iiop = 0

// Option to enable the linking of the TAO_CodecFactory library // whenever TAO is linked in link_codecfactory = 0

// Feature for LwCCM ccm_lw = 1 // Feature for CCM for disabling events. ccm_noevent = 0 // Feature for direct collocation ccm_direct_collocation = 0

valuetype_out_indirection=1

### AREA/CLASS/EXAMPLE AFFECTED: test that failed: ACE_warppers/ASNMP/examples/trap/trap.cpp

### The problem effects: The problem affects the execution After compiling trap.cpp, when the execute file is running - the problem affects building the asnmp packet correctly

### Synopsis On Wireshark - Cannot determine the protocol packet as asnmp in 64-bits machine

### Description On asn1.cpp the function: cmu_snmp::build, which serialize the pdu, fails on 64-bits machine. The reason for that is in asn1.cpp: build_int() function 'long' type is 8 bytes (the problem got fixed after changing from long to int - 4 bytes)

### Repeat by To get the error I compiled trap.cpp (test example) and run it on 64-bits machine

[jwillemsen]

Please open a pull request with a test extension and fix for the issue you found. If you need help, consider hiring someone to assist you with this. See https://www.dre.vanderbilt.edu/~schmidt/commercial-support.html for an overview of the commercial support companies including Remedy IT, the company I work for.

[gadh]

issue was fixed in pull request but not reviewed !

[jwillemsen]

Review takes time and is based on availability or support, see https://www.dre.vanderbilt.edu/~schmidt/TAO-support.html