Fix for [SR-1250] NSJSONSerialization emits non-floating-point numbers as Double

Author: “saiHemak”

Foundation/NSJSONSerialization.swift

@@ -716,21 +716,16 @@ private struct JSONReader {
                 defer { intEndPointer.deallocate(capacity: 1) }
                 let doubleEndPointer = UnsafeMutablePointer<UnsafeMutablePointer<Int8>?>.allocate(capacity: 1)
                 defer { doubleEndPointer.deallocate(capacity: 1) }
-                
                 let intResult = strtol(startPointer, intEndPointer, 10)
                 let intDistance = startPointer.distance(to: intEndPointer[0]!)
                 let doubleResult = strtod(startPointer, doubleEndPointer)
                 let doubleDistance = startPointer.distance(to: doubleEndPointer[0]!)
-                
+                let roundedDouble = doubleResult.rounded()
                 guard intDistance > 0 || doubleDistance > 0 else {
                     return nil
                 }
-                
-                if intDistance == doubleDistance {
-                    return (intResult, intDistance)
-                }
-                guard doubleDistance > 0 else {
-                    return nil
+                if doubleResult == roundedDouble || doubleResult == 0.0 {
+                    return (Int(doubleResult), doubleDistance)
                 }
                 return (doubleResult, doubleDistance)
             }

TestFoundation/TestNSJSONSerialization.swift

@@ -468,7 +468,7 @@ extension TestNSJSONSerialization {
                 XCTAssertEqual(result?[1] as? Int,       -1)
                 XCTAssertEqual(result?[2] as? Double,   1.3)
                 XCTAssertEqual(result?[3] as? Double,  -1.3)
-                XCTAssertEqual(result?[4] as? Double,  1000)
+                XCTAssertEqual(result?[4] as? Int,  1000)
                 XCTAssertEqual(result?[5] as? Double, 0.001)
             }
         } catch {
@@ -871,6 +871,7 @@ extension TestNSJSONSerialization {
             ("test_jsonObjectToOutputStreamInsufficientBuffer", test_jsonObjectToOutputStreamInsufficientBuffer),
             ("test_booleanJSONObject", test_booleanJSONObject),
             ("test_serialize_dictionaryWithDecimal", test_serialize_dictionaryWithDecimal),
+            ("test_serializeDecimalNumberJSONObject", test_serializeDecimalNumberJSONObject),
         ]
     }
 
@@ -1214,7 +1215,22 @@ extension TestNSJSONSerialization {
         XCTAssertTrue(JSONSerialization.isValidJSONObject([true]))
     }
 
-    fileprivate func createTestFile(_ path: String,_contents: Data) -> String? {
+    func test_serializeDecimalNumberJSONObject() {
+        let decimalArray = "[12.1,10.0,0.0,0.0001,20]"
+        do {
+            let data = decimalArray.data(using: String.Encoding.utf8)
+            let result = try JSONSerialization.jsonObject(with: data!, options: []) as? [Any]
+            XCTAssertEqual(result?[0] as! Double,12.1)
+            XCTAssertEqual(result?[1] as! Int,10)
+            XCTAssertEqual(result?[2] as! Int, 0)
+            XCTAssertEqual(result?[3] as! Double,0.0001)
+            XCTAssertEqual(result?[4] as! Int,20)
+        } catch {
+            XCTFail("Failed during serialization")
+        }
+    }
+
+   fileprivate func createTestFile(_ path: String,_contents: Data) -> String? {
         let tempDir = NSTemporaryDirectory() + "TestFoundation_Playground_" + NSUUID().uuidString + "/"
         do {
             try FileManager.default.createDirectory(atPath: tempDir, withIntermediateDirectories: false, attributes: nil)