Fixed memory leak in NSURLSession handling#2794
Conversation
There was a problem hiding this comment.
+ (NSURLSession * nonnull)sessionWithConfiguration:(NSURLSessionConfiguration * nonnull)configuration
This method actually requires an argument according to iOS 9 sdk nullability
|
This is a real bug that should be fixed. Apple's docs say "The session object keeps a strong reference to the delegate until your app explicitly invalidates the session. If you do not invalidate the session, your app leaks memory." alistra's comments are correct but the fix is trivial. Just add the default configuration when creating the session. I believe that @kcharwood wrote this code so maybe you want to fix this if @olegnaumenko isn't responding. |
|
I'll take a look at this soon. Thanks @phoney |
|
Rebased with some additional changes in 09f6cc1 |
kcharwood
changed the title
4 participants
No description provided.