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 DEREF_AFTER_NULL in mm_answer_keyallowed() #469

Closed

maks-mishin wants to merge 1 commit into openssh:master from maks-mishin:mm-answer-keyallowed

maks-mishin commented Mar 11, 2024

After having been compared to a NULL value at monitor.c:1206, pointer 'key' is dereferenced in function auth2_record_key.

Found by RASU JSC.

Contributor

djmdjm commented Mar 30, 2024

key=NULL is impossible here as sshkey_froms() won't return a NULL key. I'll look at removing the unnecessary key==NULL checks instead


          monitor: Remove redundant null-checks in mm_answer_keyallowed()

09ff2fb

maks-mishin closed this

maks-mishin reopened this

maks-mishin closed this by deleting the head repository

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

Labels

None yet