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

Registry: use safe libxml2 API #531

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

wismill
Copy link
Member

@wismill wismill commented Oct 13, 2024

See: #529 and the related comment in the original Gnome issue.

@wismill wismill added enhancement Indicates new feature requests registry Indicates a need for improvements or additions to Registry labels Oct 13, 2024
@wismill wismill marked this pull request as ready for review October 13, 2024 18:21
@wismill wismill added this to the 1.8.0 milestone Oct 13, 2024
@wismill
Copy link
Member Author

wismill commented Oct 13, 2024

@nwellnhof I do not see how to get rid of xmlSetGenericErrorFunc when using xmlIOParseDTD and xmlValidateDtd.

@wismill wismill force-pushed the registry/use-safe-libxml2-api branch from cd04ea7 to 3f5452e Compare October 13, 2024 18:30
@nwellnhof
Copy link

Yes, we need additional function for parsing and validating against external DTDs that take a parser context. I just created a libxml2 issue: https://gitlab.gnome.org/GNOME/libxml2/-/issues/808

Contextual functions are safer because they do not rely on a global
state.
@wismill wismill force-pushed the registry/use-safe-libxml2-api branch from 3f5452e to 8c0cf06 Compare October 14, 2024 14:35
@wismill
Copy link
Member Author

wismill commented Oct 15, 2024

Tested with libxml2-master.

@wismill wismill merged commit cd02943 into xkbcommon:master Oct 15, 2024
4 checks passed
@wismill wismill deleted the registry/use-safe-libxml2-api branch October 15, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Indicates new feature requests registry Indicates a need for improvements or additions to Registry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants