search

OpenRTM / OpenRTM-aist

OpenRTM-aist: RT-Middleware and OMG RTC implementation in C++ implemented by AIST
https://openrtm.org/
Other
19 stars 12 forks source link

LocalServiceAdmin における mutex の使用がバグに見える #616

Closed r-kurose closed 5 years ago

r-kurose commented 5 years ago

下記で m_mutex を使っているがこれは coil::Singoleton で定義されたもののに見える。 意図したものか、そうでないなら何が正しいのか。 https://github.com/OpenRTM/OpenRTM-aist/blob/7ea713b2b06e20fe86ae4ae8b9a1e91366eb665d/src/lib/rtm/LocalServiceAdmin.cpp#L273-L276

n-ando commented 5 years ago

バグです。正しくは m_services_mutex でガードすべきです。

n-ando commented 5 years ago

660 にて修正。