-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Fix overflow when searching for #> #7094
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
Fix overflow when searching for #> #7094
Conversation
Doing an ASan run locally. |
I knew there was a reason my body woke me up early today ... |
@swift-ci Please smoke test |
Is the call to |
Oh, hm. So that might also go past the buffer by one? |
Since Identifier::isEditorPlaceholder(StringRef(CurPtr, BufferEnd-CurPtr)) |
ede6bf7 increments the buffer pointer to early when searching for the <# prefix to stop lexing an operator, so the source buffer is accessed 1 byte off the end. Thanks ASan and thanks @gparker42! rdar://problem/28457876
@swift-ci Please smoke test |
@swift-ci Please smoke test f'real |
@swift-ci Pretty please smoke test |
|
@swift-ci Please asan test |
The ASan test is running but not showing up here: https://ci.swift.org/job/swift-PR-osx-ASAN-test/46 |
ASAN test succeeded. This should be good to go. |
Thanks again @gparker42 ! |
ede6bf7 increments the buffer pointer to early
when searching for the <# prefix to stop lexing an operator,
so the source buffer is accessed 1 byte off the end.
Thanks ASan and thanks @gparker42!
rdar://problem/28457876