Repository navigation
Conversation
6a46a2b to
5c3312c
Compare
|
This PR (HEAD: 5c3312c) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/831024. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
5c3312c to
d02cff4
Compare
|
This PR (HEAD: d02cff4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/831024. Important tips:
|
|
Message from 14侯廷翰(10122 侯廷翰): Patch Set 3: Code-Review+1 (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
d02cff4 to
1c40f24
Compare
|
This PR (HEAD: 1c40f24) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/831024. Important tips:
|
|
Message from Pranay Debnath: Patch Set 4: (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from 14侯廷翰(10122 侯廷翰): Patch Set 4: Code-Review+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from qiu laidongfeng: Patch Set 4: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 4: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2026-10-11T06:18:51Z","revision":"78da43ece3d0473fe4f23aaa1e2da234c79bb192"} Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from qiu laidongfeng: Patch Set 4: -Commit-Queue (Performed by <GERRIT_ACCOUNT_60063> on behalf of <GERRIT_ACCOUNT_64033>) Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 4: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 4: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
1c40f24 to
a4ede38
Compare
|
This PR (HEAD: a4ede38) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/831024. Important tips:
|
When unpacking the V7/USTAR header, numeric UID and GID values are parsed via p.parseNumeric, but are subsequently cast directly to platform-native int without bounds checking. On 32-bit platforms, values >= 2^32 silently truncate to 0 or wrap to negative numbers. This change verifies that the parsed int64 fits within [math.MinInt, math.MaxInt] and sets p.err to ErrHeader if out of bounds, matching the behavior added to mergePAX. Fixes golang#81446 Fixes golang#82121
a4ede38 to
a4d44d5
Compare
|
This PR (HEAD: a4d44d5) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/831024. Important tips:
|
|
Message from 14侯廷翰(10122 侯廷翰): Patch Set 6: Code-Review+1 PS6 is a clean rebase of PS4 (range-diff shows no code changes; the stray Re-tested PS6 locally: go test archive/tar passes on amd64 and GOARCH=386, Please don’t reply on this GitHub thread. Visit golang.org/cl/831024. |
When unpacking the V7/USTAR header, numeric UID and GID values are parsed
via p.parseNumeric, but are subsequently cast directly to platform-native
int without bounds checking. On 32-bit platforms, values >= 2^32 silently
truncate to 0 or wrap to negative numbers.
This change verifies that the parsed int64 fits within [math.MinInt,
math.MaxInt] and sets p.err to ErrHeader if out of bounds, matching the
behavior added to mergePAX.
Fixes #81446
Fixes #82121