vsomeip 3.3.5.1 (#454)

Notes:
v3.3.5.1
- Fix typo in application_impl.cpp
- Update load_balancing_option_impl.cpp
- Fix format specifier in memory_log_timer_cbk
- Isolate boost
- Remove redundant ostream manipulators
- Fix for configuration option deserialize bug
- Accept return codes within range 0x20 - 0x5E as valid

v3.3.5
- Do NOT use iterators in asynchronous operations.
- Add event identifier to "REGISTER EVENT" log message at routing manager stub.
- Fix for crash in flush function, server_endpoint_impl class
- added FD_CLOEXEC flag in all open operations
- Use "make_unique" and "make_shared" to construct "unique_ptr" and "shared_ptr"
- Removed redundant pairs of parentheses
- Do not erase while iterating.
- Changed invalid call of deserialize to serialize
- Fixed the the problem in policy::deserialize counts down the given policy size,
and after deserializing the policy it is 0.
- Monitor session IDs in SD messages and log missing IDs
- Fix timestamp format of log message
- Prevent crash when logging with DLT
- load_security_update_whitelist Unit and Benchmark Tests
- Added protection when a message is destroyed
- Pu/connect timeout refactor
- Rework the addition of services when in suspended mode
- Remove const qualifier from function return types
- Adapt to boost 1.81.0

Author: DiogoPedrozza

Android.bp

@@ -55,7 +55,7 @@ cc_defaults {
 
 cc_defaults {
     name: "vsomeip_lib_defaults",
-    
+
     cflags: [
         "-DVSOMEIP_BOOST_VERSION=107100",
         "-DVSOMEIP_INTERNAL_SUPPRESS_DEPRECATED",
@@ -80,14 +80,15 @@ cc_library_shared {
 
     cflags: [
         "-DWITHOUT_SYSTEMD",
-        "-DVSOMEIP_COMPAT_VERSION=\"3.3.0\"",
+        "-DVSOMEIP_COMPAT_VERSION=\"3.3.5\"",
         "-DVSOMEIP_BASE_PATH=\"/vendor/run/someip/\"",
         "-DUSE_DLT",
     ],
 
     ldflags: [
         "-Wl,-wrap,socket",
-        "-Wl,-wrap,accept"
+        "-Wl,-wrap,accept",
+        "-Wl,-wrap,open"
     ],
 
     rtti: true,

Android.mk

@@ -100,7 +100,7 @@ LOCAL_CFLAGS :=  \
     -frtti \
     -fexceptions \
     -DWITHOUT_SYSTEMD \
-    -DVSOMEIP_VERSION=\"3.3.0\" \
+    -DVSOMEIP_VERSION=\"3.3.5\" \
     -DVSOMEIP_BASE_PATH=\"/vendor/run/someip/\" \
     -Wno-unused-parameter \
     -Wno-non-virtual-dtor \
@@ -147,7 +147,7 @@ LOCAL_CFLAGS := \
     -frtti \
     -fexceptions \
     -DWITHOUT_SYSTEMD \
-    -DVSOMEIP_VERSION=\"3.3.0\" \
+    -DVSOMEIP_VERSION=\"3.3.5\" \
     -DVSOMEIP_BASE_PATH=\"/vendor/run/someip/\" \
     -Wno-unused-parameter \
     -Wno-non-virtual-dtor \
@@ -194,8 +194,8 @@ LOCAL_CFLAGS :=  \
     -frtti \
     -fexceptions \
     -DWITHOUT_SYSTEMD \
-    -DVSOMEIP_VERSION=\"3.3.0\" \
-    -DVSOMEIP_COMPAT_VERSION=\"3.3.0\" \
+    -DVSOMEIP_VERSION=\"3.3.5\" \
+    -DVSOMEIP_COMPAT_VERSION=\"3.3.5\" \
     -DVSOMEIP_BASE_PATH=\"/vendor/run/someip/\" \
     -Wno-unused-parameter \
     -Wno-non-virtual-dtor \
@@ -210,5 +210,7 @@ LOCAL_CFLAGS :=  \
     -Wno-header-guard \
     -Wno-overloaded-virtual \
     -Wl,-wrap,socket \
+    -Wl,-wrap,accept \
+    -Wl,-wrap,open \
 
-include $(BUILD_SHARED_LIBRARY)
+include $(BUILD_SHARED_LIBRARY)

CHANGES

@@ -1,7 +1,22 @@
 Changes
 =======
 
-v3.3.0
+v3.3.5.1
+- Fix typo in application_impl.cpp
+- Update load_balancing_option_impl.cpp
+- Fix format specifier in memory_log_timer_cbk
+- Isolate boost
+- Remove redundant ostream manipulators
+- Fix for configuration option deserialize bug
+- Accept return codes within range 0x20 - 0x5E as valid
+
+v3.3.5
+- Do NOT use iterators in asynchronous operations.
+- Add event identifier to "REGISTER EVENT" log message at routing manager stub.
+- Fix for crash in flush function, server_endpoint_impl class
+- added FD_CLOEXEC flag in all open operations
+- Use "make_unique" and "make_shared" to construct "unique_ptr" and "shared_ptr"
+- Removed redundant pairs of parentheses
 - Do not erase while iterating.
 - Changed invalid call of deserialize to serialize
 - Fixed the the problem in policy::deserialize counts down the given policy size,
@@ -10,10 +25,13 @@ and after deserializing the policy it is 0.
 - Fix timestamp format of log message
 - Prevent crash when logging with DLT
 - load_security_update_whitelist Unit and Benchmark Tests
-- Added protection when a message is destroyed.
+- Added protection when a message is destroyed
+- Pu/connect timeout refactor
 - Rework the addition of services when in suspended mode
 - Remove const qualifier from function return types
 - Adapt to boost 1.81.0
+
+v3.3.4
 - Removed VSOMEIP_DEPRECATED_UID_GID from some elements.
 - Add nullptr guards to receive_cbk.
 - Add nullptr check to receive_cbk.
@@ -28,10 +46,14 @@ and after deserializing the policy it is 0.
 - Reduce log level of some vsomeip messages.
 - Change dependency on LINK UP.
 - Change rejoining mechanism in UDP server endpoint.
+
+v3.3.3
 - Fixed android build issues
 - Added more DLT log info on credentials error
 - Fixed ICON register issues.
 - Added an  unlocked method to get the policy extension path
+
+v3.3.2
 - Manage E2E Profile04 counters per instance
 - Fix handling of subscription objects during unsubscribe.
 - Remove subscription on connection resets.
@@ -48,6 +70,8 @@ and after deserializing the policy it is 0.
 - Handling with some udp errors in send_cbk process.
 - Avoid services becoming available when the daemon is suspended.
 - refactor: remove `set_bound_sec_client()` from `local_tcp_server_endpoint_impl`.
+
+v3.3.1
 - Refactor `plugin_manager_impl::load_symbol()`
 - Use consistent naming for `security::is_client_allowed_to_access()`
 - Raise C++ standard used to C++17 on non-Windows platforms
@@ -65,6 +89,8 @@ and after deserializing the policy it is 0.
 - Make the sec client a member of the connection
 - Add support for TCP clients in the default security hooks
 - Fix security checks when using internal TCP sockets
+
+v3.3.0
 - Introduced protocol classes
 - Extended interface to allow application specific configurations
 - Allow to switch off internal routing

CMakeLists.txt

@@ -11,7 +11,7 @@ set (VSOMEIP_COMPAT_NAME vsomeip)
 
 set (VSOMEIP_MAJOR_VERSION 3)
 set (VSOMEIP_MINOR_VERSION 3)
-set (VSOMEIP_PATCH_VERSION 0)
+set (VSOMEIP_PATCH_VERSION 5)
 set (VSOMEIP_HOTFIX_VERSION 0)
 
 set (VSOMEIP_VERSION ${VSOMEIP_MAJOR_VERSION}.${VSOMEIP_MINOR_VERSION}.${VSOMEIP_PATCH_VERSION})
@@ -288,7 +288,7 @@ set_target_properties (${VSOMEIP_NAME} PROPERTIES VERSION ${VSOMEIP_VERSION} SOV
 if (MSVC)
     set_target_properties(${VSOMEIP_NAME} PROPERTIES COMPILE_DEFINITIONS "VSOMEIP_DLL_COMPILATION")
 else ()
-    set_target_properties(${VSOMEIP_NAME} PROPERTIES LINK_FLAGS "-Wl,-wrap,socket -Wl,-wrap,accept")
+    set_target_properties(${VSOMEIP_NAME} PROPERTIES LINK_FLAGS "-Wl,-wrap,socket -Wl,-wrap,accept -Wl,-wrap,open")
 endif ()
 target_include_directories(${VSOMEIP_NAME} INTERFACE
     $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/interface>

implementation/configuration/include/internal_android.hpp

@@ -119,14 +119,14 @@ typedef enum {
     IS_SUBSCRIBING
 } subscription_state_e;
 
-const std::uint32_t MESSAGE_SIZE_UNLIMITED = (std::numeric_limits<std::uint32_t>::max)();
+const std::uint32_t MESSAGE_SIZE_UNLIMITED = std::numeric_limits<std::uint32_t>::max();
 
-const std::uint32_t QUEUE_SIZE_UNLIMITED = (std::numeric_limits<std::uint32_t>::max)();
+const std::uint32_t QUEUE_SIZE_UNLIMITED = std::numeric_limits<std::uint32_t>::max();
 
 #define VSOMEIP_DEFAULT_NPDU_DEBOUNCING_NANO         2 * 1000 * 1000
 #define VSOMEIP_DEFAULT_NPDU_MAXIMUM_RETENTION_NANO  5 * 1000 * 1000
 
-const std::uint32_t MAX_RECONNECTS_UNLIMITED = (std::numeric_limits<std::uint32_t>::max)();
+const std::uint32_t MAX_RECONNECTS_UNLIMITED = std::numeric_limits<std::uint32_t>::max();
 
 const std::uint32_t ANY_UID = 0xFFFFFFFF;
 const std::uint32_t ANY_GID = 0xFFFFFFFF;

implementation/configuration/include/routing.hpp

@@ -32,7 +32,7 @@ struct routing_host_t {
         unicast_ = _other.unicast_;
         port_ = _other.port_;
 
-        return (*this);
+        return *this;
     }
 };
 
@@ -44,7 +44,7 @@ struct routing_guests_t {
         unicast_ = _other.unicast_;
         ports_ = _other.ports_;
 
-        return (*this);
+        return *this;
     }
 };
 
@@ -61,7 +61,7 @@ struct routing_t {
         host_ = _other.host_;
         guests_ = _other.guests_;
 
-        return (*this);
+        return *this;
     }
 };