diff options
| author | Patrick Rudolph <siro@das-labor.org> | 2016-09-14 19:50:16 +0200 |
|---|---|---|
| committer | Axel Davy <axel.davy@ens.fr> | 2016-10-10 23:43:51 +0200 |
| commit | 8ceb2264c571c7ca03cc9fbd08df92f3bea89372 (patch) | |
| tree | f8f9e254623a57b5592eb84c2a5ceb68d9cf84af /src/gallium/state_trackers/nine | |
| parent | f2eacef33de8d7b136b485927cb10f15d7439d48 (diff) | |
| download | external_mesa3d-8ceb2264c571c7ca03cc9fbd08df92f3bea89372.zip external_mesa3d-8ceb2264c571c7ca03cc9fbd08df92f3bea89372.tar.gz external_mesa3d-8ceb2264c571c7ca03cc9fbd08df92f3bea89372.tar.bz2 | |
st/nine: Fix assert in NineUnknown_QueryInterface
Tests showed that is allowed to call this method on
object that have a zero refcount.
Required for issue #230.
Signed-off-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Axel Davy <axel.davy@ens.fr>
Diffstat (limited to 'src/gallium/state_trackers/nine')
| -rw-r--r-- | src/gallium/state_trackers/nine/iunknown.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/gallium/state_trackers/nine/iunknown.c b/src/gallium/state_trackers/nine/iunknown.c index 6d501d8..89b82ea 100644 --- a/src/gallium/state_trackers/nine/iunknown.c +++ b/src/gallium/state_trackers/nine/iunknown.c @@ -70,7 +70,10 @@ NineUnknown_QueryInterface( struct NineUnknown *This, do { if (GUID_equal(This->guids[i], riid)) { *ppvObject = This; - assert(This->refs); + /* Tests showed that this call succeeds even on objects with + * zero refcount. This can happen if the app released all references + * but the resource is still bound. + */ NineUnknown_AddRef(This); return S_OK; } |