Question

我正在尝试创建一个简单的媒体播放器来介绍ffmpeg的世界，问题是我每次调用av_freep(void*ptr)应用程序崩溃。

如果我没有调用av_freep，我会收到内存泄漏，程序使用的内存增加到1000MB（已经测量过），下面是代码：

int16_t* audioBuffer=(int16_t*)av_malloc(AVCODEC_MAX_AUDIO_FRAME_SIZE+FF_INPUT_BUFFER_PADDING_SIZE);

if(!audioBuffer){
    MessageBox(0,"Error allocating in audioBuffer","Error: Mem",MB_ICONWARNING | MB_OK);
    return -1;
}

int sz = MEDIA->DecodeAudioFrame((void*)audioBuffer,0);

Pa_WriteStream(MEDIA->output_stream,(int16_t*)audioBuffer,MEDIA->_audio_ccontext->frame_size);

av_freep(audioBuffer);

这是我的'DecodeAudioFrame'功能代码：

int WbMedia::DecodeAudioFrame(void *audio_buf, int buf_size){
static AVFrame frame;
static AVPacket pkt;
static uint8_t *audio_pkt_data = NULL;
static int audio_pkt_size = 0;

int len1=0;

for(;;){
    bool do_rt = false;


    while(audio_pkt_size > 0){
        int obt_frame = 0;

        len1 = avcodec_decode_audio4(_audio_ccontext,&frame,&obt_frame,&pkt);
        if(len1 < 0){
            audio_pkt_size = 0;
            break;
        }

        audio_pkt_data+=len1;
        audio_pkt_size-=len1;

        if(obt_frame){
            data_size = av_samples_get_buffer_size(frame.linesize,channel_count,sample_fr,_audio_ccontext->sample_fmt,1);
            memcpy(audio_buf,frame.data[0],data_size);
        }

        if(data_size < 0){
            continue;
        }

        if(pkt.data){
            av_free_packet(&pkt);
        }
        return data_size;

    }

    if(pkt.data){
        av_free_packet(&pkt);
    }

    if(do_rt){
        return data_size;
    }

    // Try to get a new packet
    if(!audio_packets.empty()){
        WaitForSingleObject(Queue_Audio_Mutex,INFINITE);
            pkt = audio_packets.front();
            audio_packets.pop();
        ReleaseMutex(Queue_Audio_Mutex);

        audio_pkt_size = pkt.size;
        audio_pkt_data = pkt.data;
    }else{
        return -1;
    }
}
return 0;
}

我需要帮助解决这个问题，我不知道这是一个错误还是我需要做的事情。那里发生了什么？为什么它会在av_freep电话上崩溃？我该如何解决？

由于

Answer 1

av_freep将指针指向要释放的指针;它之后将释放的指针设置为NULL。因此，您需要将其用作

av_freep(&audioBuffer);

或者，您可以使用

av_free(audioBuffer);
audioBuffer = NULL;

等同。

Answer 2

int16_t* audioBuffer=(int16_t*)av_malloc(AVCODEC_MAX_AUDIO_FRAME_SIZE+FF_INPUT_BUFFER_PADDING_SIZE);

这是错误的。不推荐使用AVCODEC_MAX_AUDIO_FRAME_SIZE（在新版本中删除）。使用新的解码API，您可以获得比此更大的解码帧。

static AVFrame frame;

没有。不要在堆栈上分配AVFrame，它会中断。使用avcodec_alloc_frame()（或av_frame_alloc()新版本。）

data_size = av_samples_get_buffer_size(frame.linesize,channel_count,sample_fr,_audio_ccontext->sample_fmt,1);

你意识到你正在编写原始的框架线条（AKA平面尺寸），对吧？我想知道这是不是有意。

另外，sample_fr是什么？从某个地方来说这有些不变吗？只需使用AVFrame.nb_samples。

最后，您的代码明确假设交错音频（即在一个缓冲区中交错的所有通道的音频样本）。现在许多解码器输出平面音频（即，在单独的缓冲器中每个通道的样本）。这可能是导致崩溃的原因，因为av_samples_get_buffer_size会计算所有渠道中所有样本的总大小，而data[0]只会包含第一个渠道的样本。

如果平面音频确实是原因，您应该修改代码以支持它，或者使用libavresample将平面音频转换为交错

av_free随机崩溃应用程序 - FFMPEG C ++

2 个答案: