Am 02.09.2014 um 01:40 schrieb Dave Airlie:
From: Dave Airlie <airl...@redhat.com>

Coverity reported this, looks wrong to me.

And is probably one of the reasons why streams with multiple slices in one frame didn't worked.

Signed-off-by: Dave Airlie <airl...@redhat.com>

Reviewed-by: Christian König <christian.koe...@amd.com>

---
  src/gallium/state_trackers/omx/vid_dec_h264.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/omx/vid_dec_h264.c 
b/src/gallium/state_trackers/omx/vid_dec_h264.c
index 7f1c2fa..7b57785 100644
--- a/src/gallium/state_trackers/omx/vid_dec_h264.c
+++ b/src/gallium/state_trackers/omx/vid_dec_h264.c
@@ -681,7 +681,7 @@ static void slice_header(vid_dec_PrivateType *priv, struct 
vl_rbsp *rbsp,
        if (priv->picture.h264.field_pic_flag) {
           unsigned bottom_field_flag = vl_rbsp_u(rbsp, 1);
- if (bottom_field_flag != priv->picture.h264.bottom_field_flag);
+         if (bottom_field_flag != priv->picture.h264.bottom_field_flag)
              vid_dec_h264_EndFrame(priv);
priv->picture.h264.bottom_field_flag = bottom_field_flag;

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to