Skip to content

Commit b108585

Browse files
committed
Fix stack overflows for jackson-module-scala benchmarks
1 parent c42fcc1 commit b108585

File tree

1 file changed

+7
-1
lines changed
  • jsoniter-scala-benchmark/shared/src/main/scala/com/github/plokhotnyuk/jsoniter_scala/benchmark

1 file changed

+7
-1
lines changed

jsoniter-scala-benchmark/shared/src/main/scala/com/github/plokhotnyuk/jsoniter_scala/benchmark/JacksonSerDesers.scala

+7-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,13 @@ object JacksonSerDesers {
3030
.configure(StreamWriteFeature.USE_FAST_DOUBLE_WRITER, true)
3131
.configure(StreamReadFeature.USE_FAST_BIG_NUMBER_PARSER, true)
3232
.configure(StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION, true)
33-
.streamReadConstraints(StreamReadConstraints.builder().maxNumberLength(Int.MaxValue).build()) /* WARNING: It is an unsafe option for open systems */
33+
.streamReadConstraints(StreamReadConstraints.builder()
34+
.maxNumberLength(Int.MaxValue) // WARNING: It is an unsafe option for open systems
35+
.maxNestingDepth(Int.MaxValue) // WARNING: It is an unsafe option for open systems
36+
.build())
37+
.streamWriteConstraints(StreamWriteConstraints.builder()
38+
.maxNestingDepth(Int.MaxValue) // WARNING: It is an unsafe option for open systems
39+
.build())
3440
.build()
3541
val builder = JsonMapper.builder(jsonFactory)
3642
.addMixIn(classOf[GeoJSON.GeoJSON], classOf[MixIn])

0 commit comments

Comments
 (0)