AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
5b1ae4b285
skia2/tests/sksl/errors/ArrayOfInvalidSize.glsl

22 lines
752 B
Plaintext
Raw Normal View History

Improve error-checking during array-size conversion. The fuzzer noticed insufficient guards in IndexExpression::Convert when converting an array size from an IntLiteral to a SKSL_INT. We had code in IRGenerator which did this properly, so I moved our array-size conversion logic into SkSLType and had IndexExpression share it. Also, a variety of tests around similar error conditions were added. Change-Id: I51529dea25f9029f81ae236511610069d66be29f Bug: oss-fuzz:37462 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/441236 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
2021-08-23 13:24:33 +00:00
### Compilation failed:
error: 1: array size must be positive
error: 2: array size must be positive
error: 3: array size must be positive
error: 4: expected 'int', but found 'float'
error: 5: array size is too large
error: 6: expected 'int', but found 'bool'
error: 7: expected 'int', but found 'bool'
error: 8: expected 'int', but found 'int2'
SkSL: Turn DSL parser off again Several fuzzer issues, and one Chromium issue that's blocking the roll. Bug: chromium:1246795 Bug: skia:12423 Change-Id: I00370b74569b447e543d9a1f22c588eb493063da Reviewed-on: https://skia-review.googlesource.com/c/skia/+/445960 Reviewed-by: John Stiles <johnstiles@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: Brian Osman <brianosman@google.com>
2021-09-07 13:36:08 +00:00
error: 9: array must have a size
Improve error-checking during array-size conversion. The fuzzer noticed insufficient guards in IndexExpression::Convert when converting an array size from an IntLiteral to a SKSL_INT. We had code in IRGenerator which did this properly, so I moved our array-size conversion logic into SkSLType and had IndexExpression share it. Also, a variety of tests around similar error conditions were added. Change-Id: I51529dea25f9029f81ae236511610069d66be29f Bug: oss-fuzz:37462 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/441236 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
2021-08-23 13:24:33 +00:00
error: 10: array size must be positive
error: 11: array size must be positive
error: 12: array size must be positive
SkSL: Turn DSL parser off again Several fuzzer issues, and one Chromium issue that's blocking the roll. Bug: chromium:1246795 Bug: skia:12423 Change-Id: I00370b74569b447e543d9a1f22c588eb493063da Reviewed-on: https://skia-review.googlesource.com/c/skia/+/445960 Reviewed-by: John Stiles <johnstiles@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: Brian Osman <brianosman@google.com>
2021-09-07 13:36:08 +00:00
error: 13: expected 'int', but found 'float'
error: 14: array size is too large
error: 15: expected 'int', but found 'bool'
error: 16: expected 'int', but found 'bool'
error: 17: expected 'int', but found 'int2'
error: 18: array must have a size
Improve error-checking during array-size conversion. The fuzzer noticed insufficient guards in IndexExpression::Convert when converting an array size from an IntLiteral to a SKSL_INT. We had code in IRGenerator which did this properly, so I moved our array-size conversion logic into SkSLType and had IndexExpression share it. Also, a variety of tests around similar error conditions were added. Change-Id: I51529dea25f9029f81ae236511610069d66be29f Bug: oss-fuzz:37462 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/441236 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
2021-08-23 13:24:33 +00:00
18 errors
Reference in New Issue Copy Permalink