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

overlay: fix call to clone on s390x #1418

Merged
merged 1 commit into from
Nov 3, 2022

Conversation

giuseppe
Copy link
Member

@giuseppe giuseppe commented Nov 2, 2022

the two arguments to clone are swapped on s390x. This patch fixes the call when running on s390x.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2108887

Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com

the two arguments to clone are swapped on s390x. This patch fixes the
call when running on s390x.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2108887

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@rhatdan
Copy link
Member

rhatdan commented Nov 2, 2022

Copy link
Collaborator

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vrothberg vrothberg merged commit 4a4d8f8 into containers:main Nov 3, 2022
TomSweeneyRedHat added a commit to TomSweeneyRedHat/buildah that referenced this pull request Jan 5, 2023
Bump the c/storage to v1.43.1 so this v1.28 branch will have access to
the fix for https://bugzilla.redhat.com/show_bug.cgi?id=2140087 -
SIGSEGV: segmentation violation on s390x.  This was originally fixed
with containers/storage#1418 and is being
vendored here now as the fix is in the v1.27 branch already.

[NO NEW TESTS NEEDED]
Signed-off-by: tomsweeneyredhat <tsweeney@redhat.com>
TomSweeneyRedHat added a commit to TomSweeneyRedHat/buildah that referenced this pull request Jan 5, 2023
Bump the c/storage to v1.43.1 so this v1.28 branch will have access to
the fix for https://bugzilla.redhat.com/show_bug.cgi?id=2140087 -
SIGSEGV: segmentation violation on s390x.  This was originally fixed
with containers/storage#1418 and is being
vendored here now as the fix is in the v1.27 branch already.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=2140087

[NO NEW TESTS NEEDED]
Signed-off-by: tomsweeneyredhat <tsweeney@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants