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
x/sys/windows/svc: servicemain argv pointer truncated #24575
Comments
/cc @alexbrainman |
I am pretty sure this is a bug. But I would like to write a test before I fix this. I will try and do this when I have time. Thank you. Alex |
Change https://golang.org/cl/110160 mentions this issue: |
@gopherbot please consider this for backport to 1.10.3 |
Backport issue(s) opened: #25200 (for 1.10). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://golang.org/wiki/MinorReleases. |
@alexbrainman @zhangyoufu Can you please clarify why this should be considered for backport? Per https://github.com/golang/go/wiki/MinorReleases it should be either a regression, security issue, or serious problem with no workaround. |
I do not think this fix needs to be backported. How do we backport golang.org/x/sys repo? Alex |
Thanks, I'll close the backport issue then. For future backports, the process is the same as it is for the go repo. It looks like golang.org/x/sys has a go1.11 release branch (https://go.googlesource.com/sys), so we would just cherry pick to that branch per the instructions here: https://github.com/golang/go/wiki/MinorReleases#making-cherry-pick-cls |
Fair enough, I did not know that. It is not surprising, I live on bleeding edge :-) Alex |
What version of Go are you using (
go version
)?1.10
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?Windows x64
What did you do?
review the code
What did you expect to see?
What did you see instead?
The text was updated successfully, but these errors were encountered: