Static analyzer fixes

Check return value of read, found by Clang. See also #677

Remove unused member variable in OpenSSL ECC, found by Clang.

In ECDSA tests, if the pointer is null we should return rather
than dereferencing it. Found by Coverity.

1 files changed, 0 insertions, 1 deletions

diff --git a/src/lib/prov/openssl/openssl_ec.cpp b/src/lib/prov/openssl/openssl_ec.cpp
index 7b20b1c73..f0e70fe56 100644
--- a/src/lib/prov/openssl/openssl_ec.cpp
+++ b/src/lib/prov/openssl/openssl_ec.cpp
@@ -267,7 +267,6 @@ class OpenSSL_ECDH_KA_Operation : public PK_Ops::Key_Agreement_with_KDF
 
    private:
       std::unique_ptr<EC_KEY, std::function<void (EC_KEY*)>> m_ossl_ec;
-      size_t m_order_bits = 0;
    };
 
 }