humanoid-path-planner / hpp-corbaserver

Corba server for Humanoid Path Planner applications
BSD 2-Clause "Simplified" License
4 stars 20 forks source link

Memory management issue #112

Open jmirabel opened 4 years ago

jmirabel commented 4 years ago

Servant keys should not be the storage pointer.

This pointer may be deleted and the same address could be given to another object. In this case, the server will think it already has a servant for this pointer and will return the old servant, which is wrong.

Note that this only happens when persistent storage is set to false. A possible fix is to remove the servant from the server map when persistent storage is disabled.

jmirabel commented 4 years ago

Related to https://github.com/agimus/agimus-hpp/pull/14