Skip to content

fix NULL check in structmeta_get_module_ns - #1016

Merged
Siyet merged 1 commit into
msgspec:mainfrom
provinzkraut:fix-get-module-ns
Apr 12, 2026
Merged

Siyet merged 1 commit into
msgspec:mainfrom
provinzkraut:fix-get-module-ns

Conversation

@provinzkraut

Copy link
Copy Markdown
Member

Fix a bug in structmeta_get_module_ns that could theoretically lead to a segfault due to a missing NULL check, even though it's quite unlikely to ever hit that path

@Siyet Siyet left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, thanks Janek!

@Siyet
Siyet merged commit bcb7a2b into msgspec:main Apr 12, 2026
22 checks passed
@Siyet Siyet mentioned this pull request Apr 12, 2026
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