mirror of
https://github.com/golang/go
synced 2024-09-29 23:14:29 -06:00
encoding/xml: skip TestCVE202230633 for short tests
TestCVE202230633 uses a bunch of memory, and the input cannot be feasibly reduced while maintaining the behavior hasn't regressed. This test could be reasonably removed, but I'd rather keep it around if we can. Fixes #53814 Change-Id: Ie8b3f306efd20b2d9c0fb73122c26351a55694c4 Reviewed-on: https://go-review.googlesource.com/c/go/+/417655 Run-TryBot: Roland Shoemaker <roland@golang.org> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Bryan Mills <bcmills@google.com>
This commit is contained in:
parent
aa80228526
commit
783ff7dfc4
@ -9,7 +9,6 @@ import (
|
|||||||
"errors"
|
"errors"
|
||||||
"io"
|
"io"
|
||||||
"reflect"
|
"reflect"
|
||||||
"runtime"
|
|
||||||
"strings"
|
"strings"
|
||||||
"testing"
|
"testing"
|
||||||
"time"
|
"time"
|
||||||
@ -1112,8 +1111,8 @@ func TestCVE202228131(t *testing.T) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func TestCVE202230633(t *testing.T) {
|
func TestCVE202230633(t *testing.T) {
|
||||||
if runtime.GOARCH == "wasm" {
|
if testing.Short() {
|
||||||
t.Skip("causes memory exhaustion on js/wasm")
|
t.Skip("test requires significant memory")
|
||||||
}
|
}
|
||||||
defer func() {
|
defer func() {
|
||||||
p := recover()
|
p := recover()
|
||||||
|
Loading…
Reference in New Issue
Block a user