Skip to content

[PR] Object Tracking Global Variable patch - #249

Draft
elazaro-riverside wants to merge 11 commits into
mainfrom
libresolve-global-variable-patch
Draft

[PR] Object Tracking Global Variable patch#249
elazaro-riverside wants to merge 11 commits into
mainfrom
libresolve-global-variable-patch

Conversation

@elazaro-riverside

Copy link
Copy Markdown
Collaborator

No description provided.

…ng literals) from tracking. This ensures only mutable globals that need bounds checking are tracked while constant data like string literals are ignored.
@elazaro-riverside elazaro-riverside self-assigned this Aug 4, 2026
@elazaro-riverside elazaro-riverside added the libresolve Related to libresolve component label Aug 4, 2026

@zoogies zoogies left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm on my phone at the gym right now, but this is pretty easy to review 🙂

Per our previous discussion, I imagine that you've tested this on Redis and it fixes the issue. We should have a discussion tomorrow regarding this!

LGTM

Ethan Lazaro added 3 commits August 5, 2026 09:50
@elazaro-riverside
elazaro-riverside marked this pull request as ready for review August 5, 2026 15:14
@elazaro-riverside
elazaro-riverside marked this pull request as draft August 7, 2026 15:39
@elazaro-riverside elazaro-riverside changed the title [PR] Object Tracking Global Variable patch [DRAFT] Object Tracking Global Variable patch Aug 7, 2026
Ethan Lazaro added 3 commits August 11, 2026 09:20
…obals with padded versions. This will not cause multiple_globals.c to pass in CI.
…onstantArray type. Updated name of vulnerability specification to be consistent with testcase name. This should not pass in CI.
@elazaro-riverside elazaro-riverside changed the title [DRAFT] Object Tracking Global Variable patch [PR] Object Tracking Global Variable patch Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libresolve Related to libresolve component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants