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

Clang Thread Safety Annotation Warning in rmw_get_topic_endpoint_info #344

Closed
piraka9011 opened this issue Jan 16, 2020 · 1 comment · Fixed by #345
Closed

Clang Thread Safety Annotation Warning in rmw_get_topic_endpoint_info #344

piraka9011 opened this issue Jan 16, 2020 · 1 comment · Fixed by #345

Comments

@piraka9011
Copy link

piraka9011 commented Jan 16, 2020

Bug report

Required Info:

  • Operating System:
    • Ubuntu 18.04
  • Installation type:
    • Source
  • Version or commit hash:
  • DDS implementation:
    • FastRTPS
  • Client library (if applicable):
    • N/A

Steps to reproduce issue

Building with --mixin clang-libcxx

Expected behavior

No warnings

Actual behavior

CI results in the following warnings

reading variable 'discovered_names' requires holding mutex 'slave_target->names_mutex_' [-Wthread-safety-analysis]

Additional information

It looks like there's already a guard here

guid_map_t discovered_names RCPPUTILS_TSA_GUARDED_BY(names_mutex_);

but it doesn't look like the macro is captured by TSAN.

@emersonknapp
Copy link
Contributor

Just a nitpick - TSAN and Clang Thread Safety Annotations are different things, the title is incorrect

@piraka9011 piraka9011 changed the title TSAN Warning in rmw_get_topic_endpoint_info Clang Thread Safety Annotation Warning in rmw_get_topic_endpoint_info Jan 16, 2020
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.

2 participants