niklas/rmw_cyclonedds
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix bad conditional in rmw_serialize(). (#217)

Browse source 
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
This commit is contained in:
Michel Hidalgo 2020-08-14 14:05:01 -03:00 • committed by Alejandro Hernández Cordero
parent bc2bc9ef1b
commit aa4ee36aff
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
rmw_cyclonedds_cpp/src/rmw_node.cpp
View file

@ -1388,13 +1388,13 @@ extern "C" rmw_ret_t rmw_serialize(
const rosidl_message_type_support_t * type_support, const rosidl_message_type_support_t * type_support,
rmw_serialized_message_t * serialized_message) rmw_serialized_message_t * serialized_message)
{ {
rmw_ret_t ret;
try { try {
auto writer = rmw_cyclonedds_cpp::make_cdr_writer( auto writer = rmw_cyclonedds_cpp::make_cdr_writer(
rmw_cyclonedds_cpp::make_message_value_type(type_support)); rmw_cyclonedds_cpp::make_message_value_type(type_support));
auto size = writer->get_serialized_size(ros_message); auto size = writer->get_serialized_size(ros_message);
if ((ret = rmw_serialized_message_resize(serialized_message, size) != RMW_RET_OK)) { rmw_ret_t ret = rmw_serialized_message_resize(serialized_message, size);
if (RMW_RET_OK != ret) {
RMW_SET_ERROR_MSG("rmw_serialize: failed to allocate space for message"); RMW_SET_ERROR_MSG("rmw_serialize: failed to allocate space for message");
return ret; return ret;
} }