Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[DevASAN][CPU] bugfix for mmap return value check. #2402

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

yingcong-wu
Copy link
Contributor

@yingcong-wu yingcong-wu commented Dec 2, 2024

mmap() would return MAP_FAILED instead of NULL in case of error, we need to do a translation here.

Intel/llvm PR: intel/llvm#16466

@yingcong-wu yingcong-wu requested a review from a team as a code owner December 2, 2024 08:39
@github-actions github-actions bot added the loader Loader related feature/bug label Dec 2, 2024
@yingcong-wu
Copy link
Contributor Author

Hi @aarongreig , could you help merge this fix? Thanks.

@aarongreig
Copy link
Contributor

Hi @aarongreig , could you help merge this fix? Thanks.

Happy to merge right away as it's a trivial change but I'll need you to make the LLVM PR to update the tag. I would do it myself but you can't approve your own PR and there wont be another UR reviewer around to approve until late next week

@yingcong-wu
Copy link
Contributor Author

Hi @aarongreig , could you help merge this fix? Thanks.

Happy to merge right away as it's a trivial change but I'll need you to make the LLVM PR to update the tag. I would do it myself but you can't approve your own PR and there wont be another UR reviewer around to approve until late next week

Thank you, updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loader Loader related feature/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants