Commit 0bfab80a authored by Vittorio Giovara's avatar Vittorio Giovara

h264_sei: Group error check outside the switch block

parent 0a49a62f
...@@ -246,38 +246,28 @@ int ff_h264_decode_sei(H264Context *h) ...@@ -246,38 +246,28 @@ int ff_h264_decode_sei(H264Context *h)
switch (type) { switch (type) {
case SEI_TYPE_PIC_TIMING: // Picture timing SEI case SEI_TYPE_PIC_TIMING: // Picture timing SEI
ret = decode_picture_timing(h); ret = decode_picture_timing(h);
if (ret < 0)
return ret;
break; break;
case SEI_TYPE_USER_DATA_UNREGISTERED: case SEI_TYPE_USER_DATA_UNREGISTERED:
ret = decode_unregistered_user_data(h, size); ret = decode_unregistered_user_data(h, size);
if (ret < 0)
return ret;
break; break;
case SEI_TYPE_RECOVERY_POINT: case SEI_TYPE_RECOVERY_POINT:
ret = decode_recovery_point(h); ret = decode_recovery_point(h);
if (ret < 0)
return ret;
break; break;
case SEI_TYPE_BUFFERING_PERIOD: case SEI_TYPE_BUFFERING_PERIOD:
ret = decode_buffering_period(h); ret = decode_buffering_period(h);
if (ret < 0)
return ret;
break; break;
case SEI_TYPE_FRAME_PACKING: case SEI_TYPE_FRAME_PACKING:
ret = decode_frame_packing_arrangement(h); ret = decode_frame_packing_arrangement(h);
if (ret < 0)
return ret;
break; break;
case SEI_TYPE_DISPLAY_ORIENTATION: case SEI_TYPE_DISPLAY_ORIENTATION:
ret = decode_display_orientation(h); ret = decode_display_orientation(h);
if (ret < 0)
return ret;
break; break;
default: default:
av_log(h->avctx, AV_LOG_DEBUG, "unknown SEI type %d\n", type); av_log(h->avctx, AV_LOG_DEBUG, "unknown SEI type %d\n", type);
skip_bits(&h->gb, 8 * size); skip_bits(&h->gb, 8 * size);
} }
if (ret < 0)
return ret;
// FIXME check bits here // FIXME check bits here
align_get_bits(&h->gb); align_get_bits(&h->gb);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment