diff options
| author | James Almer <jamrial@gmail.com> | 2019-04-15 23:51:49 +0300 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2019-04-17 02:33:10 +0300 |
| commit | 137c0fcad15139921279038351048970e99838f4 (patch) | |
| tree | 7b0ac70e5f7f4796d2f883cea017e55ef325ade1 | |
| parent | 9bf520d04d6137d0772e019356356614bbf7ca82 (diff) | |
avcodec/cbs_h265: fix storage type for time_offset_value in Time Code SEI
The spec defines it as an array of signed values, inferred to 0 when not
present.
Reviewed-by: Mark Thompson <sw@jkqxz.net>
Signed-off-by: James Almer <jamrial@gmail.com>
| -rw-r--r-- | libavcodec/cbs_h265.h | 2 | ||||
| -rw-r--r-- | libavcodec/cbs_h265_syntax_template.c | 7 |
2 files changed, 6 insertions, 3 deletions
diff --git a/libavcodec/cbs_h265.h b/libavcodec/cbs_h265.h index d216caca2b..0c0e4f84b0 100644 --- a/libavcodec/cbs_h265.h +++ b/libavcodec/cbs_h265.h @@ -659,7 +659,7 @@ typedef struct H265RawSEITimeCode { uint8_t minutes_flag[3]; uint8_t hours_flag[3]; uint8_t time_offset_length[3]; - uint32_t time_offset_value[3]; + int32_t time_offset_value[3]; } H265RawSEITimeCode; typedef struct H265RawSEIMasteringDisplayColourVolume { diff --git a/libavcodec/cbs_h265_syntax_template.c b/libavcodec/cbs_h265_syntax_template.c index f1e1bb0e7e..bbd363a389 100644 --- a/libavcodec/cbs_h265_syntax_template.c +++ b/libavcodec/cbs_h265_syntax_template.c @@ -1986,8 +1986,11 @@ static int FUNC(sei_time_code)(CodedBitstreamContext *ctx, RWContext *rw, us(5, time_offset_length[i], 0, 31, 1, i); if (current->time_offset_length[i] > 0) - us(current->time_offset_length[i], time_offset_value[i], - 0, MAX_UINT_BITS(current->time_offset_length[i]), 1, i); + is(current->time_offset_length[i], time_offset_value[i], + MIN_INT_BITS(current->time_offset_length[i]), + MAX_INT_BITS(current->time_offset_length[i]), 1, i); + else + infer(time_offset_value[i], 0); } } |