CA, setContents: + Absturz

Christian Kienle · 14. Januar 2009, 15:52

Hallo,

ich habe ein kleines Problem mit Cocoa + Core Animation.

Anbei findet sich ein Projekt, welches mein Problem demonstriert.

Runterladen, auspacken, in Xcode öffnen, kompilieren und starten.

Es sollte ein Fenster aufgehen, welches ein View hat, das das mobile me Icon anzeigt.

Soweit okay.

In AppController.m befindet sich der folgende Code.

Quellcode

- (void)awakeFromNib {
CALayer *layer = [CALayer layer];
NSImage *image = [NSImage imageNamed:NSImageNameDotMac];
[image setSize:NSMakeSize(200, 200)];
NSBitmapImageRep *irep = [[NSBitmapImageRep alloc] initWithData:[image TIFFRepresentation]];
[layer setContents:(id)[irep CGImage]];
[view setWantsLayer:YES];
[view setLayer:layer];
// Entferne die // in der nächsten Zeile und starte das Programm nochmal. => Absturz
//[irep release];
}

Alles anzeigen

Es wird ein Layer erzeugt, eine NSImage-Instanz kreiert, dessen Größe geändert. Anschließend wird eine NSBitmapImageRep erzeugt.

-CGImage liefert ein autoreleased Zeiger auf ein CGImage.
-setContents: schickt ein retain an das übergebene Objekt.

Wird [irep release]; am Ende noch ausgeführt stürzt das Programm ab und mir ist nicht ganz klar wieso.

Projekt

Pascal · 14. Januar 2009, 15:54

Du kriegst ja ein autoreleased Object zurück, das solltest du dann nicht nochmal releasen.

Christian Kienle · 14. Januar 2009, 16:04

Original von Pascal
Du kriegst ja ein autoreleased Object zurück, das solltest du dann nicht nochmal releasen.

Moment: [irep CGImage] liefert ein autoreleased CGImage. Das release ich ja nicht sondern übergebe es dem Layer - mit setContents:

irep ist nicht autoreleased. Wird ja normal mit alloc+init erzeugt...

Christian Kienle · 14. Januar 2009, 16:04

Original von Objcler

Original von Pascal
Du kriegst ja ein autoreleased Object zurück, das solltest du dann nicht nochmal releasen.

Moment: [irep CGImage] liefert ein autoreleased CGImage. Das release ich ja nicht sondern übergebe es dem Layer - mit setContents:

irep ist nicht autoreleased. Wird ja normal mit alloc+init erzeugt...

Und setContents: schickt ja wieder ein retain.

Jens · 14. Januar 2009, 16:05

Original von Pascal
Du kriegst ja ein autoreleased Object zurück, das solltest du dann nicht nochmal releasen.

irep ist doch nicht autoreleased !?

Pascal · 14. Januar 2009, 16:06

Oh, Mist, entschuldige, hatte den Code zu wenig genau und deine Ausführungen zu lange angesehen...

Christian Kienle · 14. Januar 2009, 16:10

irep muss was ganz komisches machen sobald es released wird. Vielleicht liegt es auch am Casten von [irep CGImage] nach id... keine Ahnung... stehe momentan total auf dem Schlauch.

kressevadder · 14. Januar 2009, 16:18

Versuch doch mal spaßeshalber das CGImage zu retainen

Quellcode

[layer setContents:(id) CGImageRetain ( [irep CGImage] )];

kressevadder · 14. Januar 2009, 16:21

Bringt auch nicht weiter...

Christian Kienle · 14. Januar 2009, 16:31

Original von kressevadder
Bringt auch nicht weiter...

Ja das habe ich auch schon probiert.

Das was hilft ist eine Referenz auf irep zu halten, sodass irep nicht released wird. Aber das ist ja nicht Sinn der Sache.

Was könnte NSBitmapImageRep intern denn machen das so extreme Auswirkungen hat? NSBitmapImageRep müsste das "Objekt" - also die CGImage-Struktur löschen - wegwerfen - vernichten, sodass alle Zeiger auf die Struktur invalide sind... Bug?

Apfelbeisser · 14. Januar 2009, 18:40

Das ist ja komisch:

Aktiviert man die Option "wants CALayer" mit dem Interface Builder, dann funktioniert alles!

Christian Kienle · 14. Januar 2009, 20:36

Interessant:

lists.apple.com/archives/cocoa-dev/2008/Jan/msg01474.html

I suspect the problem you're encountering is that the CGImage yielded by a given cycle is sometimes outliving the NSBitmapImageRep it was created from, and whose pixel data the CGImage still references. When CoreGraphics goes to render the CGImage to a destination, it ends up walking off into memory that's been potentially reused for some other purpose.

When using -[NSBitmapImageRep CGImage], it's vitally important to keep the NSBItmapImageRep around for as long as the CGImage is in use. If you fix that problem, your crashes should disappear.

Christian Kienle · 14. Januar 2009, 20:59

Original von Objcler
Interessant:

lists.apple.com/archives/cocoa-dev/2008/Jan/msg01474.html

I suspect the problem you're encountering is that the CGImage yielded by a given cycle is sometimes outliving the NSBitmapImageRep it was created from, and whose pixel data the CGImage still references. When CoreGraphics goes to render the CGImage to a destination, it ends up walking off into memory that's been potentially reused for some other purpose.

When using -[NSBitmapImageRep CGImage], it's vitally important to keep the NSBItmapImageRep around for as long as the CGImage is in use. If you fix that problem, your crashes should disappear.

Nicht das uns das jetzt hilft - aber es scheint ein Bug zu sein... beruhigend.