diff options
| author | Joe Tsai <joetsai@digital-static.net> | 2022-06-22 20:57:50 -0700 |
|---|---|---|
| committer | Joseph Tsai <joetsai@digital-static.net> | 2022-08-08 20:44:51 +0000 |
| commit | 9a4685f22036b4e2577bb79dbfabd7c4e48146e3 (patch) | |
| tree | 835c8522189cbf231780f450fb7b9dbb7e3431dd /src/bytes | |
| parent | ced4d6fd2d415d43c0f26e01fe332e4afa80114a (diff) | |
| download | go-9a4685f22036b4e2577bb79dbfabd7c4e48146e3.tar.xz | |
strings: avoid utf8.RuneError mangling in Split
Split should only split strings and not perform mangling
of invalid UTF-8 into ut8.RuneError.
The prior behavior is clearly a bug since mangling is not
performed in all other situations (e.g., separator is non-empty).
Fixes #53511
Change-Id: I112a2ef15ee46ddecda015ee14bca04cd76adfbf
Reviewed-on: https://go-review.googlesource.com/c/go/+/413715
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Ian Lance Taylor <iant@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Diffstat (limited to 'src/bytes')
| -rw-r--r-- | src/bytes/bytes_test.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/bytes/bytes_test.go b/src/bytes/bytes_test.go index 985aa0b147..b407fe8a2d 100644 --- a/src/bytes/bytes_test.go +++ b/src/bytes/bytes_test.go @@ -755,6 +755,8 @@ var splittests = []SplitTest{ {"123", "", 2, []string{"1", "23"}}, {"123", "", 17, []string{"1", "2", "3"}}, {"bT", "T", math.MaxInt / 4, []string{"b", ""}}, + {"\xff-\xff", "", -1, []string{"\xff", "-", "\xff"}}, + {"\xff-\xff", "-", -1, []string{"\xff", "\xff"}}, } func TestSplit(t *testing.T) { |