-
Notifications
You must be signed in to change notification settings - Fork 17.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
runtime: fix invalid pointer in windows/arm64 badsignal2 #56153
Conversation
This PR (HEAD: 98de88f) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/442216 to see it. Tip: You can toggle comments from me using the |
Message from Quim Muntal: Patch Set 1: Run-TryBot+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Bryan Mills: Patch Set 1: Code-Review+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Gopher Robot: Patch Set 1: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Bill Zissimopoulos: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
This PR (HEAD: e71c147) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/442216 to see it. Tip: You can toggle comments from me using the |
Message from Austin Clements: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Cherry Mui: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
This PR (HEAD: 23dbdb5) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/442216 to see it. Tip: You can toggle comments from me using the |
Message from Bill Zissimopoulos: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Cherry Mui: Patch Set 3: Run-TryBot+1 Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Gopher Robot: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Message from Gopher Robot: Patch Set 3: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/442216. |
Initializes the R3 register with an available address in the stack. The addressed location is used to receive the number of bytes written by WriteFile. Fixes #56080 Change-Id: I0368eb7a31d2d6a098fa9c26e074eb1114a92704 GitHub-Last-Rev: 23dbdb5 GitHub-Pull-Request: #56153 Reviewed-on: https://go-review.googlesource.com/c/go/+/442216 Reviewed-by: Cherry Mui <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Run-TryBot: Cherry Mui <[email protected]> Reviewed-by: Bryan Mills <[email protected]>
This PR is being closed because golang.org/cl/442216 has been merged. |
Initializes the R3 register with an available address in the stack. The addressed location is used to receive the number of bytes written by WriteFile. Fixes golang#56080 Change-Id: I0368eb7a31d2d6a098fa9c26e074eb1114a92704 GitHub-Last-Rev: 23dbdb5 GitHub-Pull-Request: golang#56153 Reviewed-on: https://go-review.googlesource.com/c/go/+/442216 Reviewed-by: Cherry Mui <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Run-TryBot: Cherry Mui <[email protected]> Reviewed-by: Bryan Mills <[email protected]>
Initializes the R3 register with an available address in the stack. The addressed location is used to receive the number of bytes written by WriteFile.
Fixes #56080