Maks Mishin

Results 9 issues of Maks Mishin

Return value of function LSEEK (lseek64) is not checked, but it is usually checked for this function. Found by RASU JSC.

* Development branch: master * Issues fixed: add error handling for correct closing of socket `s` because of return with close socket leads to descriptor leak(CWE-403). * Brief description of...

Return value of a function 'malloc_cache_aligned' is dereferenced at autoboot.c:207 without checking for NULL, but it is usually checked for this function. Found by RASU JSC.

NULL-pointer check for key in sshconnect2.c:1714 is not necessary because it is performed in sshconnect2.c:1708 by calling of `sshkey_is_cert()` function. Found by RASU JCS.

Pointer 'rr', which is dereferenced at getrrsetbyname.c:276, is compared to a NULL value at getrrsetbyname.c:319. Found by RASU JCS.

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.

These NULL-checks for pointer `c` are redundant because the check is already at clientloop.c:1181. Found by RASU JSC.

* Development branch: master * Issues fixed: descriptor and memory leaks * Brief description of code changes: The descriptor `s` and pointer `server_res` are lost without close and free.

* Development branch: master * Issues fixed: descriptor leak * Brief description of code changes: If the pid file remains, but the process has been killed, the handle will be...