]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.16-security] archive/zip: fix panic in Reader.Open
authorRoland Shoemaker <roland@golang.org>
Tue, 2 Mar 2021 18:00:53 +0000 (10:00 -0800)
committerKatie Hockman <katiehockman@google.com>
Tue, 9 Mar 2021 17:55:16 +0000 (17:55 +0000)
commit634d28d78ccbeb6e86f8bfeba030ea8be518f8fa
treea3900a10b13f77f9665cdd6d36ec29f79d20235a
parentd86e53e896eca907ad67300c0bb495e3dd925358
[release-branch.go1.16-security] archive/zip: fix panic in Reader.Open

When operating on a Zip file that contains a file prefixed with "../",
Open(...) would cause a panic in toValidName when attempting to strip
the prefixed path components.

Fixes CVE-2021-27919

Change-Id: Ic755d8126cb0897e2cbbdacf572439c38dde7b35
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/1004761
Reviewed-by: Filippo Valsorda <valsorda@google.com>
Reviewed-by: Russ Cox <rsc@google.com>
Reviewed-by: Katie Hockman <katiehockman@google.com>
(cherry picked from commit ce22003b26eaf8e4a690757f699aae7062d41472)
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/1013753
Reviewed-by: Roland Shoemaker <bracewell@google.com>
src/archive/zip/reader.go
src/archive/zip/reader_test.go