Fix leak of url sessions in GoogleUtilities

Author: morganchen12

GoogleUtilities/Network/GULNetworkURLSession.m

@@ -32,7 +32,7 @@ @implementation GULNetworkURLSession {
 #pragma clang diagnostic ignored "-Wunguarded-availability"
   /// The session configuration. NSURLSessionConfiguration' is only available on iOS 7.0 or newer.
   NSURLSessionConfiguration *_sessionConfig;
-#pragma pop
+#pragma clang diagnostic pop
 
   /// The path to the directory where all temporary files are stored before uploading.
   NSURL *_networkDirectoryURL;
@@ -158,8 +158,7 @@ - (NSString *)sessionIDFromAsyncPOSTRequest:(NSURLRequest *)request
   }
 
   // Save the session into memory.
-  NSMapTable *sessionIdentifierToFetcherMap = [[self class] sessionIDToFetcherMap];
-  [sessionIdentifierToFetcherMap setObject:self forKey:_sessionID];
+  [self setSessionInFetcherMap:session forSessionID:_sessionID];
 
   _request = [request copy];
 
@@ -201,8 +200,7 @@ - (NSString *)sessionIDFromAsyncGETRequest:(NSURLRequest *)request
   }
 
   // Save the session into memory.
-  NSMapTable *sessionIdentifierToFetcherMap = [[self class] sessionIDToFetcherMap];
-  [sessionIdentifierToFetcherMap setObject:self forKey:_sessionID];
+  [self setSessionInFetcherMap:session forSessionID:_sessionID];
 
   _request = [request copy];
 
@@ -284,6 +282,12 @@ - (void)URLSession:(NSURLSession *)session
   // Try to clean up stale files again.
   [self maybeRemoveTempFilesAtURL:_networkDirectoryURL
                      expiringTime:kGULNetworkTempFolderExpireTime];
+
+  // Invalidate the session only if it's owned by this class.
+  NSString *sessionID = session.configuration.identifier;
+  if ([sessionID hasPrefix:kGULNetworkBackgroundSessionConfigIDPrefix]) {
+    [session finishTasksAndInvalidate];
+  }
 }
 
 - (void)URLSession:(NSURLSession *)session
@@ -651,6 +655,28 @@ - (void)URLSession:(NSURLSession *)session
 
 #pragma mark - Helper Methods
 
+- (void)setSessionInFetcherMap:(NSURLSession *)session forSessionID:(NSString *)sessionID {
+  NSURLSession *existingSession = [self sessionFromFetcherMapForSessionID:sessionID];
+  if (existingSession) {
+    // Invalidating doesn't seem like the right thing to do here since it may cancel an active
+    // background transfer if the background session is handling multiple requests. The old
+    // session will be dropped from the map table, but still complete its request.
+    NSString *message = [NSString stringWithFormat:@"Discarding URL session: %@", existingSession];
+    [_loggerDelegate GULNetwork_logWithLevel:kGULNetworkLogLevelInfo
+                                 messageCode:kGULNetworkMessageCodeURLSession019
+                                     message:message];
+  }
+  if (session) {
+    [[[self class] sessionIDToFetcherMap] setObject:session forKey:sessionID];
+  } else {
+    [[[self class] sessionIDToFetcherMap] removeObjectForKey:sessionID];
+  }
+}
+
+- (nullable NSURLSession *)sessionFromFetcherMapForSessionID:(NSString *)sessionID {
+  return [[[self class] sessionIDToFetcherMap] objectForKey:sessionID];
+}
+
 - (void)callCompletionHandler:(GULNetworkURLSessionCompletionHandler)handler
                  withResponse:(NSHTTPURLResponse *)response
                          data:(NSData *)data
@@ -669,6 +695,7 @@ - (void)callCompletionHandler:(GULNetworkURLSessionCompletionHandler)handler
   }
 }
 
+// Always use the request parameters even if the default session configuration is more restrictive.
 - (void)populateSessionConfig:(NSURLSessionConfiguration *)sessionConfig
                   withRequest:(NSURLRequest *)request API_AVAILABLE(ios(7.0)) {
   sessionConfig.HTTPAdditionalHeaders = request.allHTTPHeaderFields;

GoogleUtilities/Network/Private/GULNetworkMessageCode.h

@@ -41,4 +41,5 @@ typedef NS_ENUM(NSInteger, GULNetworkMessageCode) {
   kGULNetworkMessageCodeURLSession016 = 901016,  // I-NET901016
   kGULNetworkMessageCodeURLSession017 = 901017,  // I-NET901017
   kGULNetworkMessageCodeURLSession018 = 901018,  // I-NET901018
+  kGULNetworkMessageCodeURLSession019 = 901019,  // I-NET901019
 };