Skip to content

Conversation

@BiteTheDDDDt
Copy link
Contributor

Proposed changes

fix use after free on IRuntimeFilter::update_state()

@BiteTheDDDDt
Copy link
Contributor Author

run buildall

Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 27, 2024
@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@BiteTheDDDDt BiteTheDDDDt merged commit e1047c5 into apache:master Sep 29, 2024
gavinchou pushed a commit that referenced this pull request Sep 29, 2024
…ate() (#41397)

## Proposed changes
fix use after free on IRuntimeFilter::update_state()
cjj2010 pushed a commit to cjj2010/doris that referenced this pull request Oct 12, 2024
…ate() (apache#41397)

## Proposed changes
fix use after free on IRuntimeFilter::update_state()
amorynan pushed a commit to amorynan/doris that referenced this pull request Oct 12, 2024
…ate() (apache#41397)

## Proposed changes
fix use after free on IRuntimeFilter::update_state()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/3.0.2-merged p0_c reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants