commit 1deeb6dd84235604fa9dfedc434da357889aa858
author Yuhui.Zheng <10982575+yuhui-zheng@users.noreply.github.com> Wed Dec 04 07:52:49 2019 +0000
committer Yuhui.Zheng <10982575+yuhui-zheng@users.noreply.github.com> Wed Dec 04 07:52:49 2019 +0000
tree 778801f8a1d35d66bc11b2ef3c2aed8b0f675773
parent 9491af1fd73367134c1ae16d2eea1ac25c007816

Check socket binding result before doing anything with socket. (This is to address ARG findings.) Breaking the single return rule here, due to precedent violation at line 1039 and 1144.  

prvTransferConnect() now returns:
- pdTRUE: everything's good. pdTRUE = 1.
- -pdFREERTOS_ERRNO_ENOMEM: FreeRTOS_socket() failed. -pdFREERTOS_ERRNO_ENOMEM = -12.
- -pdFREERTOS_ERRNO_EINVAL || -pdFREERTOS_ERRNO_ECANCELED: FreeRTOS_bind() failed. Negative values.

Thus, at line 569 and line 617, needs to check != pdTRUE instead of == pdFALSE.

This commit is done on behalf of Alfred.