Sandbox.Game.MyInventory IsConnected is not thread safe

KeenSoftwareHouse / SpaceEngineers

2.94k stars 896 forks source link

Sandbox.Game.MyInventory IsConnected is not thread safe #574

Open Dummy0815 opened 7 years ago

Dummy0815 commented 7 years ago

As the 'reachableVertices' list is declared as field the usage of this list inside IsConnected is not thread safe. The list is only used inside the function and cleared before every usage, I suggest to use an local list instead.

InflexCZE commented 6 years ago

The code in question is no longer in production codebase of SE.