Updating JSON parsing depth based on fact that fails on APPLE under debug around 80.

Author: stgates

Release/src/json/json_parsing.cpp

@@ -189,7 +189,16 @@ class JSON_Parser
     const typename std::char_traits<CharType>::int_type m_eof;
     size_t m_currentColumn;
     size_t m_currentParsingDepth;
+    
+// On APPLE debug overflow happens around 80.
+// The DEBUG macro is defined in XCode but we don't in our CMakeList
+// so for now we will keep the same on debug and release. In the future
+// this can be increase on release if necessary.
+#if defined(__APPLE__)
+    static const size_t maxParsingDepth = 64;
+#else
     static const size_t maxParsingDepth = 128;
+#endif
 };
 
 template <typename CharType>

Release/tests/Functional/json/parsing_tests.cpp

@@ -1,4 +1,4 @@
-/***
+/***
 * ==++==
 *
 * Copyright (c) Microsoft Corporation. All rights reserved. 
@@ -490,12 +490,20 @@ utility::string_t make_deep_json_string(size_t depth)
 
 TEST(deeply_nested)
 {
+#if defined(__APPLE__)
+    size_t safeDepth = 64;
+    size_t overDepth = 65;
+#else
+    size_t safeDepth = 128;
+    size_t overDepth = 129;
+#endif
+                                     
     // This should parse without issues:
-    auto strGood = make_deep_json_string(128);
+    auto strGood = make_deep_json_string(safeDepth);
     json::value::parse(strGood);
 
     // But this one should throw:
-    auto strBad = make_deep_json_string(129);
+    auto strBad = make_deep_json_string(overDepth);
     VERIFY_PARSING_THROW(json::value::parse(strBad));
 }