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

backport updates and fixes to v1.15.x #2953

Merged
merged 7 commits into from Aug 11, 2023

Conversation

flavorjones
Copy link
Member

@flavorjones flavorjones commented Aug 11, 2023

something changed recently, not sure if it's the compiler in CI, or
the library, or a combination.

(cherry picked from commit 871cd51)
@flavorjones flavorjones added this to the v1.15.x patch releases milestone Aug 11, 2023
@flavorjones flavorjones reopened this Aug 11, 2023
@flavorjones flavorjones changed the base branch from main to v1.15.x August 11, 2023 14:11
@flavorjones flavorjones reopened this Aug 11, 2023
@flavorjones flavorjones force-pushed the flavorjones-backport-v1.15.x-20230811 branch from f2ee926 to ad59d0e Compare August 11, 2023 14:12
@flavorjones flavorjones added the backport Backport of a PR to the current release branch label Aug 11, 2023
flavorjones and others added 6 commits August 11, 2023 11:16
since Minitest 5.19 doesn't load it by default

(cherry picked from commit a29042b)
(cherry picked from commit 5b65501)
The `properties` field of an `xmlNode` element points to an `xmlAttr`.
The first few fields of `xmlAttr` are in common with `xmlNode`, but not
the `properties` field which doesn't exist in an `xmlAttr`.

The `memsize_node` function was passing an `xmlAttr` to a
recursive call and then trying to do the same with the properties of
that.

This led to type confusion and subsequent crashes.

Fixes: #2923
(cherry picked from commit 81762fa)
which turns out to be because xmlDTD has "attributes" where xmlNode
has "properties"

(cherry picked from commit 19a64ba)
@flavorjones flavorjones force-pushed the flavorjones-backport-v1.15.x-20230811 branch from ad59d0e to 8460bfe Compare August 11, 2023 15:17
@flavorjones flavorjones changed the title backport some fixes to v1.15.x backport updates and fixes to v1.15.x Aug 11, 2023
@flavorjones flavorjones merged commit 769faec into v1.15.x Aug 11, 2023
123 checks passed
@flavorjones flavorjones deleted the flavorjones-backport-v1.15.x-20230811 branch August 11, 2023 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport of a PR to the current release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants