Android JNI中的Native Crash SIGSEGV

时间:2017-01-11 18:18:57

标签: java android c++ java-native-interface superpowered

我在我的应用中随机获得了一个Native Crash signal 11 (SIGSEGV), code 1 (SEGV_MAPERR)。 App循环遍历文件并在C ++代码中分析它们并返回一个浮点数组。这是在AsyncTask中完成的,它在处理文件时运行一段时间。我在导致崩溃的代码中做错了什么?或者它是一个超级能源问题?谢谢。

这是AsyncTask doInBackground函数:

protected String doInBackground(Object... urls) {

            for (int i = 0; i < songFiles.size(); i++) {
                SongFile temp = songFiles.get(i);
                try {
                    float[] f = Analyser.getInfo(temp.getPath());
                        if (f != null && f.length > 1) {
                            ...save to DB
                        }
                    }

                } catch (Exception e) {
                }
            }
            return "";
        } 

Java和C ++代码之间的函数:

extern "C" JNIEXPORT jfloatArray Java_com_superpowered_SuperpoweredPlayer_getInfo(JNIEnv *env, jobject instance,jstring filepath) {
    jfloatArray ret;
    char *Path= (char *) env->GetStringUTFChars(filepath, JNI_FALSE);

    ret = (jfloatArray)env->NewFloatArray(2);

    float *values = superpoweredPlayer->getKey(Path);

    env->SetFloatArrayRegion(ret, 0, 2, values);
    env->ReleaseStringUTFChars(filepath, Path);
    return ret;

}

C ++函数getKey:

float *SuperpoweredPlayer::getKey(char *url) {

    SuperpoweredDecoder *decoder = new SuperpoweredDecoder();

    //decoder initialize from the URL input
    const char *openError = decoder->open(url, false, 0, 0);
    if (openError) {
        delete decoder;
        return NULL;
    };

    // Create the analyzer.
    SuperpoweredOfflineAnalyzer *analyzer = new SuperpoweredOfflineAnalyzer(decoder->samplerate, 0, decoder->durationSeconds);

    // Create a buffer for the 16-bit integer samples coming from the decoder.
    short int *intBuffer = (short int *)malloc(decoder->samplesPerFrame * 2 * sizeof(short int) + 16384);
    // Create a buffer for the 32-bit floating point samples required by the effect.
    float *floatBuffer = (float *)malloc(decoder->samplesPerFrame * 2 * sizeof(float) + 1024);

    // Processing.
    while (true) {

        // Decode one frame. samplesDecoded will be overwritten with the actual decoded number of samples.
        unsigned int samplesDecoded = decoder->samplesPerFrame;
        if (decoder->decode(intBuffer, &samplesDecoded) == SUPERPOWEREDDECODER_ERROR) break;
        if (samplesDecoded < 1) break;

        // Convert the decoded PCM samples from 16-bit integer to 32-bit floating point.
        SuperpoweredShortIntToFloat(intBuffer, floatBuffer, samplesDecoded);

        // Submit samples to the analyzer.
        analyzer->process(floatBuffer, samplesDecoded);

        // Update the progress indicator.
        // progress = (double)decoder->samplePosition / (double)decoder->durationSamples;
    };



    // Get the result.
    unsigned char *averageWaveform = NULL, *lowWaveform = NULL, *midWaveform = NULL, *highWaveform = NULL, *peakWaveform = NULL, *notes = NULL;
    int waveformSize, overviewSize, keyIndex;
    char *overviewWaveform = NULL;
    float loudpartsAverageDecibel, peakDecibel, bpm, averageDecibel, beatgridStartMs = 0;
    analyzer->getresults(&averageWaveform, &peakWaveform, &lowWaveform, &midWaveform, &highWaveform, &notes, &waveformSize, &overviewWaveform, &overviewSize, &averageDecibel, &loudpartsAverageDecibel, &peakDecibel, &bpm, &beatgridStartMs, &keyIndex);

    float *ret;
    ret=(float*)malloc(2*sizeof(float));
    ret[0] = bpm;
    ret[1] = keyIndex;

    // Cleanup.
    delete decoder;
    delete analyzer;
    free(intBuffer);
    free(floatBuffer);

    // Done with the result, free memory.
    if (averageWaveform) free(averageWaveform);
    if (lowWaveform) free(lowWaveform);
    if (midWaveform) free(midWaveform);
    if (highWaveform) free(highWaveform);
    if (peakWaveform) free(peakWaveform);
    if (notes) free(notes);
    if (overviewWaveform) free(overviewWaveform);

    return ret;

}

1 个答案:

答案 0 :(得分:3)

请仔细检查您是否正在释放所有内存,您正在分配AND以相反的顺序释放内存/删除对象,例如如果你创建了对象A然后是B,你应该删除B然后删除A.

  1. 内存泄漏:Java_com_superpowered_SuperpoweredPlayer_getInfo()
  2. 中未释放float* values
  3. 我会释放分析器中的内存 - >首先是getresults(),然后是floatBuffer和intBuffer,然后删除分析器然后解码器。
  4. 你确定使用free()释放在analyzer-&gt; getresults()中分配的内存是否正确(且安全),因为superpowered中的free可能与你的代码中的free不同(如果其中任何一个是静态链接的话)标准C库)?