Нулевой указатель в CFRelease (сообщается статическим anylizer) - PullRequest
Новая
       3

Нулевой указатель в CFRelease (сообщается статическим anylizer)

1 голос
/ 02 марта 2012

Я получил этот код: 

- (void)saveImageWithData:(NSData*)jpeg andDictionary:(NSDictionary*)dicRef andName:(NSString*)name
{
    self.capturedImageName = name;
    self.dict = dicRef;

    NSLog(@"%@",dicRef);

    CGImageSourceRef  source ;
    source = CGImageSourceCreateWithData((__bridge CFDataRef)jpeg, NULL);

    CFStringRef UTI = CGImageSourceGetType(source); //this is the type of image (e.g., public.jpeg)

    NSMutableData *dest_data = [NSMutableData data];


    CGImageDestinationRef destination = CGImageDestinationCreateWithData((__bridge CFMutableDataRef)dest_data,UTI,1,NULL);

    if(!destination) {
        NSLog(@"***Could not create image destination ***");
    }

    //add the image contained in the image source to the destination, overwriting the old metadata with our modified metadata
    CGImageDestinationAddImageFromSource(destination,source,0, (__bridge CFDictionaryRef) dicRef);

    //tell the destination to write the image data and metadata into our data object.
    //It will return false if something goes wrong
    BOOL success = NO;
    success = CGImageDestinationFinalize(destination);

    if(!success) {
        NSLog(@"***Could not create data from image destination ***");
    }

    //now we have the data ready to go, so do whatever you want with it
    //here we just write it to disk at the same path we were passed

    NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES); 
    NSString *documentsDirectory = [paths objectAtIndex:0]; // Get documents folder
    NSString *dataPath = [documentsDirectory stringByAppendingPathComponent:@"ARPictures"];

    NSError *error;
    if (![[NSFileManager defaultManager] fileExistsAtPath:dataPath])
        [[NSFileManager defaultManager] createDirectoryAtPath:dataPath withIntermediateDirectories:NO attributes:nil error:&error]; //Create folder

    NSString *fullPath = [dataPath stringByAppendingPathComponent:[NSString stringWithFormat:@"%@.jpg", name]]; //add our image to the path

    [dest_data writeToFile:fullPath atomically:YES];

    self.img = [[UIImage alloc] initWithData:dest_data]; 
    self.capturedImageData = [[NSData alloc] initWithData:dest_data];

    //cleanup

    CFRelease(destination);
    CFRelease(source);

}

Но когда я запускаю статический анализатор, он говорит мне, что:

Аргумент нулевого указателя при вызове CFRelease

Но в соответствии с моей логикой я должен выпустить CGImageSourceRef, который создается CGImageDestinationCreateWithData .

Кроме того, я думаю, что его освобождение может быть ошибкой, потому что я просто соединяю его, что означает, что дуга все еще управляет этим объектом, так как изначально он был NSMutableData .

Что еще хуже, я прочитал, что CFRelease (Null) не годится.

Я очень запутался, любая помощь будет признательна.

Edit:

Хорошо, я поставил указатели перед отправкой их в CFRelease, они есть! 

2012-03-03 11:35:34.709 programtest[4821:707] <CGImageDestination 0x331790 [0x3f92d630]>

2012-03-03 11:35:34.723 programtest[4821:707] <CGImageSource 0x33ee80 [0x3f92d630]>

Итак, вернемся к первоначальному вопросу, как почему статический анализатор говорит мне, что я посылаю аргумент нулевого указателя? и как мне это исправить / отключить?

спасибо

PD: Возможно ли, что я посылаю не нулевой указатель, а указатель на ноль?

Ответы [ 2 ]

6 голосов
/ 03 марта 2012

CFRelease завершится сбоем, если вы передадите значение NULL, поэтому анализатор предупреждает вас, что пункт назначения и источник там могут быть NULL.Просто добавьте чек для NULL: 

if ( destination != NULL )
    CFRelease(destination);

if ( source != NULL )
    CFRelease(source);
0 голосов
/ 03 марта 2012

Статический анализатор говорит вам, что возможно для одного из аргументов быть нулевым.

Например, если вы разместите возврат после 

NSLog(@"***Could not create image destination ***");

Предупреждение прекращается?

...