diff options
| author | Jacob Keeler <jacob.keeler@livioradio.com> | 2020-06-18 11:09:42 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-18 11:09:42 -0400 |
| commit | baf63cadaa6ce045b4915222601a377e86daf442 (patch) | |
| tree | acfdd7fd1e5f0241c93db91ca2850fa5a1668147 /src/components/policy | |
| parent | cab7e593787f1f18eb3847aa27bfc9b90c066197 (diff) | |
| download | sdl_core-baf63cadaa6ce045b4915222601a377e86daf442.tar.gz | |
Fix Handshake Flow issues (#3441)
* Fix issue with External PTU timeout
When attempting to start an encrypted service, the mobile proxy would not receive a response if the PTU timed out. This is because the retry count is based on the number of OnSystemRequests received from the HMI. Since that number would never exceed the number of retries, `IsAllowedRetryCountExceeded` would never return false. This will now return false after the final retry times out.
* Add error code check in ProcessInternalError
If the proxy responds with NOT_SUPPORTED or SSL_INVALID_DATA, Core will mark the handshake as failed.
Diffstat (limited to 'src/components/policy')
| -rw-r--r-- | src/components/policy/policy_external/src/policy_manager_impl.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/components/policy/policy_external/src/policy_manager_impl.cc b/src/components/policy/policy_external/src/policy_manager_impl.cc index 082d4a37ba..12ab1224fd 100644 --- a/src/components/policy/policy_external/src/policy_manager_impl.cc +++ b/src/components/policy/policy_external/src/policy_manager_impl.cc @@ -1368,7 +1368,7 @@ bool PolicyManagerImpl::IsAllowedRetryCountExceeded() const { LOG4CXX_AUTO_TRACE(logger_); sync_primitives::AutoLock auto_lock(retry_sequence_lock_); - return retry_sequence_index_ > retry_sequence_seconds_.size(); + return retry_sequence_index_ >= retry_sequence_seconds_.size(); } void PolicyManagerImpl::IncrementRetryIndex() { |