ios - malloc 错误 - 已释放对象的校验和不正确 - 对象可能在被释放后被修改

Question

我正在尝试获取 NSData 对象的子数据，同时根据我的个人需要获取多个字节的某个值。

实际上这会影响 .wav 声音文件的音量。

但是在几次调用以下函数后，我在 malloc 语句中得到了 malloc 错误。

+(NSData *) subDataOfData: (NSData *) mainData withRange:(NSRange) range volume (CGFloat) volume
{
    // here is the problematic line:
    Byte * soundWithVolumeBytes = (Byte*)malloc(range.length); 
    Byte * mainSoundFileBytes =(Byte *)[mainData bytes];

    for (int i=range.location ; i< range.location + range.length; i=i+2)
    {
        // get the original sample
        int16_t sampleInt16Value = 0;
        sampleInt16Value = (sampleInt16Value<<8) + mainSoundFileBytes[i+1];
        sampleInt16Value = (sampleInt16Value<<8) + mainSoundFileBytes[i];

        //multiple sample 
        sampleInt16Value*=volume;

        //store the sample
        soundWithVolumeBytes[i] = (Byte)sampleInt16Value;
        soundWithVolumeBytes[i+1] =(Byte) (sampleInt16Value>>8);

    }


    NSData * soundDataWithVolume = [[NSData alloc] initWithBytes:soundWithVolumeBytes length:range.length];
    free(soundWithVolumeBytes);

    return [soundDataWithVolume autorelease];

}

谢谢 ！！

Answer 1

当 的值range.location不为零时，您的for循环会修改超出分配范围的位置。这些行

soundWithVolumeBytes[i] = ...
soundWithVolumeBytes[i+1] = ...

写入位置从range.location到range.location+range.length-1，但分配的范围仅从零到range.length。您需要将行更改为

soundWithVolumeBytes[i-range.location] = ...
soundWithVolumeBytes[i+1-range.location] = ...

此外，由于您增加了 2，因此最后一次迭代可能会访问缓冲区末尾之后的一个字节，以防万一range.location+range.length。