| Summary: | Avoid possible NULL pointer dereference in monitor.c | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Portable OpenSSH | Reporter: | Xidong Wang <wangxidong> | ||||
| Component: | Miscellaneous | Assignee: | Assigned to nobody <unassigned-bugs> | ||||
| Status: | CLOSED WONTFIX | ||||||
| Severity: | enhancement | CC: | djm | ||||
| Priority: | P5 | ||||||
| Version: | 8.5p1 | ||||||
| Hardware: | Other | ||||||
| OS: | Windows 10 | ||||||
| Attachments: |
|
||||||
There should be no situations in which pmonitor can be NULL in the function, as the calling function monitor_read() references is before and after the call closing bugs resolved before openssh-8.9 |
Created attachment 3496 [details] mm_answer_term() in file monitor.c In function mm_answer_term() in file monitor.c, it's better to check whether pmonitor is NULL before it's dereferenced in line 1643.