minor fixes for Coverity reports

[JacobBarthelmeh]

minor fixes for Coverity reports

[JacobBarthelmeh]

Retest this please Jenkins. Possible network timeout: PRB-single-flag.txt_95

SSL curve name is SECP256R1

Unexpected server result

removing ready file: /tmp/workspace/PRB-generic-config-parser/wolfssl/tmp.l0SKcXeQK1/wolf_ocsp_s2_readyF1162286

removing ready file: /tmp/workspace/PRB-generic-config-parser/wolfssl/tmp.l0SKcXeQK1/wolf_ocsp_s2_readyF2162286

removing ready file: /tmp/workspace/PRB-generic-config-parser/wolfssl/tmp.l0SKcXeQK1/wolf_ocsp_s2_readyF3162286

removing ready file: /tmp/workspace/PRB-generic-config-parser/wolfssl/tmp.l0SKcXeQK1/wolf_ocsp_s2_readyF4162286

removing ready file: /tmp/workspace/PRB-generic-config-parser/wolfssl/tmp.l0SKcXeQK1/wolf_ocsp_s2_readyF5162286

retrying...

FAIL scripts/ocsp-stapling_tls13multi.test (exit status: 143)

<<<

Error: Test runtime=4603sec. Make Check RESULT = 2

Exiting with status: 1

Script ran for 5167 seconds.

script returned exit code 1

[douzzer]

Hmm -- why are you reverting d350ba6 here? Is there something special about this use of XFREE() according to Coverity?

[JacobBarthelmeh]

thanks, I should have checked x for null here not dCert

[douzzer]

This call isn't needed -- whenever wc_AesNew() sets ret to nonzero, it always returns NULL, and there is nothing to free.

[JacobBarthelmeh]

likely a false positive then, I missed that the return would always be null if ret is set to nonzero

[douzzer]

whoops, need to actually return WC_TEST_RET_ENC_EC(ret) here, since cleanup: will wc_FreeRng() unconditionally.