Question

I’m doing an iPhone application which uses a navigation control to browse through some data. This data is stored in a sqlite database and I have a class which gets it and returns it in a NSMutableArray of NSStrings.

The problem is that in the first screen of the navigation everything works prefectly, but in the second screen (another view which is pushed) the same code fails because the NSMutableArray gets corrupted. In the debugger I can see that it is returned correctly, but when it’s time to use it the pointers have become corrupted and the application crashes.

I have put breakpoints in all my functions, and I can’t see anywhere where it can get corrupted. And as the first view, which uses the same exact code, even accesing the same eact tables, works correctly I don’t really know where to look.

If anyone want to have a look at the code I have uploaded it to my site: http://sachafuentes.com/iBacus.zip

Thanks in advance.

UPDATE:

The problem lies in the function where I get the data, which looks like (this is a simplified version with some pseudo-code).

-(NSMutableArray *) getPaises {  NSMutableArray * paises;  paises = [[NSMutableArray alloc] init];  while( get new row ) {   NSString *aPais = get the value;   [paises addObject:aPais];   [aPais release];  }  return paises; } 

If I comment out [aPais release] everything works, but to me this looks like a memory leak, as the NSString won’t be released.

Answer 1

Look for wrong memory management, that’s the likeliest cause for crashes. I think you release your objects somewhere where you shouldn’t and therefore you have dangling pointers in the array.

Update1

[aPais release] is wrong, as you don’t retain it anywhere in this method. Returned values should always be autoreleased. You should have the same amount of retain as release in your code.

(Some people argue that a retain can also be an alloc init, but I always try to use alloc init autorelease, but this is a matter of style.)

BTW

You should autorelease your array, you’re only retaining it here with [[alloc] init].