Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

UB in libregexp #127

Closed
linusg opened this issue Nov 24, 2023 · 1 comment · Fixed by #136
Closed

UB in libregexp #127

linusg opened this issue Nov 24, 2023 · 1 comment · Fixed by #136

Comments

@linusg
Copy link
Contributor

linusg commented Nov 24, 2023

I never upstreamed this because upstreaming to a dead project seemed pointless, but maybe qjs-ng wants to adopt the patch: https://codeberg.org/kiesel-js/kiesel/src/branch/main/patches/libregexp.patch
@saghul
Copy link
Contributor

saghul commented Nov 24, 2023

We certainly take patches here. Would you mind sending a PR?

@linusg linusg mentioned this issue Nov 25, 2023
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix null pointer arithmetic UB in libregexp
2 participants
@saghul @linusg