rmw icon indicating copy to clipboard operation
rmw copied to clipboard
verified publisher icon ros2

Initialize the NULL strucutre with static value.

Open fujitatomoya opened this issue 1 year ago • 4 comments

closes https://github.com/ros2/rmw/issues/377

fujitatomoya avatar Sep 09 '24 20:09 fujitatomoya

@comicfans could you try this patch to see if it fixes your asan warning?

fujitatomoya avatar Sep 09 '24 20:09 fujitatomoya

Makes sense to me, since the the struct is nested.

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

methylDragon avatar Sep 09 '24 21:09 methylDragon

@clalancette sounds good to me, let me try to replace them all.

fujitatomoya avatar Sep 10 '24 00:09 fujitatomoya

@comicfans could you try this patch to see if it fixes your asan warning?

confirm this fix the problem

comicfans avatar Sep 10 '24 07:09 comicfans

@clalancette can you take a look at this?

fujitatomoya avatar Sep 12 '24 16:09 fujitatomoya

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

fujitatomoya avatar Sep 12 '24 16:09 fujitatomoya

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

fujitatomoya avatar Sep 12 '24 22:09 fujitatomoya

@clalancette all comments are resolved, lgtm?

fujitatomoya avatar Sep 12 '24 22:09 fujitatomoya

RHEL is unstable, https://ci.ros2.org/job/ci_linux-rhel/1336/testReport/.

  • Linux-rhel Build Status

fujitatomoya avatar Sep 13 '24 16:09 fujitatomoya