Project

General

Profile

バグ #2341

ExecutionContextProfile setOwnerのバグ他

Added by n-ando over 10 years ago. Updated over 10 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
-
Start date:
02/04/2012
Due date:
% Done:

100%

Estimated time:

Description

ExecutionContextProfile::setOwner()でm_profile.ownerへの参照の代入が間違っているので修正する。
また、
  • addComponent()
  • setOwner()
  • getOwner()
  • getKind()
    でm_profileのロックが抜けているのでガードを追加する。

setOwner()の以下の行では、

  m_profile.owner = RTC::RTObject::_duplicate(m_profile.owner); // line:233

m_profile.ownerをm_profile.ownerに入れられてますが、
これは、以下が正しいのではと思います。いかがでしょうか?

 m_profile.owner = RTC::RTObject::_duplicate(rtobj);

さらに、上記の処理は、m_profileMutexがガードしなくてもよろしかったでしょうか?


Related issues

Related to OpenRTM-aist (C++) - バグ #2342: ExecutionContextProfile setOwnerのバグ他終了2012-02-04

Associated revisions

Revision 664 (diff)
Added by fsi-katami over 10 years ago

setOwner()'s bug was fixed. Some mutex guards have been added. refs #2341

History

#1 Updated by Anonymous over 10 years ago

  • Status changed from 新規 to 解決
  • % Done changed from 0 to 100

#2 Updated by n-ando over 10 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF