[ 
https://issues.apache.org/jira/browse/TIKA-424?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12894589#action_12894589
 ] 

Nick Burch commented on TIKA-424:
---------------------------------

Actually, it looks like it's just a bug in the ID3v2.4 specs...

I've just tried creating some ID3v2.4 files using mid3v2, and it does 
size=bytes just like the sample file. No-one seems to do 4* like the spec doc 
says. Interestingly, the spec doc has 4*size in the layout section, then goes 
on to describe the size as being just like ID3v2.3, i.e. no 4*. I think it's a 
spec bug :(

In r981590 I have updated tika to follow what all the other programs do, and 
added a testcase with a sample file from mid3v2

> Avoid ArrayIndexOutOfBoundsException on some mp3 files
> ------------------------------------------------------
>
>                 Key: TIKA-424
>                 URL: https://issues.apache.org/jira/browse/TIKA-424
>             Project: Tika
>          Issue Type: Improvement
>          Components: parser
>    Affects Versions: 0.7
>            Reporter: Nick Burch
>            Assignee: Jukka Zitting
>             Fix For: 0.8
>
>         Attachments: mp3-fix.patch, test2.mp3
>
>
> There can currently be an ArrayIndexOutOfBoundsException if the tag of an mp3 
> is empty but null padded (see TIKA-418 for someone reporting one against the 
> wrong bug...)
> Attached patch correctly detects this condition and returns an empty string.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to