Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#4275 closed defect (fixed)

Internal Player can't handle MJPEG in a MOV container

Reported by: stuartm Owned by: Janne Grunau
Priority: minor Milestone: 0.21
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

FFplay from mid October has no problem with the format, but the Internal Player is unable to even start playback.

#
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'p1000393.mov':
#
  Duration: 00:01:48.0, start: 0.000000, bitrate: 9992 kb/s
#
  Stream #0.0(eng): Video: mjpeg, yuvj420p, 640x480, 30.00 fps(r)
#
  Stream #0.1(eng): Audio: pcm_u8, 8000 Hz, mono, 64 kb/s

Log with -v playback attached.

Attachments (3)

mjpeg_mov_failure.log (28.8 KB) - added by stuartm 12 years ago.
-v playback
av_new_stream.diff (1.5 KB) - added by greg 11 years ago.
add_new_stream2.diff (1.3 KB) - added by greg 11 years ago.

Download all attachments as: .zip

Change History (10)

Changed 12 years ago by stuartm

Attachment: mjpeg_mov_failure.log added

-v playback

comment:1 Changed 12 years ago by Janne Grunau

Owner: changed from Isaac Richards to Janne Grunau
Status: newassigned

comment:2 Changed 12 years ago by Janne Grunau

Milestone: unknown0.21
Status: acceptedstarted

comment:3 Changed 11 years ago by greg

The attached patch fixes a similar issue for me on a h264 encoded mp4 file.

Changed 11 years ago by greg

Attachment: av_new_stream.diff added

comment:4 Changed 11 years ago by greg

oops, new diff

Changed 11 years ago by greg

Attachment: add_new_stream2.diff added

comment:5 Changed 11 years ago by danielk

That second patch seems to work ok with my test files + with all my DTV channels in LiveTV. But I don't have ones with separate timing streams to test with.

comment:6 Changed 11 years ago by Janne Grunau

Resolution: fixed
Status: startedclosed

(In [16147]) revert our modifications to av_new_stream()

They are mpeg-ts specific and causes playback issues for other container formats. Error debugged by Greg, Daniel and I tested several mpeg-ts files and DVB liveTV without noticing regressions. Closes #4275

comment:7 Changed 11 years ago by Janne Grunau

(In [16192]) Merges revision [16147] from trunk: revert our modifications to av_new_stream()

They are mpeg-ts specific and causes playback issues for other container formats. Error debugged by Greg, Daniel and I tested several mpeg-ts files and DVB liveTV without noticing regressions. Closes #4275

Note: See TracTickets for help on using tickets.