sync: Handle missing error string for abnormal statuses (#1402)

Author: russellhancox

Source/santasyncservice/SNTSyncStage.mm

@@ -195,20 +195,17 @@ - (NSData *)dataFromRequest:(NSURLRequest *)request
 
   // If the final attempt resulted in an error, log the error and return nil.
   if (response.statusCode != 200) {
-    long code;
-    NSString *errStr;
-    if (response.statusCode > 0) {
-      code = response.statusCode;
-      errStr = [NSHTTPURLResponse localizedStringForStatusCode:response.statusCode];
-    } else {
+    long code = response.statusCode;
+    NSString *errStr = [NSHTTPURLResponse localizedStringForStatusCode:response.statusCode];
+    if (requestError.localizedDescription) {
       code = (long)requestError.code;
       errStr = requestError.localizedDescription;
     }
     LOGE(@"HTTP Response: %ld %@", code, errStr);
     if (error != NULL) {
       *error = [NSError errorWithDomain:@"com.google.santa.syncservice"
                                    code:code
-                               userInfo:@{NSLocalizedDescriptionKey : errStr}];
+                               userInfo:@{NSLocalizedDescriptionKey : errStr ?: @""}];
     }
     return nil;
   }