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

Bump github.com/vbatts/tar-split #1131

Merged
merged 1 commit into from Nov 9, 2017
Merged

Conversation

nalind
Copy link
Collaborator

@nalind nalind commented Nov 8, 2017

Update vendor/github.com/vbatts/tar-split to v0.10.2, to fix CVE-2017-14992, per vbatts/tar-split#42.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 8, 2017
@rhatdan
Copy link
Contributor

rhatdan commented Nov 8, 2017

LGTM
/test all

@TomSweeneyRedHat
Copy link
Contributor

Change LGTM, but the tests are not at all happy.

@runcom
Copy link
Member

runcom commented Nov 9, 2017

failing because of #1130

@runcom
Copy link
Member

runcom commented Nov 9, 2017

@nalind please rebase and I'll merge

Update vendor/github.com/vbatts/tar-split to v0.10.2, to fix
CVE-2017-14992, per vbatts/tar-split#42.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
@nalind
Copy link
Collaborator Author

nalind commented Nov 9, 2017

@runcom rebased, thanks!

@runcom
Copy link
Member

runcom commented Nov 9, 2017

/test all

@runcom
Copy link
Member

runcom commented Nov 9, 2017

/test e2e_rhel

@runcom runcom merged commit 190650e into cri-o:master Nov 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants