Project

General

Profile

バグ #2342

ExecutionContextProfile setOwnerのバグ他

Added by n-ando over 12 years ago. Updated over 12 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 (Java) - バグ #2341: ExecutionContextProfile setOwnerのバグ他終了2012-02-04

Associated revisions

Revision 2296 (diff)
Added by n-ando over 12 years ago

[compat,doc/impl,bugfix] setOwner()'s bug was fixed. Some mutex guards have been added. refs #2342

Revision 2296 (diff)
Added by n-ando over 12 years ago

[compat,doc/impl,bugfix] setOwner()'s bug was fixed. Some mutex guards have been added. refs #2342

History

#1 Updated by n-ando over 12 years ago

  • Status changed from 新規 to 終了
  • % Done changed from 0 to 100

Also available in: Atom PDF