[FFmpeg-devel] [PATCH 1/3] lavf/mxf: fix parsing of the month from mxf timestamps

[Michael Niedermayer]

On Sat, Apr 13, 2013 at 03:45:55PM +0200, Hendrik Leppkes wrote:
> ---
>  libavformat/mxfdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 9eeba55..adc5c68 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1662,7 +1662,7 @@ static int mxf_timestamp_to_str(uint64_t timestamp, char **str)
>  {
>      struct tm time;
>      time.tm_year = (timestamp >> 48) - 1900;
> -    time.tm_mon  = (timestamp >> 48 & 0xF) - 1;
> +    time.tm_mon  = (timestamp >> 40 & 0xF) - 1;

>      time.tm_mday = (timestamp >> 32 & 0xF);
>      time.tm_hour = (timestamp >> 24 & 0XF);
>      time.tm_min  = (timestamp >> 16 & 0xF);

these look odd as well, the & 0xf stuff more precissely


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130413/c3bd959a/attachment.asc>