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.

Sign up for GitHub

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

Jump to bottom

Fix head index not wrapped around #1

Open

jswirl wants to merge 1 commit into willemt:master from jswirl:master

jswirl commented Apr 23, 2016

me->head in cbuf_poll() needs to be wrapped


          Fix head index not wrapped around

0be94a8

willemt self-assigned this

armarpc commented May 10, 2021

I can confirm that this change is necessary, without it there is an invalid access as cbuf_peek returns a pointer past the buffer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet