Opened 8 years ago

Closed 8 years ago

#3645 closed defect (fixed)

Remuxing adpcm_ms to caf fails for some samples

Reported by: nu774 Owned by:
Priority: normal Component: avformat
Version: git-master Keywords: caf
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no

Description

How to reproduce:

% ffmpeg -i foo.wav -c:a adpcm_ms bar.caf

The resulting file seems to be playable by ffmpeg based software, but not by QuickTime 7.

Attachments (1)

MSADPCM.wav (1.4 MB ) - added by Carl Eugen Hoyos 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Carl Eugen Hoyos, 8 years ago

Component: undeterminedavformat
Keywords: caf added
Priority: importantnormal

Please provide the complete, uncut console output to make this a valid ticket.

comment:2 by Carl Eugen Hoyos, 8 years ago

Resolution: needs_more_info
Status: newclosed

I was unable to reproduce this (only tested with QuickTime 7).

I would like to fix this (if there is an issue) so please reopen if you can add any additional information.

comment:3 by nu774, 8 years ago

OK, it seems that MS-ADPCM encoded by ffmpeg seems to be playable on QuickTime as you say, so my report was improper.
What I actually tried was remuxing from existing MS-ADPCM wav file encoded by Microsoft standard MS-ADPCM ACM codec.
Please look at samples at https://dl.dropboxusercontent.com/u/76042064/MSADPCM_samples.zip .
In the zip archive contained :

  • MSADPCM.wav: the original WAV file, encoded by Microsoft ACM codec.
  • MSADPCM.caf: Remuxed into CAF by Apple API, contains kuki chunk, playable by QuickTime.
  • MSADPCM.ffmpeg.caf: Remuxed into CAF by ffmpeg, lacks kuki chunk, not correctly playable by QuickTime.

As far as I understand, MSADPCM codec needs some codec setup data (coefs) that are stored in ADPCMWAVEFORMAT defined in mmreg.h, which goes under fmt chunk in case of WAV file / kuki chunk in case of CAF.

comment:4 by Carl Eugen Hoyos, 8 years ago

Resolution: needs_more_info
Status: closedreopened

Don't you agree that you would save both of us time if you just pasted the command line that produces the unplayable output together with the complete, uncut console output?

by Carl Eugen Hoyos, 8 years ago

Attachment: MSADPCM.wav added

comment:5 by Carl Eugen Hoyos, 8 years ago

Reproduced by developer: set
Resolution: fixed
Status: reopenedclosed
Summary: caf muxer doesn't write kuki chunk for adpcm_msRemuxing adpcm_ms to caf fails for some samples
Version: unspecifiedgit-master

Should be fixed in 1acb5ca0 - thank you for the report!

Note: See TracTickets for help on using tickets.