Fix race condition in case goal completes while calling getState()

[Gamezar]

In WAITING_FOR_RESULT or WAITING_FOR_CANCEL_ACK, yet we are in SimpleGoalState DONE. This is a bug in SimpleActionClient

Happens in multiple thread environment where first thread got a copy of CommState while doing getState() and other thread updated goal state to DONE

[sloretz]

Thank you for the PR!

ROS Noetic will reach end-of-life on May 31st, 2025. Every change comes with a risk of introducing regressions, and there isn't much time left to fix them. To make sure this PR doesn't introduce any regressions please:

• Describe how you tested this change
• Recruit at least one more person to review this PR and try it out on their system

[sloretz]

ROS 1 is end-of-life (EOL) as of today, May 31st 2025. I am archiving this repository because:

• it only supports ROS 1
• it isn't needed anymore in ROS 2

If you still rely on ROS 1, read this page to learn about your options.