Question

我从事Mac OSX的图像处理应用程序，并且存在大量内存泄漏问题（我正在使用ARC）。在主窗口上，我有一个滑块，它可以修改像素值并更新图像。但是当我改变滑块值时，我的应用程序会分配越来越多的内存（几分钟的“滑动”分配甚至高达10 GB的内存！）。

App以灰度级的大图像（30-40mb）运行。我已经使用我的像素创建了3个C阵列并对它们进行了操作但是我调用free但似乎它们没有被释放。

更改滑块的值会触发该方法：

-(void)changeCurrentMinOrMax
{
    imageProcessQueue = dispatch_queue_create("rwt.tz", NULL);
    dispatch_async(imageProcessQueue, ^{
        // Change display range
        [self setDisplayRangeWithMin:_currentMin andMax:_currentMax];
        // Pack pixels into filtered raw image data
        [self packPixelsIntoFilteredImageData];
        // Create filtered image data
        [self createImage:_rawFilteredImageData];
    });
}

这里是被调用方法的实现：

- (void)setDisplayRangeWithMin:(int)min andMax:(int)max
{
    // Calculate number of gray levels
    NSUInteger numberOfGrayLevels = (NSUInteger)pow(2, _bitsPerPixel);
    // Calculate display range
    int range = max - min;
    // Set treshold
    for (unsigned long i = 0; i < numberOFPixels; i++) {
        if (originalPixels[i] < min) {
            pixels[i] = min;
        } else if (originalPixels[i] > max) {
            pixels[i] = max;
        } else {
            pixels[i] = originalPixels[i];
        }
        // map it again into 0-65535 values of gray
        pixels[i] = (UInt16)((numberOfGrayLevels - ZERO_INDEX) * (float)((pixels[i] - min) / (float)range));
    }
}

- (void)packPixelsIntoFilteredImageData
{
    UInt8 *revertedImageDataArray = malloc(sizeOfBitmap);
    unsigned long j = 0;
    for (unsigned long i = 0; i < sizeOfBitmap; i += 2) {
        revertedImageDataArray[i] = (UInt8)((pixels[j] & 0xFF00) >> 8);
        revertedImageDataArray[i+1] = (UInt8)(pixels[j] & 0x00FF);
        j++;
    }

    // pack an array into NSData again
    _rawFilteredImageData = [NSData dataWithBytes:revertedImageDataArray
                                           length:sizeOfBitmap];
    free(revertedImageDataArray);
    revertedImageDataArray = NULL;
}

- (NSImage *)createImage:(NSData *)imgData
{
    _bitsPerComponent = [imgData length] / (_width * _height) * BYTE_SIZE;
    _bitsPerPixel = _bitsPerComponent;
    _bytesPerRow = _width * _bitsPerPixel / BYTE_SIZE;

    CGDataProviderRef provider =
    CGDataProviderCreateWithCFData((CFDataRef)CFBridgingRetain(imgData));
    // which colorspace is better?
    CGColorSpaceRef colorSpace = CGColorSpaceCreateWithName(kCGColorSpaceGenericGrayGamma2_2);
    //CGColorSpaceRef colorSpace = CGColorSpaceCreateDeviceGray();
    CGImageRef imageRef = CGImageCreate((size_t)_width,
                                        (size_t)_height,
                                        _bitsPerComponent,
                                        _bitsPerPixel,
                                        _bytesPerRow,
                                        colorSpace,
                                        kCGImageAlphaNone,
                                        provider,
                                        NULL,
                                        false,
                                        kCGRenderingIntentDefault);
    CGColorSpaceRelease(colorSpace);
    CGDataProviderRelease(provider);

    NSSize size = NSMakeSize((CGFloat) _width, (CGFloat)_height);

    dispatch_async(dispatch_get_main_queue(), ^{
        [self willChangeValueForKey:@"image"];
        _image = [[NSImage alloc] initWithCGImage:imageRef
                                             size:size];
        [self didChangeValueForKey:@"image"];
        CGImageRelease(imageRef);
    });
    return _image;
}

我在dealloc中释放了我的C数组：

- (void)dealloc
{
    // Free memory allocated for C arrays
    if (pixels) {
        free(pixels);
        pixels = NULL;
    }
    if (originalPixels){
        free(originalPixels);
        originalPixels = NULL;
    }
    if (imageDataArray8) {
        free(imageDataArray8);
        imageDataArray8 = NULL;
    }
    // Remove observers
    [self removeObserver:self forKeyPath:@"currentMax"];
    [self removeObserver:self forKeyPath:@"currentMin"];
}

如果我在其他地方使用其他C数组，我可以用同样的方法释放它们。看起来内存没有被释放（即使我关闭当前图像并打开新图像）并且触发了dealloc。你知道发生了什么吗？因为那个我把头发撕掉了！

Answer 1

您有一行代码说：

CGDataProviderRef provider = CGDataProviderCreateWithCFData((CFDataRef)CFBridgingRetain(imgData));

根据过渡到ARC发行说明的Managing Toll-Free Bridging部分告诉我们：

__bridge_retained或CFBridgingRetain会将一个Objective-C指针强制转换为Core Foundation指针，并将所有权转移给您。您有责任调用CFRelease或相关函数来放弃对象的所有权。

这里根本不需要CFBridgingRetain。而且你正在泄露，因为你永远不会发布你传递给CFDataRef的{{1}}（你释放了提供者，而不是CGDataProviderCreateWithCFData）。您也可以让ARC保留所有权并为您清理。这比添加代码更容易（a）跟踪您使用CFDataRef转让所有权的CFDataRef，然后（b）手动CFBridgingRetain CFRelease。您可以按如下方式更改该行，以便根本不转移所有权，ARC将负责其余部分（对于CFDataRef，即：）

NSData

不相关，但CGDataProviderRef provider = CGDataProviderCreateWithCFData((__bridge CFDataRef)imgData);正在返回createImage，这是异步设置的，因此，当您点击_image行时，_image将不会设置return。您可以修复此问题，以便返回适当的值，但事实证明您不使用createImage的返回值，因此将返回类型更改为void可能更容易。但是目前的代码只是有点令人困惑，这表明当你真的没有时，你会回来。

内存泄漏与malloc / free与弧

1 个答案: