1. 07 Sep, 2012 2 commits
  2. 13 Jul, 2012 3 commits
    • Jean-Paul Saman's avatar
      src/xvba_video_glx.c: fix refcounting surface. · 8a9898b6
      Jean-Paul Saman authored
      A GLX surface can be in use by multiple VASurfaces, therefor make
      sure refcounting is done properly. The code did not honor this and
      could destroy a surface that was still in use by another GLX Surface
      object.
      
      The code is changed to enforce proper refcounting. Basically using
      
         destroy_glx_surface(driver_data, obj_glx_surface);
      
      is suspicious and should be replaced by:
      
         glx_surface_unref(driver_data, obj_glx_surface);
      
      instead. This function decreases the refcount on obj_glx_surface and
      when it reaches 0, then it destroys the surface properly.
      8a9898b6
    • Jean-Paul Saman's avatar
      src/xvba_video.c: Do not leak subpicture associations. · 265d9f65
      Jean-Paul Saman authored
      xvba_DestroySurfaces() leaked subpicture associations, since the
      for loop did not actually iterate over the subpicture association
      array.
      265d9f65
    • Jean-Paul Saman's avatar
      src/xvba_videx_{x11,glx}: add debugging code · a2b59c02
      Jean-Paul Saman authored
      Assert if prerequisites are not met.
      a2b59c02
  3. 11 Jul, 2012 7 commits
  4. 30 Jun, 2012 1 commit
  5. 13 Jun, 2012 2 commits
  6. 08 Jun, 2012 6 commits
  7. 07 Jun, 2012 3 commits
  8. 05 Jun, 2012 1 commit
  9. 23 Apr, 2012 1 commit
  10. 13 Apr, 2012 2 commits
  11. 10 Jan, 2012 5 commits
    • Jean-Paul Saman's avatar
      Split check for if-clause · 8ae2becd
      Jean-Paul Saman authored
      The boolean operator || requires an evaluation of both expressions.
      If the expressions evaluates pointers, then care must be taken that
      both expressions have valid pointer. In this case it cannot be guaranteed,
      thus splitting the check is needed.
      8ae2becd
    • Jean-Paul Saman's avatar
      xvba_video_glx.c: split check for if-clause · 5930ef22
      Jean-Paul Saman authored
      The boolean operator || requires an evaluation of both expressions.
      If the expressions evaluates pointers, then care must be taken that
      both expressions have valid pointer. In this case it cannot be guaranteed,
      thus splitting the check is needed.
      5930ef22
    • Jean-Paul Saman's avatar
      object_heap.c: calculate the correct index into the heap. · 515c1a0d
      Jean-Paul Saman authored
      The location of an object in the heap was wrongly determined, by applying the HEAP_ID_MASK to the object id. Instead the heap->id_offset should have been substracted from the object id. The object id is created by using the i-th position plus heap->id_offset.
      515c1a0d
    • Jean-Paul Saman's avatar
      xvba_video_glx.c: split check in transfer_surface. · 4a3eac85
      Jean-Paul Saman authored
      The boolean operator || requires an evaluation of both expressions.
      If the expressions evaluates pointers, then care must be taken that
      both expressions have valid pointer. In this case it cannot be guaranteed,
      thus splitting the check is needed.
      4a3eac85
    • Jean-Paul Saman's avatar
      remember updated associations. · ae8e2fa8
      Jean-Paul Saman authored
      ae8e2fa8
  12. 23 Dec, 2011 3 commits
  13. 09 Dec, 2011 1 commit
  14. 08 Nov, 2011 3 commits