1708 WAV tags saved by Mp3tag fail to show [#14360][#14361]

To get bugs in the current release fixed, please report them here.

Moderator: Gurus

1708 WAV tags saved by Mp3tag fail to show [#14360][#14361]

Postby chrisjj » Thu Aug 17, 2017 7:16 pm

(In my browser, each wide image gets truncated by the forum. To see it fully, open it in a new tab.)

The good news is that Mp3tag now supports WAV.

The bad news is that the tags don't show in MediaMonkey V4.1.3.1708

If I take a file made by GoldWave with WAV tag that shows fine in MediaMonkey V4.1.3.1708 and save it in Mp3tag v2.83i (with default config), the tag fields fail to show in MediaMonkey.

Mp3tag v2.81i - OK
Image

Windows 7 Explorer - OK
Image

MediaMonkey V4.1.3.1708 - faulty
Image

FooBar 2000 - OK
Image

Hex compare
Image

Looks like an MM bug.

Any suggestions for a workaround on this version of MM?

I tried working around by changing the config inc. Mp3tag Options Tag Mpeg despite that Help says this should not affect WAV, and I got a result that didn't show the problem and had a different encoding:

Image

but I was unable to reproduce this workaround.

(Reported at https://forums.mp3tag.de/index.php?showtopic=22803 )
Chris
chrisjj
 
Posts: 4926
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby chrisjj » Fri Aug 18, 2017 7:11 am

Workaround discovered:

I tried Mp3tag ISO:

Image

Image

Success!

Image

And in foobar2000:

Image

Binary FTR:

Image
Chris
chrisjj
 
Posts: 4926
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby Peke » Fri Aug 25, 2017 9:18 pm

Hi,
Sorry for late reply was abroad. Can you please send me DL links to all WAV files you used in tests (both good and bad one) so that I can HEX compare and try to track the issue?
Best regards,
Pavle
MM Core Developer and Admin of free MediaMonkey extensions Hosting
Image
Image
Peke
 
Posts: 10409
Joined: Tue Jun 10, 2003 7:21 pm
Location: Serbia

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby chrisjj » Sat Aug 26, 2017 7:15 am

Thanks Peke. These files

Image

are here: https://www.dropbox.com/s/hnuzlfx2mqcyt ... e.zip?dl=0 .
Chris
chrisjj
 
Posts: 4926
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby Peke » Sun Aug 27, 2017 11:07 am

Hi,
I've been able to analyze here are results:
  1. "2 Untitled1 saved by GW 6.30 - title OK in Mp3tag, Explorer and MM.wav" -> Contains only Standard "LISTINFO" chunk and thus all Chunks are under RIFF ISO standard file is OK
  2. "3 Untitled1 saved by Mp3tag v2.83 - title OK in Mp3tag and Explorer, missing in MM.wav" -> "ID3 " Chunk defined to be length 41 Bytes but actually it is 42, uses ID3v2.3 tag format
  3. "4 Untitled1 saved by Mp3tag v2.83i - title OK in Mp3tag and Explorer, missing in MM.wav" -> Identical to "3 Untitled1 saved by Mp3tag v2.83 - title OK in Mp3tag and Explorer, missing in MM.wav" so it observes same issue
  4. "5 Untitled1 saved by Mp3tag v2.83i changed config - title OK in Mp3tag, Explorer and MM - now bad in MM!.wav" -> same as above only change it uses UNICODE ID3v2.4 tag format
  5. "6 Untitled1 saved by Mp3tag v2.83i UTF8 - title OK in Mp3tag and Explorer, bad in MM.wav" -> same as above only change it uses UTF8 ID3v2.4 tag format
  6. "7 Untitled1 saved by Mp3tag v2.83i ISO - title OK in Mp3tag, Explorer and MM.wav" -> "ID3 " present but formatted according to RIFF/WAV ISO Format CHUNK and have correct length it shows/import OK eg. it is correctly terminated without extra ZERO Byte on file end unlike previous 4

NOTE: "ID3 " Chunk is not standard chunk by RIFF ISO see http://www.robotplanet.dk/audio/wav_meta_data/ , https://github.com/mpruett/audiofile/issues/3 , http://wavmetadata.blogspot.rs/ (one with most correct info)

Unfortunately there is a BUG in both MP3TAG and in MMW PLUGIN which makes this bit complicated to debug.

For MMW I'll Add bug in order to Support ID3v2.3 "ID3 " Chunk eg. files 3 and 4 and add correction to Ignore junk ZERO Byte on file end and as you already opened issue on MP3tag you can rise bug there. For that Purpose I uploaded manually corrected files 5 and 6 onto http://www.happymonkeying.com/XoXoTemp/ ... _issue.zip so that you can include them in your archive on Dropbox for future use.

Here is a PIC of my manual FIX (NOTE: FILE size in RIFF Header also needs to be corrected as highlighted)
Image
Best regards,
Pavle
MM Core Developer and Admin of free MediaMonkey extensions Hosting
Image
Image
Peke
 
Posts: 10409
Joined: Tue Jun 10, 2003 7:21 pm
Location: Serbia

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby Peke » Sun Aug 27, 2017 11:23 am

Added as http://www.ventismedia.com/mantis/view.php?id=14360 and http://www.ventismedia.com/mantis/view.php?id=14361

NOTE: Unless MP3TAG fixes write bug on its end where it adds JUNK Byte even it is somewhat common practice to make even byte then apps not aware of the bug/feature can even further corrupt the file making irreversibly non playable.

Also please Cross point to this Forum topic as it provides more details.
Best regards,
Pavle
MM Core Developer and Admin of free MediaMonkey extensions Hosting
Image
Image
Peke
 
Posts: 10409
Joined: Tue Jun 10, 2003 7:21 pm
Location: Serbia

Re: 1708 WAV tags saved by Mp3tag fail to show [#14360][#143

Postby Peke » Wed Aug 30, 2017 2:32 pm

Fixed MMW 4.1.18.1846+ to read such files even no changes will be made in MP3Tag.

Also ID3v2.3 with UTF16 tags are now correctly read.
Best regards,
Pavle
MM Core Developer and Admin of free MediaMonkey extensions Hosting
Image
Image
Peke
 
Posts: 10409
Joined: Tue Jun 10, 2003 7:21 pm
Location: Serbia

Re: 1708 WAV tags saved by Mp3tag fail to show

Postby chrisjj » Wed Aug 30, 2017 3:02 pm

Thanks for looking into this Peke.

Peke wrote:Chunk defined to be length 41 Bytes but actually it is 42, uses ID3v2.3 tag format

I think that's incorrect. I think the chunk length is 0x41 bytes as specified. I think the last byte is outside the chunk because it is a pad byte.

Image

As per https://en.wikipedia.org/wiki/Resource_ ... ile_Format
4 bytes: an ASCII identifier for this chunk (examples are "fmt " and "data"; note the space in "fmt ").
4 bytes: an unsigned, little-endian 32-bit integer with the length of this chunk (except this field itself and the chunk identifier).
variable-sized field: the chunk data itself, of the size given in the previous field.
a pad byte, if the chunk's length is not even.


When I lengthen the title by one character, the MM fail disappears:

Image
Chris
chrisjj
 
Posts: 4926
Joined: Wed Feb 14, 2007 5:14 pm
Location: UK

Re: 1708 WAV tags saved by Mp3tag fail to show [#14360][#143

Postby Peke » Thu Aug 31, 2017 9:23 am

You are right MMW was not calculating Pad char. It is fixed in 1846 all your example files work. It is debatable if CHUNK length should include Padded bytes or not I found it somewhat unclear.
Best regards,
Pavle
MM Core Developer and Admin of free MediaMonkey extensions Hosting
Image
Image
Peke
 
Posts: 10409
Joined: Tue Jun 10, 2003 7:21 pm
Location: Serbia


Return to Bug Reports (mmw)

Who is online

Users browsing this forum: No registered users and 12 guests