Skip to content
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

Can't join the target namespace if the PID number is bigger than 65535 #532

Closed
zhouxinyu opened this issue Jul 24, 2018 · 5 comments
Closed
Labels
bug no-auto-close Don't auto-close as a stale issue stale-issue

Comments

@zhouxinyu
Copy link

Hi,

I found criu will treat the PID number as a file path when using join-ns feature if the number is bigger than 65535:

|| (val < 0) || (val > 65535)) {

if ((errno == ERANGE) || (endptr == str)
			|| (*endptr != '\0')
			|| (val < 0) || (val > 65535)) {
		str = NULL;
		return -1;
	}

Is it necessary to have this limitation?

@avagin
Copy link
Member

avagin commented Nov 30, 2018

Cc: @xemul

@xemul
Copy link
Member

xemul commented Apr 1, 2019

Sorry for just noticing this :)

This is not a "limitation", this is an attempt to distinguish /proc/ path from anything else (e.g. ip tool's bindmounts). And looking at the code I'd say it's quite buggy :(

@github-actions
Copy link

A friendly reminder that this issue had no activity for 30 days.

@avagin avagin added bug no-auto-close Don't auto-close as a stale issue and removed stale-issue labels Jan 26, 2021
@github-actions
Copy link

A friendly reminder that this issue had no activity for 30 days.

@github-actions
Copy link

github-actions bot commented Apr 3, 2021

A friendly reminder that this issue had no activity for 30 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug no-auto-close Don't auto-close as a stale issue stale-issue
Projects
None yet
Development

No branches or pull requests

3 participants