IPP 5.2 UMC::MP3Encoder bug that causes crash

[agent_smith]

I'm using w_ipp_ia32_p_5.2.057.exe and w_ipp-samples_p_5.2.049.zip, the latest availabe for download.

The following code C++ code:

{
UMC::AudioCodecParams *lpAudioEncoderParams = new UMC::AudioCodecParams();
UMC::MP3Encoder *lpAudioEncoder = new UMC::MP3Encoder();

lpAudioEncoderParams->m_info_out.bitrate = 128000;
lpAudioEncoderParams->m_info_out.sample_frequency = 44100;
lpAudioEncoderParams->m_info_out.channels = 2;
lpAudioEncoderParams->m_info_out.channel_mask = 3;
lpAudioEncoderParams->m_info_out.stream_type = UMC::MP1L3_AUDIO;
lpAudioEncoderParams->m_info_out.bitPerSample = 16;

lpAudioEncoderParams->m_info_in = lpAudioEncoderParams->m_info_out;

UMC::Status res = lpAudioEncoder->Init(lpAudioEncoderParams);

delete lpAudioEncoder;
lpAudioEncoder = NULL;

delete lpAudioEncoderParams;
lpAudioEncoderParams = NULL;
}

causes application crash (Invalid Address specified to RtlFreeHeap).

I traced the problem and it turned out that the function mp3encClose_layer3() causes it.
It is implemeted that way:

static void mp3encClose_layer3(MP3Enc *state)
{
if(state->pMDCTSpec36)
ippsFree(state->pMDCTSpec36);
if(state->pMDCTSpec12)
ippsFree(state->pMDCTSpec12);

state->com.si_private_bits = 0;
}


but it should be something like:

static void mp3encClose_layer3(MP3Enc *state)
{
if (state->pMDCTSpec36)
ippsMDCTFwdFree_32f(state->pMDCTSpec36);
state->pMDCTSpec36 = NULL;

if (state->pMDCTSpec12)
ippsMDCTFwdFree_32f(state->pMDCTSpec12);
state->pMDCTSpec12 = NULL;

state->com.si_private_bits = 0;
}

The latest beta version did not have that problem.

[Vladimir_Dudnik]

Hello,

thanks for your report, I've created issue for you on Premier Intel Support, so you will be notifed regarding progress on this issue.

Regards,
Vladimir