Serial-ATA / lofty-rs

Audio metadata library
Apache License 2.0
185 stars 34 forks source link

ID3v2 IPLS/TPIL Involved People List is not read correctly #213

Closed FriederHannenheim closed 1 year ago

FriederHannenheim commented 1 year ago

Reproducer

Run the tag_reader example with the attached file (Audio has been removed for copyright reasons but the tag remains, also rename it to mp3, I renamed it to txt since Github doesn't allow mp3 upload)

Summary

I stepped through the lofty code with a debugger and the TIPL tag (IPLS in the file) is read incorrectly. IPLS is a list of terminated strings (source). Lofty tries to read it as one string and so only the first item is read. Then lofty will try to read the next frame in the data of the IPLS tag and crash.

Expected behavior

The IPLS frame is read correctly

Assets

!!! RENAME TO MP3 !!! testz.txt