Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🧑‍🌾 Rolling CI warning: format ‘%s’ expects a matching ‘char*’ #170

Open
Crola1702 opened this issue Feb 26, 2025 · 2 comments · May be fixed by #171
Open

🧑‍🌾 Rolling CI warning: format ‘%s’ expects a matching ‘char*’ #170

Crola1702 opened this issue Feb 26, 2025 · 2 comments · May be fixed by #171
Assignees

Comments

@Crola1702
Copy link

Crola1702 commented Feb 26, 2025

System Info

  • OS
    • Ubuntu 20.04
  • ROS version and installation type
    • Rolling
  • RTI Connext DDS version and installation type
    • CI Install

Bug Description

Reference builds:

Warning:

  • rmw_context.cpp:473: format ‘%s’ expects a matching ‘char*’ argument [-Wformat=] 473 | "failed to assert DDS property: '%s' = '%s'",
In file included from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/include/rmw_connextdds/log.hpp:20,
                 from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/include/rmw_connextdds/context.hpp:24,
                 from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/include/rmw_connextdds/rmw_impl.hpp:25,
                 from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:26:
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp: In member function ‘rmw_ret_t rmw_context_impl_s::configure_security(DDS_DomainParticipantQos*)’:
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:473:7: warning: format ‘%s’ expects a matching ‘char*’ argument [-Wformat=]
  473 |       "failed to assert DDS property: '%s' = '%s'",
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/include/rmw_connextdds/log.hpp:195:3: note: in expansion of macro ‘RMW_CONNEXT_LOG_ERROR_A’
  195 |   RMW_CONNEXT_LOG_ERROR_A(fmt_, __VA_ARGS__) \
      |   ^~~~~~~~~~~~~~~~~~~~~~~
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:472:5: note: in expansion of macro ‘RMW_CONNEXT_LOG_ERROR_A_SET’
  472 |     RMW_CONNEXT_LOG_ERROR_A_SET(
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/install/rmw/include/rmw/rmw/error_handling.h:31,
                 from /home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:24:
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:473:7: warning: format ‘%s’ expects a matching ‘char*’ argument [-Wformat=]
  473 |       "failed to assert DDS property: '%s' = '%s'",
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:472:5: note: in expansion of macro ‘RMW_CONNEXT_LOG_ERROR_A_SET’
  472 |     RMW_CONNEXT_LOG_ERROR_A_SET(
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/jenkins-agent/workspace/nightly_linux_repeated/ws/src/ros2/rmw_connextdds/rmw_connextdds_common/src/common/rmw_context.cpp:473:48: note: format string is defined here
  473 |       "failed to assert DDS property: '%s' = '%s'",
      |                                               ~^
      |                                                |
      |                                                char*
@fgallegosalido
Copy link
Collaborator

fgallegosalido commented Feb 26, 2025

This seems to come from #167. The fix is just adding to the line 474 in the file rmw_context.cpp the string DDS_SECURITY_PRIVATE_KEY_PROPERTY. This is consistent with the rest of the changes,

@fujitatomoya fujitatomoya linked a pull request Feb 26, 2025 that will close this issue
@fujitatomoya
Copy link
Collaborator

@fgallegosalido thanks, here is the fix #171.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants