Skip to content
This repository has been archived by the owner on Aug 24, 2019. It is now read-only.

Fix for possible infinite loop for persistent store in bad state #39

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

zbigy
Copy link

@zbigy zbigy commented Dec 7, 2015

There exists an issue that if the NSPersistentStroeCoordinator failed to add a persistent store, then the coordinator was in a bad state. In this case, the code just continued to use that coordinator even though it was in a bad state, and then performing fetches or saves on the coordinator would cause a crash.

The fix only sets the static coordinator upon successfully adding a persistentStore, otherwise leaving it as nil. The second part of this is making it so that if the coordinator is nil, we still allow the code to run to make a new one to it can try and make it again.

zbigy added 5 commits December 4, 2015 18:01
fixing bug if persistent store in bad state
This allows for subclassing of the persistent store coordinator and is
consistent with it’s use in the rest of the file (where self is always
used)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant