Memory Leak NSURL

Juji · 25. August 2011, 10:27

Hallo,
es gibt zwar schon ein Thema zu Memory Leaks mit NSURL, aber das hat mir leider nicht weiter geholfen.
Ich will einen NSMutableString nutzen um eine NSURL zu erstellen. Dazu tue ich folgendes:

Quellcode

//link ist vom Typ NSURL*
//stringForUri ist vom Typ NSMutableString
currentRssItem.link = [NSURL URLWithString:stringForUri]; //is leaking

Es handelt sich um einen Parser, daher wird diese Zeile oft aufgerufen - dementsprechend erzeugt es ein gehöriges Memory Leak... Ursprünglich war meine Vermutung, dass stringForUri leakt, aber ich kann den Fehler einfach nicht finden. Hier einmal die Zeilen, in dennen ich mit stringForUri arbeite:

Quellcode

//Beginn des Parserelements
stringForUri = [NSMutableString stringWithCapacity:30 ]; //autoreleased
[stringForUri setString:@""];
//Found Characters
[stringForUri appendString:[NSString stringWithFormat:@"%@",cleanString]];
//Ende des Elements
currentRssItem.link = [NSURL URLWithString:stringForUri];

Habe ich etwas entscheidendes übersehen?
Ich wäre sehr dankbar für Eure Hilfe!

Viele Grüße,
Juji

scriptedSheep · 25. August 2011, 10:32

Ich glaube du brauchst doch da gar keinen NSMUtableString, oder? Ich würde das einfach so machen:

Quellcode

stringURI = [NSString stringWithFormat:@"%@ %@", myFirstPart, mySecondPart];

macmoonshine · 25. August 2011, 10:41

Das sieht erstmal gut aus. So ganz glaube ich aber nicht an einen Leak in NSURL. Du könntest mal probieren, ob der Leak auch auftritt, wenn Du den Mutablestring in einen String umwandelst:

Quellcode

currentRssItem.link = [NSURL URLWithString:[[stringForUri copy] autorelease]];

Aber wie schon angemerkt, brauchst Du wirklich einen Mutablestring?

Juji · 25. August 2011, 10:41

Hallo scriptedSheep,
Dachte ich auch zuerst, aber wenn der Parser über das Dokument läuft,dann parst er nicht immer ein vollständiges Element zwischen den <element> ... </element>-Tags. Gerade bei Sonderzeichen macht er dann einen Bruch und parst mit einem erneuten Aufruf der Found-Characters-Methode weiter. Das heißt, ich muss mir den stringForUri aus den einzelnen Teilen zusammensetzten. und wenn dann das Endelement kommt </element> dann ist stringForUi fertig und ich kann in in link-element speichern...

scriptedSheep · 25. August 2011, 10:46

Quellcode

strinForURI = [NSString stringWithFormat:@"%@%@",stringForURI,cleanString];

? Ich mag manche der Mutable-Sachen einfach nicht

Juji · 25. August 2011, 11:05

Hallo,
also jetzt bin ich gänzlich verwirrt. Ist mein Instruments-Leaks kaputt?

Ich habe nun stringForUri als NSString gewählt und mal deine Zeile, scripptedSheep, benutzt, sowohl diese als auch die Zuweisung zur URL leaked... Der Code sieht dann ja folgendermaßen aus:

Quellcode

//Beginn Tag
stringForUri = [NSString stringWithFormat:@"" ]; //autoreleased
//Found Charakter
// trim the string to remove whitespaces like blanks and line feeds
//string > NSString*
NSString* helpstring = [NSString stringWithFormat:@"%@",[string stringByTrimmingCharactersInSet: [NSCharacterSet whitespaceCharacterSet]]];
NSString* cleanString = [NSString stringWithFormat:@"%@",[helpstring stringByReplacingOccurrencesOfString:@"\n" withString:@" " ]];
stringForUri = [NSString stringWithFormat:@"%@%@",stringForUri,cleanString];
// End Tag
currentRssItem.link = [NSURL URLWithString:stringForUri];

Alles anzeigen

Ich habe gerade noch mal die speicheraddressen von cleanString, stringForUri, etc angeschaut, aber es sind alles unterschiedliche. Und alles ist Autoreleased...

Thallius · 25. August 2011, 11:06

Kann es sein, dass du gar keinen Autorelease Pool hast ? Also läuft das Ganze vlt in einem eigenen Thread ?

Gruß

Claus

scriptedSheep · 25. August 2011, 11:17

Das wärs ja

Da kommen doch eigentlich immer abertausende Warnings in der Konsole^^

Juji · 25. August 2011, 11:22

Ähm, jetzt bin ich überfragt. Der Parser ist in einer eigenen Klasse. Diese wird vom ViewController aufgerufen. Sagen wir so - ich habe noch nie einen eigenen Autorelease Pool erstellt.. Ich nehme an, das ist ungünstig?

scriptedSheep · 25. August 2011, 11:27

Naja, ich glaube nicht, dass du das in einem eigenen Thread hast, ersten scheinst du nicht zu wissen was du meinst und zweitens kommen dann immer massenhaft Logs in der Konsole von wegen das Sachen leaken weil kein Pool da ist, so blöd die zu übersehen kann man eigentlich nicht sein

Juji · 25. August 2011, 11:34

Okay. Dann arbeite ich nur in einem Thread. Bisher hat das mit dem Autorelease auch sehr gut funktioniert. Das ist halt die einzige Stelle an der die App leakt.

scriptedSheep · 25. August 2011, 11:35

Mal den Analyzer drüber laufen lassen?

Juji · 25. August 2011, 11:52

Jup, habe ich schon gemacht. Ich habe auch schon NSZombieEnabled mitlaufen lassen, falls mir was abstützt.
Bei Instruments lasse ich mir dann den Call Tree der Leaks anzeigen, mit Invert Call Tree und Hide System Libaries. Ich zweifle gerade an der ordnungemäßen Nutzung

Juji · 26. August 2011, 12:55

Hey,
ich habe da jetzt noch mal ne Frage. Ich habe die Zeile jetzt folgendermaßen umgeschrieben

Quellcode

NSString * copy = [NSString stringWithString:stringForUri];
currentRssItem.link = [[NSURL alloc] initWithString:copy];

Nun wird mir nur [NSURL alloc] unterstrichen. Bedeutet dies, dass nur dieser Teil leaked? Oder leaked trotzdem die ganze Zeile?
Ich gebe später currentRSSITem.link wieder frei.

Danke,
Juji

Thallius · 26. August 2011, 12:57

Wenn link ein retain property ist dann hast du hier einen doppelten retain (einmal den von alloc/init und einmal den des link selber) Beim Freigeben des link wird aber nur einer freigegeben. Du hast also ein leak. In dem Fall must du entweder

Quellcode

link=[[NSURL alloc] initWithString:copy];

oder schöner

Quellcode

currentResItem.link=[[[NSURL alloc] initWithString:copy] autorelease];

Gruß

Claus

Juji · 26. August 2011, 13:07

Hallo,
Ja, der Link ist ein retain Property. Ich habe nun das Autorelease angefügt und habe deutlich weniger Leaks

Wenn ich die Klasse SimpleRSSContent (von diesem Typ ist currentRSSItem) intialisiere, reserviere ich keinen Speicher für den link und gebe daher nichts frei beim dealloc.

Trotzdem ist mir etwas schleierhaft, warum NSString * copy = [NSString stringWithString:stringForUri]; leaked. Hier wird mit wieder der rechte Teil der Zuweisung unterstreichen. Nebenbei: gibt die Unterstreichung einen Hinweis auf die genaue Position des Leaks?

Danke!

Thallius · 26. August 2011, 13:15

also das stringWithString leaked definitiv nicht. Das muss was anderes sein. Wenn Du magst kannst Du mir das Projekt gerne mal kurz schicken und sehe mal drüber. Manchmal ist man ja auch einfach betriebsblind und ich habe eh gerade Feierabend gemacht und bereits mich auf meinen Urlaub morgen vor.

Gruß

Claus