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

BUG: fix s390/s390x handling of shm* syscalls #206

Closed
wants to merge 2 commits into from

Conversation

drakenclimber
Copy link
Member

Issue #193 exposed a few cases where libseccomp isn't completely supporting shm*() syscalls on s390/s390x. This commit adds IPC support for shmat(), shmdt(), shmget(), and shmctl(). Socket support for the aforementioned syscalls was unchanged and should still work as before.

I ran this through arch-syscall-validate, but I really don't have a way to test these changes. @cpaelzer or others - do you have a way to test this?

On s390, shmat, shmdt, shmget, and shmctl can be done either
via sockets or ipc.  Prior to this commit, the s390 code only
supported these calls via sockets.  This commit adds support
for both sockets and ipc.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
On s390x, shmat, shmdt, shmget, and shmctl can be done either
via sockets or ipc.  Prior to this commit, the s390x code only
supported these calls via sockets.  This commit adds support
for both sockets and ipc.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 90.003% when pulling 5d7015e on drakenclimber:issues/193 into ed3b35c on seccomp:master.

@drakenclimber drakenclimber self-assigned this Feb 19, 2020
@drakenclimber drakenclimber added this to the v2.4.3 milestone Feb 19, 2020
@drakenclimber
Copy link
Member Author

@pcmoore, I think this is the last open issue for the v2.4.3 release.

@pcmoore
Copy link
Member

pcmoore commented Feb 20, 2020

These changes look reasonable, but lets see if @cpaelzer can give them a test (I don't have access to s390[x] hardware).

As an aside, look at that code coverage :}

@pcmoore pcmoore changed the title Fix s390/s390x handling of shm* syscalls BUG: fix s390/s390x handling of shm* syscalls Feb 22, 2020
@pcmoore
Copy link
Member

pcmoore commented Feb 23, 2020

At this point, let's just assume it all works on s390[x], if there is a bug we'll fix it.

Merged via 0a4c030, thanks @drakenclimber.

@pcmoore
Copy link
Member

pcmoore commented Feb 23, 2020

Merged into the release-2.4 branch via 266e1b4.

@pcmoore pcmoore closed this Feb 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants