summaryrefslogtreecommitdiff
path: root/src/components/application_manager
Commit message (Collapse)AuthorAgeFilesLines
* Wrap unused param (while logger is off) in ifdefhotfix/14Justin Dickow2016-04-121-1/+2
| | | | Have to do this because compiler treating warnings as errors
* Merge pull request #456 from ↵4.0.10AGaliuzov2016-04-0612-10/+527
|\ | | | | | | | | dev-gh/hotfix/DialNumber_functionality_implementation Adds implementation of DialNumber RPC
| * More style changesAndrey Oleynik2016-04-064-7/+7
| |
| * Code style changesAndrey Oleynik2016-04-052-2/+2
| |
| * Adds latest number validation changesAndrey Oleynik2016-04-041-1/+1
| | | | | | | | | | | | | | | | | | | | According to latest information number should be verified as: "Phone number is a string, which can be up to 40 chars. All characters shall be stripped from string except digits 0-9 and * # , ; +" Implements: APPLINK-11266
| * Adds implementation of DialNumber RPCAndrey Oleynik2016-04-0412-8/+525
| | | | | | | | | | | | | | DialNumber allows to initiate call from user's device trough device attached to the system. Implements: APPLINK-11266
* | Merge pull request #449 from ↵AGaliuzov2016-04-041-8/+13
|\ \ | | | | | | | | | | | | okoreniak/hotfix/Fixed_SDL_do_not_send_two_OnHMIStatus_notifications_to_LAUNCHED_App Fixed SDL to do not send two OnHMIStatus notification to launched app…
| * | Fixed SDL to do not send two OnHMIStatus notification to launched applicationOleksandr Koreniak2016-03-301-8/+13
| | | | | | | | | | | | | | | Updated processing of RegisterAppInterfaceRequest::SendRegisterAppInterfaceResponseToMobile(). Added test for the application hmilevel before assigning the default hmilevel and performing resumptions.
* | | Merge pull request #433 from ↵AGaliuzov2016-04-041-73/+416
|\ \ \ | |_|/ |/| | | | | | | | okoreniak/hotfix/Fixed_Validator_In_SystemRequest_QUERY_APPS_processing Hotfix/fixed validator in system request query apps processing
| * | Code for synonyms validation moved to functionOleksandr Koreniak2016-04-041-103/+127
| | |
| * | Added validator in processing of SystemRequest(QUERY_APPS)Oleksandr Koreniak2016-04-011-77/+396
| | | | | | | | | | | | Added class QueryAppsDataValidatorTemplate<SmartObject> to validate request data.
* | | Remove line containing PRId64hotfix/446Justin Dickow2016-03-291-1/+0
| | | | | | | | | | | | This was causing issues with certain compilers, and the code is never run.
* | | Merge pull request #430 from ↵AGaliuzov2016-03-241-0/+8
|\ \ \ | | | | | | | | | | | | | | | | dev-gh/hotfix/Buttons.GetCapabilities_response_handling_fix Fixes ressetting of default buttons capabilities
| * | | Fixes ressetting of default buttons capabilitiesAndrey Oleynik2016-03-231-0/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | In case of error response for Buttons.GetCapabilities SDL was resetting default buttons capabilities (from JSON file) to null. This change adds checking of response result code to avoid that issue. Closes-bug: APPLINK-22416
* | | Following APPLINK-21499 discussion, set mobile command timeoutAsen Kirov2016-03-231-1/+2
|/ / | | | | | | | | | | | | | | | | | | equal to slider RPC timeout plus default command timeout Conflicts: src/components/application_manager/src/commands/mobile/slider_request.cc Conflicts: src/components/application_manager/src/commands/mobile/slider_request.cc
* | Fixes issue with SDL data resetting during MASTER_RESETAndrey Oleynik2016-03-112-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were two deadlocks during unregistering of all apps because of MASTER_RESET: - ApplicationManagerImpl::UnregisterAllApplications vs ConnectionHandler::CloseSessionServices - ApplicationManagerImpl::UnregisterApplication vs ApplicationManagerImpl::RemoveAppFromTTSGlobalPropertiesList That was causing reset flow blocking and data had not been cleaned up completely. Closes-bug: APPLINK-22117 Conflicts: src/components/application_manager/include/application_manager/application_manager_impl.h src/components/application_manager/src/application_manager_impl.cc src/components/connection_handler/include/connection_handler/connection_handler_impl.h src/components/connection_handler/src/connection_handler_impl.cc
* | Merge pull request #404 from smartdevicelink/hotfix/fix_pt_sendMessagetoSDK4.0.5AGaliuzov2016-03-072-6/+3
|\ \ | | | | | | Fix for sending policy table snapshot (Genivi implementation)
| * | Fix for sending policy table snapshothotfix/fix_pt_sendMessagetoSDKJackLivio2016-03-032-6/+3
| |/ | | | | | | | | | | Policy handler was checking an unimplemented contained last_used_app_ids_. Replaced with 'GetAppIdForSending() to properly find app id to send policy snapshot to. Also in message_helper.cc, I fixed the url paremeter to what actually corresponds to onSystemRequests in the mobile_api.xml
* | Add NOT to checking if app is in full for OK button notificationhotfix/ok_button_notification_fixJackLivio2016-02-151-1/+1
|/
* Merge pull request #375 from smartdevicelink/hotfix/fix_IviInfoUpdated_queryAGaliuzov2016-02-091-1/+1
|\ | | | | Fixed IviInfoUpdated App Query
| * Fixed IviInfoUpdated App Queryhotfix/fix_IviInfoUpdated_queryJackLivio2016-02-031-1/+1
| | | | | | | | The query in IviInfoUpdated was only returing connected navigation apps. It now returns apps that are subscribed to vehicle info.
* | Merge pull request #349 from ↵AGaliuzov2016-02-081-39/+37
|\ \ | |/ |/| | | | | LuxoftSDL/hotfix/SDL_transfers_PlayTone_parameter_to_HMI_in_Speak_request_from_mobile SDL transfer PlayTone parameter to HMI in speak request.
| * SDL transfer PlayTone parameter to HMI in speak request.dtrunov2016-01-251-39/+37
| | | | | | | | | | | | | | Added ability to remove fake parameter according with MOBILE_API.xml Closes-bug:[APPLINK-17980](https://adc.luxoft.com/jira/browse/APPLINK-17980)
* | Merge pull request #359 from ↵Artem Nosach2016-02-021-2/+2
|\ \ | | | | | | | | | | | | LuxoftSDL/hotfix/Fix_build_fail_with_disabled_logging Fix build fail with disabled logging
| * | Fix build fail with disabled loggingboniaslasher2016-01-271-2/+2
| | | | | | | | | | | | | | | | | | Build fail due to missing link to map definition, which automatically added when logging enabled. Also removed c-style call.
* | | Change updateRequestTimeout to TerminateRequestHerasym Oleh2016-01-281-2/+2
| | | | | | | | | | | | | | | | | | | | | Fix incorrect behavior Related: APPLINK-17652
* | | Add DeleteChoices to ontimeOutHerasym Oleh2016-01-281-0/+1
|/ / | | | | | | | | | | | | | | | | Fix defect: SDL doesn't send VR.DeleteCommand to HMI in case ChoiceSet commands and HMI sends a VR.AddCommand responses with error Related: APPLINK-17652
* | Merge pull request #337 from smartdevicelink/hotfix/dcheck_crash4.0.3AGaliuzov2016-01-251-2/+0
|\ \ | | | | | | Remove DCHECKs that seem to always be tripping everyone up
| * | Remove DCHECKs that seem to always be tripping everyone uphotfix/dcheck_crashJustin Dickow2016-01-191-2/+0
| |/
* | removed redundant includes fixes #223hotfix/redundant_includesJustin Dickow2016-01-191-2/+0
|/
* Merge remote-tracking branch 'genivi-upstream/release/4.0.0'4.0.1Artem Nosach2015-12-145-5/+35
|\
| * Merge pull request #284 from LuxoftSDL/hotfix/Core_dump_if_make_MASTER_RESETAGaliuzov2015-12-072-2/+14
| |\ | | | | | | Hotfix/core dump if make master reset
| | * Increase of scope of lock in method Handle()dtrunov2015-12-011-6/+5
| | | | | | | | | | | | | | | | | | Increase of scope of lock in method Handle() in order to exclude problem with core crash. Closes-bug: APPLINK-17367
| | * Fix core crash during MASTER_RESETdtrunov2015-12-012-2/+15
| | | | | | | | | | | | | | | | | | Forbid processing message from mobile if Core begins stopping of work. Closes-bug: APPLINK-17367
| * | Merge pull request #282 from ↵AGaliuzov2015-12-041-2/+2
| |\ \ | | | | | | | | | | | | | | | | LuxoftSDL/hotfix/SDL_writes_to_DB_incorrect_value_as_imageType_after_IGN_CYCLE Add checking of SO field.
| | * | Add checking of SO field.dtrunov2015-12-011-2/+2
| | |/ | | | | | | | | | | | | | | | | | | Add checking of SO field during creation of request to HMI. If user calls to nonexistent field from SO, SO creates this field with default value. Closes-bug: APPLINK-13006
| * | Fix deadlocks in AM when stopping SDL and log socket endless blockingAsen Kirov2015-11-303-1/+19
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we stop AM we unregister apps and call StopSavePersistentDataTimer(). If the timer callback by chance is active at this moment we can have a deadlock between AM and ResumeController. To prevent it limit the lifetime of ApplicationListAccessor and allow ResumeController to finish his job. Another option is to move resume_controller().OnSuspend() call before unregitering apps. Another deadlock problem arises when while destroying AM an app tries to register (3 seconds before that actually). Then AM tries to destroy ResumeController, but RC is in ApplicationResumptiOnTimer() callback and tries to get AM instance, and blocks on AM singleton's lock. The fix is to StopRestoreHmiLevelTimer() before destroyng AM. A third reason for SDL not stopping normally is when a logger is writing on a socket, but the client on the other side is not reading (and is not closing the socket neither). Then write() blocks and logger, and therefore SDL, cannot be stopped. The fix is to put timeout option to the socket.
* | Send policy app id during application registrationagaliuzov2015-11-192-2/+1
|/ | | | | | | | There was the bug in notification parameter naming the SmartObject has removed invalid parameter `policyAppId` the proper name for the parameter is `policyAppID`. Closes-Bug: [APPLINK-18660](https://adc.luxoft.com/jira/browse/APPLINK-18660)
* Merge pull request #250 from ↵AGaliuzov2015-10-274-4/+30
|\ | | | | | | | | LuxoftSDL/hotfix/Fix_race_condition_in_Singleton_and_AppManager_failure_to_stop Fix race condition in Singleton and Application Manager failure to stop
| * Fix race condition in Singleton and Application Manager failure to stopAsen Kirov2015-10-224-4/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When SDL is started, but the HMI is not, and we try to register a mobile app (RegisterAppInterfaceRequest), then in this situation SDL can't be stopped with ctrl+C - it enters in an endless cycle. The reason for the problem is that we can't delete the AM, because the RequestController thread of AM is still running (waiting in a cycle for the HMI to respond). Also a second AM is created, because in the Singleton we set to 0 the instance pointer before deleting it and someone calls instance() before destroy() finishes, because there is no common lock. The separate locks create a race condition. The fix is to use a single mutex for the Singleton methods, introduce a is_stopping_ flag in AM, set it to true in AM's Stop() method, and also destroy RequestController's thread pool there. Then check stop flag in RegisterAppInterfaceRequest::Run() and exit the HMI waiting cycle there.
* | Merge pull request #248 from ↵AGaliuzov2015-10-271-21/+38
|\ \ | | | | | | | | | | | | LuxoftSDL/hotfix/OnVehicleData_send_to_app_that_unsubscribed_vehicleData Fix OnVehidacleData sended to app with unsubscribed vehicleData
| * | Fix OnVehidacleData sended to app with unsubscribed vehicleDataValeri Prodanov2015-10-211-21/+38
| |/ | | | | | | | | | | | | The vehicleData parameters in the notification massage (received from hmi) are used for finding the subscribed applications i.e all app. id`s are saved as well as every subscribed parameter. Then for every app. specific notification is sent.
* | Moved, Enabled and partially refactored Unit testsiAndrew52015-10-2014-0/+5217
| |
* | Disabled some tests in CmakeLists to enable compilation with TESTSiAndrew52015-10-191-2/+2
|/
* Fixes core dump while sending ResetGlobalPropertiesAndrey Oleynik2015-10-021-3/+3
| | | | | | | Due to response deletion by request controller there is possibility of using object members being destructed. Closes-bug: APPLINK-16029
* Merge pull request #220 from LuxoftSDL/hotfix/SystemRequest_crash_sdl4.0AGaliuzov2015-09-252-22/+19
|\ | | | | Fix crash when processing SystemRequest with QUERY_APPS
| * Fix crash when processing SystemRequest with QUERY_APPSAnatoly Leshin2015-09-212-22/+19
| | | | | | | | | | | | Process tts_name as string, not as array. tts_name in QueryApps json file is "string" according to APPLINK-11731, tts_name in UpdateApplist is "array of strings" according to API.xml Fix: APPLINK-15783
* | Merge pull request #207 from LuxoftSDL/hotfix/Fix_HMI_level_setting_on_RAIAGaliuzov2015-09-253-15/+14
|\ \ | | | | | | Fixes setting of HMI level for consequtive permissions validation.
| * | Fixes setting of HMI level for consequtive permissions validation.Andrey Oleynik2015-09-023-15/+14
| |/ | | | | | | | | | | | | | | | | | | | | | | HMI level must be set for just registred application since permissions validation rely on HMI level. Fixes: APPLINK-15635 Conflicts: src/components/application_manager/src/commands/mobile/register_app_interface_request.cc src/components/application_manager/src/commands/mobile/register_app_interface_response.cc src/components/application_manager/src/resumption/resume_ctrl.cc
* | Merge pull request #206 from ↵AGaliuzov2015-09-251-12/+13
|\ \ | | | | | | | | | | | | LuxoftSDL/hotfix/Fix_crash_on_getting_connection_key Fixes crash while trying to get app connection key