[FFmpeg-trac] #1817(undetermined:reopened): create_time contains wrong value for Canon EOS 550D movies

FFmpeg trac at avcodec.org
Mon Oct 15 19:32:10 CEST 2012


#1817: create_time contains wrong value for Canon EOS 550D movies
-------------------------------------+-------------------------------------
             Reporter:  pwes         |                    Owner:
                 Type:  defect       |                   Status:  reopened
             Priority:  normal       |                Component:
              Version:  git-master   |  undetermined
             Keywords:  mov          |               Resolution:
  creation_time                      |               Blocked By:
             Blocking:               |  Reproduced by developer:  0
Analyzed by developer:  0            |
-------------------------------------+-------------------------------------

Comment (by pwes):

 Here's what fixes situation:
 {{{
 index bf695e7..b2594db 100644
 --- a/libavformat/mov.c
 +++ b/libavformat/mov.c
 @@ -783,7 +783,6 @@ static void mov_metadata_creation_time(AVDictionary
 **metadata, time_t time)
      char buffer[32];
      if (time) {
          struct tm *ptm;
 -        if(time >= 2082844800)
              time -= 2082844800;  /* seconds between 1904-01-01 and Epoch
 */
          ptm = gmtime(&time);
          if (!ptm) return;
 }}}

 Looks like it was introduced due to #1471, in GitHub repo it's
 https://github.com/FFmpeg/FFmpeg/commit/23eeffcd48a15e73fb2649b712870b6d101c5471
 (I can't find this commit in git://source.ffmpeg.org/ffmpeg.git).

 After applying the above change the date is shown correctly:
 {{{
 ffmpeg version git-2012-10-15-3ca8a23 Copyright (c) 2000-2012 the FFmpeg
 developers
   built on Oct 15 2012 19:31:14 with gcc 4.6 (Ubuntu/Linaro
 4.6.3-1ubuntu5)
   configuration: --enable-gpl --enable-libfaac --enable-libmp3lame
 --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-librtmp
 --enable-libtheora --enable-libvorbis --enable-libvpx --enable-x11grab
 --enable-libx264 --enable-nonfree --enable-version3
   libavutil      51. 76.100 / 51. 76.100
   libavcodec     54. 66.100 / 54. 66.100
   libavformat    54. 32.101 / 54. 32.101
   libavdevice    54.  3.100 / 54.  3.100
   libavfilter     3. 19.102 /  3. 19.102
   libswscale      2.  1.101 /  2.  1.101
   libswresample   0. 16.100 /  0. 16.100
   libpostproc    52.  1.100 / 52.  1.100
 Guessed Channel Layout for  Input Stream #0.1 : stereo
 Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'MVI_5304.MOV':
   Metadata:
     major_brand     : qt
     minor_version   : 537331968
     compatible_brands: qt  CAEP
     creation_time   : 2012-10-15 13:16:59
   Duration: 00:00:01.02, start: 0.000000, bitrate: 18889 kb/s
     Stream #0:0(eng): Video: h264 (Constrained Baseline) (avc1 /
 0x31637661), yuvj420p, 640x480, 17337 kb/s, 50 fps, 50 tbr, 50k tbn, 100k
 tbc
     Metadata:
       creation_time   : 2012-10-15 13:16:59
     Stream #0:1(eng): Audio: pcm_s16le (sowt / 0x74776F73), 48000 Hz,
 stereo, s16, 1536 kb/s
     Metadata:
       creation_time   : 2012-10-15 13:16:59
 }}}

-- 
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/1817#comment:5>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list