Fix max lenght in UDP port string (#315)

* Fix max lenght in UDP port string

* Fix CI

Signed-off-by: Pablo Garrido <pablogs9@gmail.com>

* Update

* Linting

* Update

* Fix uncrustify

---------

Signed-off-by: Pablo Garrido <pablogs9@gmail.com>
(cherry picked from commit aa15666)

# Conflicts:
#	.github/workflows/ci.yml

Author: pablogs9

.github/workflows/ci.yml

@@ -23,15 +23,18 @@ jobs:
 
         - name: Download dependencies
           run: |
-            apt update && apt install -y python3-pip git python3-rosdep python3-colcon-common-extensions curl ros-$ROS_DISTRO-performance-test-fixture
+            apt update && apt install -y python3-pip git python3-rosdep python3-colcon-common-extensions curl ros-$ROS_DISTRO-performance-test-fixture gcovr
             git clone -b ros2 https://github.com/eProsima/Micro-CDR src/Micro-CDR
             git clone -b ros2 https://github.com/eProsima/Micro-XRCE-DDS-Client src/Micro-XRCE-DDS-Client
             git clone -b jazzy https://github.com/micro-ROS/rosidl_typesupport_microxrcedds src/rosidl_typesupport_microxrcedds
             git clone -b jazzy https://github.com/ros2/rmw src/rmw
             touch src/rosidl_typesupport_microxrcedds/test/COLCON_IGNORE
 
             # Install coverage tools
+<<<<<<< HEAD
             apt install -y gcovr
+=======
+>>>>>>> aa15666 (Fix max lenght in UDP port string (#315))
             . /opt/ros/$ROS_DISTRO/setup.sh
             rosdep init && rosdep update
             rosdep install --from-paths src -r
@@ -47,6 +50,18 @@ jobs:
             colcon test --event-handlers console_direct+ --packages-select=rmw_microxrcedds --return-code-on-test-failure
             ./build/rmw_microxrcedds/test/test-sizes 2> memanalisys_out
 
+<<<<<<< HEAD
+=======
+        # - name: Static memory
+        #   continue-on-error: true
+        #   if: github.event_name == 'pull_request'
+        #   uses: machine-learning-apps/pr-comment@master
+        #   env:
+        #     GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        #   with:
+        #     path: memanalisys_out
+
+>>>>>>> aa15666 (Fix max lenght in UDP port string (#315))
         - name: Coverage
           run: |
             cd src/rmw-microxrcedds/

rmw_microxrcedds_c/include/rmw_microros/rmw_microros.h

@@ -49,7 +49,7 @@ extern "C"
 #elif defined(RMW_UXRCE_TRANSPORT_IPV6)
   #define MAX_IP_LEN         39
 #endif  // ifdef RMW_UXRCE_TRANSPORT_IPV4
-#define MAX_PORT_LEN         5
+#define MAX_PORT_LEN         6  // uint16_t max size + NULL-end string
 #define MAX_SERIAL_DEVICE    50
 
 typedef struct rmw_uxrce_transport_params_t

rmw_microxrcedds_c/src/rmw_init.c

@@ -92,7 +92,7 @@ rmw_init_options_init(
     return RMW_RET_INVALID_ARGUMENT;
   }
 
-  if (strlen(RMW_UXRCE_DEFAULT_PORT) <= MAX_PORT_LEN) {
+  if (strlen(RMW_UXRCE_DEFAULT_PORT) < MAX_PORT_LEN) {
     snprintf(
       init_options->impl->transport_params.agent_port,
       MAX_PORT_LEN, "%s", RMW_UXRCE_DEFAULT_PORT);

rmw_microxrcedds_c/src/rmw_microros/init_options.c

@@ -97,7 +97,7 @@ rmw_ret_t rmw_uros_options_set_udp_address(
     return RMW_RET_INVALID_ARGUMENT;
   }
 
-  if (port != NULL && strlen(port) <= MAX_PORT_LEN) {
+  if (port != NULL && strlen(port) < MAX_PORT_LEN) {
     snprintf(rmw_options->impl->transport_params.agent_port, MAX_PORT_LEN, "%s", port);
   } else {
     RMW_UROS_TRACE_MESSAGE("default port configuration error")