diff options
author | AKalinich-Luxoft <AKalinich@luxoft.com> | 2018-05-25 20:28:44 +0300 |
---|---|---|
committer | AKalinich-Luxoft <AKalinich@luxoft.com> | 2018-05-31 13:51:53 +0300 |
commit | 2f93ba5c58812ff714f63b0cb219b3dc3017b021 (patch) | |
tree | 8fc975ec63b3d6269a6d04b81dfcaefcd68fda7a /src/components/include | |
parent | 75ce32ae1430ebdbdb0d52bedc2420a4462a3144 (diff) | |
download | sdl_core-2f93ba5c58812ff714f63b0cb219b3dc3017b021.tar.gz |
Additional fixes after ATF testing
Fixed OnCertificateUpdated notification callback for empty certificate
Fixed GetSystemTime triggering
Fixed handshake resuming on system time arrived
Fixed wrong logic in case system time is not ready
Removed redundant logic for non-navi applications
Removed PTU triggerring for navi on empty certificate in DB
Diffstat (limited to 'src/components/include')
3 files changed, 9 insertions, 6 deletions
diff --git a/src/components/include/protocol_handler/session_observer.h b/src/components/include/protocol_handler/session_observer.h index 3482c6569c..242775bf25 100644 --- a/src/components/include/protocol_handler/session_observer.h +++ b/src/components/include/protocol_handler/session_observer.h @@ -66,7 +66,6 @@ struct SessionContext { uint32_t hash_id_; bool is_protected_; bool is_new_service_; - bool is_ptu_required_; /** * @brief Constructor @@ -78,8 +77,7 @@ struct SessionContext { , service_type_(protocol_handler::kInvalidServiceType) , hash_id_(0) , is_protected_(false) - , is_new_service_(false) - , is_ptu_required_(false) {} + , is_new_service_(false) {} /** * @brief Constructor @@ -105,8 +103,7 @@ struct SessionContext { , service_type_(service_type) , hash_id_(hash_id) , is_protected_(is_protected) - , is_new_service_(false) - , is_ptu_required_(false) {} + , is_new_service_(false) {} }; /** diff --git a/src/components/include/security_manager/security_manager.h b/src/components/include/security_manager/security_manager.h index 6ad5e96989..61ba43c74f 100644 --- a/src/components/include/security_manager/security_manager.h +++ b/src/components/include/security_manager/security_manager.h @@ -164,6 +164,11 @@ class SecurityManager : public protocol_handler::ProtocolObserver, virtual void NotifyOnCertificateUpdateRequired() = 0; /** + * @brief Notify all listeners that handshake was failed + */ + virtual void NotifyListenersOnHandshakeFailed() = 0; + + /** * @brief Check if policy certificate data is empty * @return true if policy certificate data is empty otherwise false */ diff --git a/src/components/include/security_manager/security_manager_listener.h b/src/components/include/security_manager/security_manager_listener.h index 5942839b58..00a4c68134 100644 --- a/src/components/include/security_manager/security_manager_listener.h +++ b/src/components/include/security_manager/security_manager_listener.h @@ -50,8 +50,9 @@ class SecurityManagerListener { /** * @brief Notification about handshake failure + * @return true on success notification handling or false otherwise */ - virtual void OnHandshakeFailed() = 0; + virtual bool OnHandshakeFailed() = 0; /** * @brief Notify listeners that certificate update is required. |