I have found reason why bad main_data_begin pointer happend

[dadidzf]

var mp3Decode = AV.Decoder.find('mp3');
var readChunk = mp3Decode.prototype.readChunk
mp3Decode.prototype.readChunk = function () {
    var sync = this.mp3_stream.sync;
    var next_frame = this.mp3_stream.next_frame;
    try {
        return readChunk.bind(this)();
    }
    catch (err) {
        this.mp3_stream.sync = sync;
        this.mp3_stream.next_frame = next_frame;
        throw err;
    }
}

The above code shows how i fix the bad main_data_begin pointer issue.
The resason is because, the point (next_frame) of the mp3_stream is not reverted while catch err.
After add the code in other place, i fix that problem.

I don't know the best way to fix it, in mp3.js repository directly, so i just add the issue here.

[pschuegr]

Hi @dadidzf, nice find! Can you explain where you put this code? I'm trying to sort out this problem too.

[pschuegr]

Never mind! I see you just have to put it in your own code before calling the AV funcs. @devongovett would you accept a PR for this?

[dadidzf]

I use Aurora.js and mp3.js together, but i didn't modify them to fix it, I put the code in my own application project. i have verify this code works fine. if we want to add PR for this, i think we should modify the function "readChunk" in src/decoder.js.