-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
sync race issue in nsexec.c #1236
Comments
ping @cyphar , I'm working on a fix now. |
hqhq
added a commit
to hqhq/runc
that referenced
this issue
Dec 19, 2016
Fixes: opencontainers#1236 Signed-off-by: Qiang Huang <[email protected]>
hqhq
added a commit
to hqhq/runc
that referenced
this issue
Jan 20, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
georgethebeatle
pushed a commit
to cloudfoundry-attic/runc
that referenced
this issue
Jan 23, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
georgethebeatle
pushed a commit
to cloudfoundry-attic/runc
that referenced
this issue
Jan 23, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
stevenh
pushed a commit
to stevenh/runc
that referenced
this issue
Feb 1, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
stevenh
pushed a commit
to stevenh/runc
that referenced
this issue
Feb 1, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
stevenh
pushed a commit
to stevenh/runc
that referenced
this issue
Feb 3, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
karampok
pushed a commit
to cloudfoundry-attic/runc
that referenced
this issue
Feb 3, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
stevenh
pushed a commit
to stevenh/runc
that referenced
this issue
Feb 6, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
georgethebeatle
pushed a commit
to cloudfoundry-attic/runc
that referenced
this issue
Feb 7, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
hqhq
added a commit
to hqhq/runc
that referenced
this issue
Feb 18, 2017
Fixes: opencontainers#1236 Fixes: opencontainers#1281 Signed-off-by: Qiang Huang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When I use the latest runc with docker, got a lot of failure in integration tests, they all failed with
exit status 25
, full output looks like:To be explicit, we failed at: https://github.com/opencontainers/runc/blob/master/libcontainer/nsenter/nsexec.c#L579-L581
The code was added by: #1154
But I don't think it introduced this issue, it's probably because we are using the same socketpair to sync with child and grandchild, so there is race condition that parent read data in the wrong sequence.
The text was updated successfully, but these errors were encountered: