| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| | |
Handle omitted parameter permissions properly in OnVehicleData notification
|
| | |
|
| | |
|
|/
|
|
|
| |
* Add OnWayPointChange handling for NAVIGATION app services
* Only send OnWayPointChange to mobile from active waypoints provider
|
| |
|
| |
|
|\
| |
| |
| |
| | |
smartdevicelink/fix/gpcd_consumer_friendly_messages
Re-add messages field in policy table json
|
|/
|
|
| |
consumer_friendly_messages.messages was being omitted from the policy table json rather than just the snapshot, this reintroduces this field (this field is still cleared out when generating the snapshot in `CheckSnapshotInitialization`)
|
|
|
|
|
|
|
| |
* Fix policy validation of OnVehicleData message
* Fix unit tests
Co-authored-by: Mykola Korniichuk <mkorniichuk@luxoft.com>
|
|\
| |
| | |
Fix session heartbeat starting
|
| | |
|
| | |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
fix/fix_session_heartbeat_starting
# Conflicts:
# src/components/connection_handler/include/connection_handler/heartbeat_monitor.h
# src/components/connection_handler/src/connection.cc
# src/components/connection_handler/src/heartbeat_monitor.cc
# src/components/protocol_handler/src/protocol_handler_impl.cc
|
| | |
| | |
| | |
| | |
| | | |
This prevents SDL from starting HB session tracking every
time SDL receives HB request from mobile.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In the current implementation SDL starts session
heartbeat right after application registration.
However the correct behavior is to start session
heartbeat after first heartbeat request from mobile app.
Following changes were done:
- Removed starting session heartbeat on app registration
- Added session heartbeat start on frist heartbeat request
from mobile app
- Added check to prevent starting of session heartbeat if
it was disabled in settings
|
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Cover connection handler and policy handler with unit tests
Few tests added to connection_handler_impl_test.cc and to
connection_test.cc
There were 2 mocks for telemetry observer. So, one is removed
(src/components/include/test/protocol_handler/mock_telemetry_observer.h)
Fixed mock_telemetry_observer.h (src/components/protocol_handler/
test/include/protocol_handler/mock_telemetry_observer.h)
Added tests to protocol_handler_tm_test.cc
set_hash_id and get_hash_id are covered with unit tests.
set_hash_id, get_hash_id, ConvertPacketDataToString,
get_protocol_observers_count declarations are added to
protocol_handler_impl.h under BUILD_TEST flag.
Resolves: #2449
* fix according to comments
* Unit test for get_hash_id and set_hash_id
Removed declarations for internal functions get_hash_id and set_hash_id
ConvertPacketDataToString moved to convert_utils
as ConvertBinaryDataToString
Fixed unit tests for get_hash_id and set_hash_id
* replace dcheck or return to if in ConvertBinaryDataToString
due to issue in smoke tests
* minor UT fixes
* Update doxygen for convert_utils
* Address comments and fix UT
Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
|
|\ \ |
|
| |\ \
| | | |
| | | | |
7.0.0 Release
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* remove cloud app exception from StateControllerImpl::OnApplicationRegistered
* Fix/ Postpone application activation if registration is not complete yet (#3557)
* Postpone application activation if registration is not complete yet
* fixup! Postpone application activation if registration is not complete yet
* fixup! Postpone application activation if registration is not complete yet
Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
* fix style
* fix unit tests
Co-authored-by: Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com>
Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
There were noticed two issues related to a/v streaming:
1. Due to object self destruction after TerminateRequest() call
any attempt to access member field of that object may cause an
undefined behavior - it might be a core crash or corrupted value
sometimes. In this particular case, SDL tries to access `message_`
field through `application_id()` function after object destruction.
As a result, sometimes SDL crashes and sometimes this function just
returns 0. Because of that, SDL was not able to find application by
zero id and broke the retry sequence. This causes random failures
of some ATF scripts. To avoid that issue, all retry logic was
extracted into the separate function and `TerminateRequest` was
moved after that function. This will guarantee that there is no
attempts to access object fields after its destruction.
2. There was noticed that SDL makes one redundant retry attempt.
That was because of late retry value increment. To fix that issue,
increment has been placed before retry amount check.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* fix issue 2808:SDL Crash seen in SDL upon receiving rpc - OnEmergencyEvent with enabled:true
* change code stype
* Fix conflicts
* Fix log
* Fix crash
* Address comments
Co-authored-by: zhangwenqin <zhangwenqin@iauto.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Rework processing of the OnExitApplication
(RESOURCE_CONSTRAINT) notification
* Update UTs according to changes
Co-authored-by: sniukalov <sniukaov@luxoft.com>
Co-authored-by: Aleksandr Kutsan <AKutsan@luxoft.com>
Co-authored-by: jacobkeeler <jacob.keeler@livioradio.com>
Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com>
|
| | | |
| | | |
| | | |
| | | |
| | | | |
* Fix formatting for logs after logger overhaul
Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com>
|
| | | |
| | | |
| | | | |
* Add missed NACK reasons, update UTs and add minor changes
|
| | | |
| | | |
| | | |
| | | |
| | | | |
* Set is_resuming flag to false
* Change return value in RestoreAppHMIState and StartAppHmiStateResumption
|
| | | |
| | | |
| | | | |
* Add defvalue to policy table schema
|
| | |\ \
| | | | |
| | | | | |
Fix broken Mobile API link in README
|
| | |/ /
| |/| | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
* Add check for non-existent parent ID to AddSubMenuRequest
* fixup! Add check for non-existent parent ID to AddSubMenuRequest
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
There was a problem with duplicate subscriptions to shared data
due to time gap between check for already subscribed apps in
SDLRPCPlugin and further processing of corresponding requests in
WayPointsPendingResumptionHandler. That's why this check was moved
to WayPointsPendingResumptionHandler.
Also refactoring of some methods WayPointsPendingResumptionHandler
class was done to make their logic clearer and easier to understand.
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Add the DialNumber RPC to the Base-4 and Base-6 functional groups
* Move DialNumber into a separate functional group
* Fix added functional group
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Wait for UsbHandler thread end in UsbHandler dtor
The Thread dedicated for handling libusb events should exit
successfully on it's own.
This guarantees that `libusb_close()` being called on all
devices, as well as `libusb_exit()` call is properly sequenced.
Thread exits in recommended by libusb doc way:
1. set exit flag
2. deregister hotplug callbacks, which will wake up
`libusb_handle_events()` once again.
But this commit changes the way the Thread being joined. It
ensures that `join()` is called with `kNoStop` flag, which
prevents force stop using `pthread_cancel()`.
* Using atomic bool for shutdown_requested_ flag
Making UsbHandler::shutdown_requested_ flag atomic to guarantee
mutual access to it.
exchange() call is needed for helgrind to feel sure that there is
no data race.
* Remove redundant DCHECK
This assertion has been exposed by the current fix
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Re-order rai logic for plugins
* style
* Add mock method
* Add method descriptions
* Add timing content
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Fix 3173 bug:multi-thread access one pointer at same time caused a wild pointer problem in HMICapabilitiesImpl class
* modify test code for 3173
* fix Unit tests
* Fix merge conflicts
Co-authored-by: zhangwenqin <zhangwenqin@iauto.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In a while called method locks mutex for protecting
data and if USB connection is failed it means that
method will call and will try to lock the same mutex
again. That's why this mutex should be recursive.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Fix streamer activity in case of suspend
The problem of the current implementation is that SDL
kills streaming thread responsible for sending a/v
streaming data to HMI when streaming timeout expires.
This issue is observed when mobile app dumps 10-seconds
audio file to SDL during 2 seconds and after that app
does not send any data. In that case HMI will play
audio file during 2 seconds + timeout = 5 seconds. At
the 5th second SDL just kills streaming thread with all
pending messages, however audio service is still open.
As a result not a whole audio file is played.
The correct behavior from SDL side in that case is not
kill streaming thread when streaming timeout was expired.
SDL should kill streaming thread only when service is
actually stopped.
Current SDL behavior was updated to align with a correct
behavior described above.
* Adjust fix to work with audio socket streaming
* Revert "[WIP] initial fix of socket streaming by guessing playback time (#3057)"
This reverts commit 7c442abce73c462e7724a64b3ce037bcb8e9093e.
* Remove unused code after revert
* Fix style issues
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
GetInteriorVehicle request RPC should process the next result codes as success:
WRONG_LANGUAGE, RETRY, SAVED.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit makes no functional or insecure changes.
It only prevents possible SIGSEGVs.
Don't use raw pointers to WebsocketSession in almost all cases.
For function arguments, pointers changed to references
for preventing null pointer occurrence.
For containers pointers changed to weak_ptr for preventing
expired references.
Removed wrong inheritance for CMessageBroker
|
| | |\ \
| | | | |
| | | | |
| | | | |
| | | | | |
LuxoftSDL/fix/move_duplicate_conversion_functions_to_message_helper
Move duplicate conversion functions to Message Helper
|
| | | | | |
|
| | | |\ \
| | | |/ /
| | |/| |
| | | | | |
fix/move_duplicate_conversion_functions_to_message_helper
|