结构内存分配(泄漏和崩溃，iPhone上的评测)

Question

我有一个结构定义为：

struct VImage
{
uint8_t *imageData;
int dimensions[3];
...
}

我对代码进行了分析，以查找泄漏和其他问题。考虑以下代码：

for (int i = 0; i < 100; i++)
{
// after we have the image, we use the cheap rgb function to get rgb data
struct VImage *vImage = [SomeClass someFunction:someArgument];
free(vImage);
}

这使我的代码崩溃。我想知道为什么会这样。然而，条件i < 10不会使应用程序崩溃。对仪器进行的剖面分析没有显示有任何泄漏；然而，分配激增，我使用了大量的实时内存；所以我想知道以这种方式释放结构是否正确，或者free是否是一个异步函数，可以释放某人的内存，但不仅仅是我要求它释放内存的时候。

编辑：所以我被要求提供这个someFunction方法的细节，我会的。我认为该方法中没有内存泄漏，但尽管如此，我还是将其放在这里：

+ (struct VImage *)grayScaleImageDataFromImage:(UIImage *)image
{
int imageWidth = image.size.width, imageHeight = image.size.height;
uint32_t *rgbImageData = (uint32_t *) malloc(imageWidth * imageHeight * sizeof(uint32_t));
memset(rgbImageData, 0, imageWidth * imageHeight * sizeof(uint32_t));
CGColorSpaceRef colorSpace = CGColorSpaceCreateDeviceRGB();
// create context with RGBA pixels
CGContextRef context = CGBitmapContextCreate(rgbImageData, imageWidth, imageHeight, 8, imageWidth * sizeof(uint32_t), colorSpace, kCGBitmapByteOrder32Little | kCGImageAlphaNoneSkipLast);
CGContextSetInterpolationQuality(context, kCGInterpolationHigh);
CGContextSetShouldAntialias(context, NO);
// paint to fill pixels array
CGContextDrawImage(context, CGRectMake(0, 0, imageWidth, imageHeight), [image CGImage]);
CGContextRelease(context);
CGColorSpaceRelease(colorSpace);
// now convert to grayscale using CCIR 601 <=> Y' = 0.299 R' + 0.587 G' + 0.114 B'.
uint8_t *grayScaleImageData = (uint8_t *) malloc(imageWidth * imageHeight);
for(int y = 0; y < imageHeight; y++)
{
for(int x = 0; x < imageWidth; x++)
{
uint32_t rgbPixel = rgbImageData[y*imageWidth+x];
grayScaleImageData[y*imageWidth + x] = 0.299*((rgbPixel>>24)&255) + 0.587*((rgbPixel>>16)&255) + 0.114*((rgbPixel>>8)&255);
}
}
struct VImage *vImage = createGrayScaleDescription(grayScaleImageData, imageHeight, imageWidth);
free(rgbImageData);
return vImage;
}
struct VImage *createGrayScaleDescription(uint8_t *grayScaleImage, int height, int width)
{
struct VImage *vImage = malloc(sizeof(struct VImage));
vImage->grayScaleImage = grayScaleImage;
vImage->numberOfDimensions = 2;
vImage->dimensions[0] = height, vImage->dimensions[1] = width, vImage->dimensions[2] = 1;
vImage->isGrayscale = true;
return vImage;
}

所以我所做的就是通过一些计算，使用rgb数据创建一个灰度图像。不过，我认为这里没有任何问题。

Answer 1

首先，您正在泄漏grayScaleImageData:指向的内存

在灰度级ImageDataFromImage中进行

uint8_t *grayScaleImageData = (uint8_t *) malloc(imageWidth * imageHeight);

这很好，然后在createGrayScaleDescription中将vImage->grayScaleImage设置为指向该内存，这也很好。

但是，在1到100的循环中，您释放了vImage，而没有首先释放vImage->grayScaleImage指向的内存。这是内存泄漏。

我不认为这会破坏你的应用程序，但这是你需要修复的问题。没有其他东西向我扑来，但我会仔细看看。

编辑：仔细检查，您似乎超出了分配的缓冲区grayScaleImageData。

你用

uint8_t *grayScaleImageData = (uint8_t *) malloc(imageWidth * imageHeight);

但是你用填充缓冲区

uint32_t rgbPixel = rgbImageData[y*imageWidth+x];
grayScaleImageData[y*imageWidth + x] = 0.299*((rgbPixel>>24)&255) + 0.587*((rgbPixel>>16)&255) + 0.114*((rgbPixel>>8)&255);

您已经分配了(imageWidth*imageHeight)8位uint，但随后您用32位uint填充它。

也许这就是导致你崩溃的原因？也许试着把malloc改成

uint32_t *grayScaleImageData = (uint32_t *) malloc(imageWidth * imageHeight * sizeof (uint32_t));

就像你上面为rgbImageData所做的那样？或者在将其设置为grayScaleImageData:之前，将位偏移计算的结果转换为uint8_t

uint32_t rgbPixel = rgbImageData[y*imageWidth+x];
grayScaleImageData[y*imageWidth + x] = (uint8_t)(0.299*((rgbPixel>>24)&255) + 0.587*((rgbPixel>>16)&255) + 0.114*((rgbPixel>>8)&255));