ahmad009 / slimdx

Automatically exported from code.google.com/p/slimdx
MIT License
0 stars 0 forks source link

[Patch] Setting effect state manager does not correctly release previous manager. #815

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In Effect9.cpp, Effect::StateManager::set calls delete on shim instead of 
Release although shim is a ref counted object. This causes an access violation 
when the DirectX effect tries to release its pointer to the manager.

In addition, passing nullptr as the method parameter is not correctly 
supported. In this case, no wrapper object must be created. Otherwise the next 
time the effect uses the state manager, the state manager will use its wrapped 
pointer (which is null) to query for state changes which causes a null pointer 
exception.

Original issue reported on code.google.com by a...@ventuz.com on 19 Jul 2011 at 1:15

Attachments:

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r2117.

Original comment by Mike.Popoloski on 26 Jul 2011 at 2:10