Created attachment 15751 [details] [review]
attached patch fixes two dereference before NULL check problems and
removes a effectivly unused variable (which was bad free'd).
Thanks for the patch. Let Zhenyu to commit.
Below block is not 100% correct. bufmgr_ttm should also be initialized later.
@@ -619,12 +621,14 @@ dri_ttm_fence_reference(dri_fence *fence)
- dri_fence_ttm *fence_ttm = (dri_fence_ttm *)fence;
+ dri_fence_ttm *fence_ttm;
dri_bufmgr_ttm *bufmgr_ttm = (dri_bufmgr_ttm *)fence->bufmgr;
+ fence_ttm = (dri_fence_ttm *)fence;
DBG("fence_unreference: %p (%s)\n", &fence_ttm->fence, fence_ttm->name);
Eric, can we close this bug now?
Fix committed for i810 xvmc part. The other part appears to be for the intel-batchbuffer branch, which is a dead end at this point (we're redoing it on GEM)