Skip to content

plugins types BUGFIX leaked value on pattern mismatch - #2600

Open
affan-arch wants to merge 1 commit into
CESNET:develfrom
affan-arch:date-time-pattern-leak
Open

affan-arch wants to merge 1 commit into
CESNET:develfrom
affan-arch:date-time-pattern-leak

Conversation

@affan-arch

Copy link
Copy Markdown

lyplg_type_store_date_and_time(), lyplg_type_store_date() and lyplg_type_store_time() return with LY_CHECK_RET() when the pattern check fails, so a value passed with LYPLG_TYPE_STORE_DYNAMIC (any XML/JSON value with an entity or escape, e.g. <l>2005-05-31T23:15:15&amp;</l>) is never freed; jump to cleanup like the hints check right above does.

@affan-arch
affan-arch force-pushed the date-time-pattern-leak branch from 9661911 to ba7542c Compare October 26, 2026 15:14
@affan-arch
affan-arch changed the base branch from master to devel October 26, 2026 15:14
@affan-arch

Copy link
Copy Markdown
Author

opened this one against master by mistake, retargeted it to devel and rebased. it applied cleanly and utest_yang_types still passes.

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.

1 participant