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

Author: “saiHemak”

Foundation/NSJSONSerialization.swift

@@ -702,21 +702,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

@@ -264,7 +264,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 {
@@ -598,6 +598,7 @@ extension TestNSJSONSerialization {
             ("test_invalidJsonObjectToStreamBuffer", test_invalidJsonObjectToStreamBuffer),
             ("test_jsonObjectToOutputStreamInsufficeintBuffer", test_jsonObjectToOutputStreamInsufficeintBuffer),
             ("test_booleanJSONObject", test_booleanJSONObject),
+            ("test_serializeDecimalNumberJSONObject", test_serializeDecimalNumberJSONObject),
         ]
     }
 
@@ -941,6 +942,21 @@ extension TestNSJSONSerialization {
         XCTAssertTrue(JSONSerialization.isValidJSONObject([true]))
     }
 
+    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")
+        }
+    }
+
     private func createTestFile(_ path: String,_contents: Data) -> String? {
         let tempDir = NSTemporaryDirectory() + "TestFoundation_Playground_" + NSUUID().uuidString + "/"
         do {