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

fix(cruby): patch libxml2 to address GNOME/libxml2#200 #2182

Merged
merged 3 commits into from Feb 6, 2021

Conversation

flavorjones
Copy link
Member

What problem is this PR intended to solve?

This patch shrinks the libxml2 input buffer in a few parser functions.

Fixes #2132 using the proposed fix for GNOME/libxml2#200 in https://gitlab.gnome.org/GNOME/libxml2/-/merge_requests/100

Have you included adequate test coverage?

Yes, the test case from #2132 is reproduced here and run against JRuby and packaged libxml2 (but not system libxml2).

Does this change affect the behavior of either the C or the Java implementations?

This fixes an edge case in the C implementation that does not exist in the Java implementation.

Also, use skip instead of avoiding test definition.
This patch shrinks the libxml2 input buffer in a few parser functions.

Fixes #2132
@flavorjones flavorjones force-pushed the 2132-fix-huge-input-lookup-on-large-cdata branch from ec13839 to e0da4d2 Compare February 5, 2021 22:02
@flavorjones flavorjones merged commit bd72b5a into main Feb 6, 2021
@flavorjones flavorjones deleted the 2132-fix-huge-input-lookup-on-large-cdata branch February 6, 2021 19:05
@flavorjones flavorjones added this to the v1.11.x patch releases milestone Mar 8, 2021
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.

Huge input lookup with large CDATA when parsing (SAX)
1 participant