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

Update batik deps due to CVE-2018-8013 #284

Merged
merged 2 commits into from
Oct 17, 2018
Merged

Update batik deps due to CVE-2018-8013 #284

merged 2 commits into from
Oct 17, 2018

Conversation

ghenadiibatalski
Copy link
Contributor

@ghenadiibatalski ghenadiibatalski commented Oct 15, 2018

Our security engineer stumbled over the CVE-2018-8013

I increased the required version to 1.10 and run into the issue BATIK-1229 so the dependency to batik-ext seems to be necessary to work around this issue.

Ghenadii Batalski added 2 commits October 15, 2018 11:15
@danfickle
Copy link
Owner

Thanks a lot. It should be noted that untrusted svg images should not be used due to the ability of svg to include file uris to incorporate raster images. At the very least, this gives it the ability to probe the file system. Anyway, thanks again, merging now.

@danfickle danfickle merged commit 665c28b into danfickle:open-dev-v1 Oct 17, 2018
@danfickle
Copy link
Owner

P.S I’ll do a release over the weekend.

@danfickle
Copy link
Owner

RC17 just released. Thanks @ghenadiibatalski.

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.

2 participants