EXC_BAD_ACCESS while adding data to array, using 2 NSOperations simultaneously

Question

I have:

1) Starting 2 asynchron NSUrlRequests simultaneously

2) As soon as one of the two requests has loaded XML data, an NSOperationQueue is used to start a XML parser. Hereby, the ParseOperations work excatly as in Apple's LazyTableImages example.

InfoRequestHelper.m

// ...

case GetBlogEntries:
{
    BlogParseOperation *parser = [[BlogParseOperation alloc] initWithXMLString:result delegate:self];
    parser.tag = helper.requestTag;

    [queue addOperation:parser]; // this will start the "ParseOperation"

    [parser release];

    break;
}
case GetTweets:
{
    TwitterParseOperation *parser = [[TwitterParseOperation alloc] initWithXMLString:result delegate:self];
    parser.tag = helper.requestTag;

    [queue addOperation:parser]; // this will start the "ParseOperation"

    [parser release];

    break;
}

// ...

3) When parsing is finished parser:didFinishParsing: fires.

InfoRequestHelper.m

- (void)parser:(ParseOperationBase *)parser didFinishParsing:(NSArray *)entries
{
    // Save data, remove completed request from list
    [self.requestsInProgress removeObjectForKey:parser.tag];
    [self.resultObjects addObjectsFromArray:entries];  // <= !!! EXC_BAD_ACCESS !!! here

    // ..    
}

The problem: When the first event arrives here, objects can be added to the array. But when the second arrives, there is an EXC_BAD_ACCESS.

Update: This post looks interesting: http://www.depl0y.com/?p=345, going to try this... — Walter A. Jablonowski, Aug 28 '11 at 15:49

Robin · Answer 1 · 2011-03-29T18:16:36.797

1

Edit: I think you are trying to access the resultObjects variable in both the parsing methods. and resultObjects is a mutableArray. so the problem is when you are trying to add or remove an object from your variable from one function another function is also trying to access it. That will give you an Error. I have learned that too when creating a multi-threaded app.

You can see the apple documentation on thread unsafe classes. and the best way to avoid it is to use a NSArray object instead.

And when you want to add or remove an object from your variable you can first put the contents into a mutable Array and do the manipulation on it and then assign it back to the original object.

I hope this makes sense. I know how hard it would have been to find such errors.

edited Mar 29 '11 at 18:16

answered Feb 09 '11 at 08:23

Robin

10,011
5
49
75

Thank you for your answer. Does not solve the problem ... the app keeps throwing EXC_BAD_ACCESS. – Walter A. Jablonowski Feb 09 '11 at 08:57
try clean build and delete the build folder from your project directory – Robin Feb 09 '11 at 09:01

score 0 · Answer 2 · edited May 23 '17 at 11:55

0

If you are using the same array for data fill up, then I think you are missing a basic threading concept. You must not access variable in write-mode from more than one threads.

You should first acquire a lock on the array and then proceed.

edited May 23 '17 at 11:55

Community

1
1

answered Feb 09 '11 at 12:54

Madhup Singh Yadav

8,110
7
51
84

@Walter\ Jablonowski Can you please post the code by editing this same question? – Madhup Singh Yadav Feb 14 '11 at 04:53

EXC_BAD_ACCESS while adding data to array, using 2 NSOperations simultaneously

2 Answers2