| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |\
| |
| | |
Properly populate rejectedParams for EndServiceNACK
|
| | |\ |
|
| | | | |
|
| |\ \ \
| |_|/
|/| | |
Fix issue where building the logger would rebuild all other 3rd party libraries
|
| | |\ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Fix LastState instance thread safe access
Fix LastState instance thread safe access
and update dictionary usages in app launch and resumption.
There was implemented LastStateWrapper class for providing exclusive
thread safe access to LastState instance from all places in the program.
Also passing LastState object was replaced with LastStateWrapper object
to avoid direct access to LastState instance from another components.
Thus there were a small updates in architecture of AM and TM classes.
* Fixed mocks and unit tests related to LastState class
* Add LastStateWrapper interface and LastStateWrapperPtr typedef
* Fix tests after adding LastStateWrapper interface
* Deprecate methods with changed signature
* Adapt RCConsentManager according to LastState functionality
There are some deprecated methods in LastState that are used by
RCConsentManager.
In addition, LastState::dictionary method returns reference to
dictionary no more - only copy of latter.
* Adapt AppServiceManager to new LastState usage
* Adapt transport manager to new LastState usage
* Adapt RPCPlugin to new LastState usage
* Adapt AppServiceRPCPlugin to new LastState usage
* Adapt RCRPCPlugin to new LastState usage
* Adapt SDLRPCPlugin to new LastState usage
* Adapt VehicleInfoPlugin to new LastState usage
* Adapt existing unit tests
* Fix ENABLE_LOG off build failure
* Replace old-style loops with range based
* Fix potential mutex deadlocks in resumption storage
* Add Mutable Data Accessor.
The entity that provides the thread-safe mutable access to data.
* Fix submodule issues after merge
Co-authored-by: Alexander Kutsan (GitHub) <akutsan@luxoft.com>
Co-authored-by: Yaroslav Mamykin (GitHub) <33784535+YarikMamykin@users.noreply.github.com>
Co-authored-by: Igor Gapchuk (GitHub) <41586842+IGapchuk@users.noreply.github.com>
Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Fix core crash on accesing the application pointer.
The issue will occur when the state controller gets an application
collection from the application manager and tries to apply a functor
to each application.
One of the application's pointers could be uninitialized and the state
controller should check if pointer is valid.
* fixup! Fix core crash on accesing the application pointer.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Prevent creation of SSL context during certificate updating
Added sync primitive into CryptoManagerImpl as class member
and added lock into OnCertificateUpdated, CreateSSLContext
methods for preventing race conditions
* fixup! Prevent creation of SSL context during certificate updating
Co-authored-by: Yevhenii Dementieiev (GitHub) <57259850+ydementieiev@users.noreply.github.com>
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* Fix for SDL Core crash after quickly and repeatedly connect/disconnect USB cable
* modify code style
|
| | |/
|/|
| |
| | |
Co-authored-by: Collin <iCollin@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix for No mechanism for managing the switching sequence(Start/StopStreaming)
* change the code style
* change the code style
* Update src/components/application_manager/src/application_manager_impl.cc
Co-Authored-By: Collin <iCollin@users.noreply.github.com>
* Update src/components/application_manager/src/application_manager_impl.cc
Co-Authored-By: Collin <iCollin@users.noreply.github.com>
* change to cpp11 based loop and remove the useless else
Co-authored-by: Collin <iCollin@users.noreply.github.com>
|
| | |
| |
| | |
remove reference to deprecated StyledStreamWriter
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* #3211 [SDL 0234] Proxy Library RPC Generation
* Split existing `InterfaceGenerator` into `InterfaceParser` and `InterfaceGenerator`
* Moved InterfaceParser into `smartdevicelink/rpc_spec`
* Rejoined `InterfaceParser` as `tools/rpc_spec` Git submodule from `smartdevicelink/rpc_spec`
* Refactored `InterfaceGenerator` to use Parser from `tools/rpc_spec` Git submodule and be compatible with Python 3.5
* Replaced url SSH to HTTPS in submodule
* refactoring according to comments in pull/202
* move common parsers to rpc_spec repo
* change cmake argument
* fix cmake argument list
* fix cmake argument list
* fix cmake argument list
* small refactoring as requested in code review
* adding *.xsd
* adding *.xsd
* xmlschema dependency added for python interface generator
* align changes from parser rpc_spec
* align changes from parser rpc_spec
* align with rpc_spec
* moved xmlschema python dependency installation to build script
* installing all python requirements
* Corrected requirements file
* Additional dependencies for requirements
* Correcting requirements
* adding jsoncpp submodule
* fixing unit tests
* fixing unit tests
* rename CodeFormatAndQuality
* updated rpc_spec reference
Co-authored-by: Aleksandr Mishchenko <amishchenko@luxoft.com>
Co-authored-by: Kostiantyn Sologubov <ksologubov@luxoft.com>
|
| | | |
|
| |\ \
| | |
| | | |
update license
|
| | | | |
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Send UPDATING state after receiving a successful SDL.PolicyUpdate response
* Restart the timeout in case SDL received BC.OnSystemRequest with type PROPRIETARY or HTTP from HMI in UPDATING state
* Fix UPDATE_NEEDED - UPDATING sequence in case of second app register
* Add unit tests
* fixup! Send UPDATING state after receiving a successful SDL.PolicyUpdate response
* fixup! Restart the timeout in case SDL received BC.OnSystemRequest with type PROPRIETARY or HTTP from HMI in UPDATING state
Co-authored-by: Yaroslav Mamykin (GitHub) <33784535+YarikMamykin@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix processing of VR only PI response
1. Added sending of UI_ClosePopUp request to HMI after
VR.PerformInteraction response.
2. Added sending response with choice_id to mobile in case of successful
VR.PerformInteraction response
* Fix adapt existing unit tests
* Add new unit tests
* fixup! Fix processing of VR only PI response
* fixup! Fix processing of VR only PI response
* fixup! Add new unit tests
* fixup! Fix processing of VR only PI response
* fixup! Add new unit tests
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
* Delete old version of jsoncpp lib and add new like submodule, change deprecated interface for new.
* Logs output without extra special symbols
* fixup! Delete old version of jsoncpp lib and add new like submodule, change deprecated interface for new.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix timeout for close application
When application restarts streaming service, while close app timeout
not expired, SDL will unregister application.
* fix clear timer pool
* Fix clear pool timer to be periodic
* fixup! Fix clear pool timer to be periodic
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Update malformed message's criterias according the documentation
* Change UT according new rules
* Unify logger's message
* fixup! Update malformed message's criterias according the documentation
* Fix build with ENABLE_LOG_OFF
* fixup! Fix build with ENABLE_LOG_OFF
* Add short condition for variable and fix build with Enable_log off
* fixup! Add short condition for variable and fix build with Enable_log off
Co-authored-by: Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com>
Co-authored-by: Yevhenii Dementieiev (GitHub) <57259850+ydementieiev@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
adapter (#3134)
* Allow code compatibility with flag TELEMETRY_MONITOR On/Off
Added missed virtual keyword for SetTelemetryMonitor method.
Updated related interface and dependent mock classes
* Add missed mock classes and update existing ones
* Update TCP transport Adapter unit test logic
Replaced mock object with real TCP transport adapter
to be used with mock object to allow unit testing.
* Rework TransportManagerDefault class to allow its unit testing
1) Replaced local variables(raw pointers) by class members
2) Reworked Transport Manager unit tests
* Add missed logic when Connection failed to ThreadedSocketConnection::threadMain()
Added missed "else" case to ThreadedSocketConnection::threadMain()
when connection establishing is failed
* Add the Terminate method to the connection interface.
To avoid race condition and deadlock during destroying object of TcpServerOriginatedSocketConnection the Terminate method was added to the ThreadedSocketConnection.
Race condition can happen between the call FindEstablishedConnection from TcpTransportAdapter::Store and Terminate from TransportAdapterImpl.
In case when TcpServerOriginatedSocketConnection destroyed after FindEstablishedConnection called we will get deadlock , because in destructor of TcpServerOriginatedSocketConnection we call StopAndJoinThread(), but now we are in the same thread.
Co-authored-by: Andriy Byzhynar (GitHub) <AByzhynar@luxoft.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* SDL restores AddCommands in the order they were created
SDL generates and assigns internal_id for each AddCommand from mobile app
and restore AddCommands by this internal_id
* Fix message helper unit tests
* Fix resume controller unit tests
* Fix cmd_id in SendAddVRCommandToHMI
* fixup! SDL restores AddCommands in the order they were created
* Go to the next command if command id missing
Co-authored-by: Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com>
Co-authored-by: Yevhenii Dementieiev (GitHub) <57259850+ydementieiev@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* fix UpdateDeviceList content
fixed UpdateDeviceList contetnt for case, when device connected
ower tcp protocol was unexpectedly disconnected and
device has only one connection.
* answering review
* Fix redundant UpdateDeviceList issue
Co-authored-by: Mykola Korniichuk (GitHub) <42380041+mkorniichuk@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Handle wrong message type from HMI was added
* Fixed ignored request from HMI in case of invalid data.
utils::SharedPtr replaced with std::shared_ptr for ProcessMessageFromHMI
and ProcessMessageFromMobile in rpc_handler.
New parameter "report" in HandleWrongMessageType in order to get a proper
error message in response.
Redundant log message removed from sdl_activate_app_request.
INVALID_DATA now will be sent as a response to HMI
in case of wrong parameters in the request.
Co-authored-by: Mykola Korniichuk (GitHub) <42380041+mkorniichuk@users.noreply.github.com>
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix SDL app stream processing with non-streamable state.
For determining whether an application is allowed to stream,
SDL considered only HMI_Level.
Now streaming state is also taken into account.
Fix notifications for not streamable apps
* Fix unit tests after rename function in new inplimentation
* Minor fixes
* const parameter for StartEndStreamTimer
* Fix logging messages
* end_stream_timer shanged to SingleShot
* fix unit test for state_controller
* fixup! Minor fixes * const parameter for StartEndStreamTimer * Fix logging messages * end_stream_timer shanged to SingleShot * fix unit test for state_controller
* rename HMILevelAllowsStreaming -> HMIStateAllowsStreaming
* fixup! Fix SDL app stream processing with non-streamable state.
Co-authored-by: Yevhenii Dementieiev (GitHub) <57259850+ydementieiev@users.noreply.github.com>
|
| |/ |
|
| |\
| |
| | |
Fix/sdl crash during set app icon
|
| | |
| |
| |
| | |
Update SetAppIcon request according to the changes in file system
|
| | | |
|
| | |
| |
| | |
* Fix versioning appliance for vehicle data
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix misplaced OnDataVideoStreaming
OnDataVideoStreaming notification was sent for each application
independently. Now one would be sent from application_manager with
respect to service status
* Fix OnDataStreamingNotification
* rename parameters
* fixup! rename parameters
|
| |\ \
| | |
| | | |
Fix BC.PolicyUpdate double sending
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* Fix wrong success code for ReadDID request truncated data
* fixup! Fix wrong success code for ReadDID request truncated data
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
App. (#2365)
* Bugfix `SDL forwards OnButtonEvent notification`
Bugfix `SDL forwards OnButtonEvent notification
of CUSTOM_BUTTON to BACKGROUND App.`
Issue number #967
* Check that SDL forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App
* fixup! Bugfix `SDL forwards OnButtonEvent notification`
Co-authored-by: Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com>
Co-authored-by: Igor Gapchuk (GitHub) <41586842+IGapchuk@users.noreply.github.com>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
A crash occurs because we use an overload of
boost::beast::websocket::stream::write without error_code.
After the error occurs we add an error message to the logger
and continue working.
In SendMessage_UnpreparedConnection_WithoutFall test we use sleep because we can't ability to catch processed event.
This issue is that LoopThreadDelegate requires a WebsocketSession to work.
WebsocketSession init LoopThreadDelegate with 'this', so we have no way to simulate it without additional installers.
In future, to avoid sleep, we can:
Create a wrapper class for the socket, then provide the socket layout for the WebsocketSession in unit tests and wrap the following interfaces:
async_read
write down
get_executor
async_accept
|
| | | |
| | |
| | |
| | |
| | |
| | | |
In case of duplicated name of application SDL didn't increment
'count_of_rejections_duplicate_name' counter in policy db.
Added incrementing of counter.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
There is the issue when SDL incorrect validates display capabilities
as a Smart Object. The route cause of the issue is the missed "alertIcon"
parameter in mapping <string value, hmi_apis::Common_ImageFieldName::eType>
. During validation, a Smart Object SDL can't find mandatory parameter and
fail validation. As a result, SDL doesn't set display capabilities into
the HMI capabilities entity.
|
| | |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Fix undefined behavior after SO cleanup
After cleanup_data() call, delete is used to deallocate
memory by pointer, however pointer still hold a value
of address of deallocated memory. By that reason SO
still have an ability to access that data what might
cause UB including core crash in the random places.
This function has been updated to set pointers to NULL
as well as smart object type to prevent accident access
to deallocated memory.
* fixup! Fix undefined behavior after SO cleanup
* Fix CSmartSchema corruption while copying
The issue is that HMICapabilities class is storing
different capability fields by raw pointer. In some
cases when one thread is trying to copy smart object
by pointer, another thread may try to set the same
field and destroys a previous one. This may cause a
half-copied smart object instance, like smart object
with the schema pointing to already destroyed schema
item. When such smart object is destroyed, we may see
a core crash or unexpected SDL behavior.
To avoid such issues, raw pointers were raplced with
shared pointers, set by swap() operation. This provide
a guarantee that object will not be destroyed while
copying by some thread.
|
| |\ \
| |/
|/| |
Update develop with hotfix 6.0.1
|
| | |\
|/ /
| | |
Hotfix Release 6.0.1
|
| | |\
| | |
| | | |
Fix application of hybrid app preference parameter for cloud applications
|
| | | | |
|
| | | | |
|
