RFR: 8282578: AIOOBE in javax.sound.sampled.Clip [v2]

[Alexander Zuev]

On Wed, 7 Dec 2022 15:34:48 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

>> Alexander Zuev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Got rid of the try/catch and using the proper per-case data length
>>    analysis.
>>  - Merge branch 'master' into JDK-8282578
>>  - Merge branch 'master' of https://github.com/azuev-java/jdk into JDK-8282578
>>  - Merge branch 'master'
>>  - 8282578: AIOOBE in javax.sound.sampled.Clip
>>    
>>    Surround SysEx message processing block with try/catch allowing MIDI subsystem to
>>    attempt to ingest the rest of the file.
>>    
>>    Added test case.
>
> src/java.desktop/share/classes/com/sun/media/sound/SoftMainMixer.java line 166:
> 
>> 164:                                         break;
>> 165:                                     }
>> 166:                                     // http://www.midi.org/about-midi/tuning_extens.shtml
> 
> Unrelated to change, but looks like those links become unavailable years ago.

Yes. That's the problem referencing the external site for information. Unfortunately the documentation layout on midi site changed and one needs to read trough all of it to figure out the correct path that both relates to the midi-1.0 standard that we support and is not overwritten by some later addendum making it non-informative.

-------------

PR: https://git.openjdk.org/jdk/pull/9016