summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix stylefix/style_3417Jacob Keeler2020-06-101-2/+1
|
* Merge pull request #3417 from smartdevicelink/feature/daemon_scriptJacob Keeler2020-06-097-30/+187
|\ | | | | Add SDL Core Daemon Script
| * Fix usage logJacob Keeler2020-06-091-1/+1
| |
| * Remove debug logJacob Keeler2020-06-091-1/+0
| |
| * Add daemon.sh to unify scripts for external and regularfeature/daemon_scriptjacobkeeler2020-06-084-196/+155
| |
| * Fix missing -H flag in policy server dependency stepJacob Keeler2020-06-081-1/+1
| |
| * Add start.sh which uses the new core.shjacobkeeler2020-06-082-0/+22
| |
| * Address commentsjacobkeeler2020-06-082-43/+43
| |
| * Fix broken return code in core_stopjacobkeeler2020-06-022-2/+2
| |
| * Add features to make the daemon script more reliablejacobkeeler2020-06-023-32/+154
| | | | | | | | | | | | | | Create dedicated PID file directory Check running processes before starting or stopping Core Add `kill` command to destroy all running instances of Core Change main SDL thread to be identifiable by name
| * First version of daemon script from proposaljacobkeeler2020-05-215-30/+85
| |
* | fix build error in http unit tests (#3422)Collin2020-06-091-1/+3
| | | | | | | | | | * fix build error in http unit tests * fix style
* | Sdl must store onWayPointChange internally (#2490)Pavel Zhdanov (GitHub)2020-06-088-11/+93
| | | | | | | | | | | | | | | | | | | | | | | | * Add saving way points during notification - When OnWayPointNotification appeared the wayPoint is saving - When new app subscribing on wayPointNotification stored way points are sending to app * Add unit tests * fix style, update to remove (uint32_t) overload of SubscribeAppForWayPoints Co-authored-by: collin <collin+i@collinmcqueen.com>
* | Bugfix/protocol handler memory issues (#2270)Sho Amano2020-06-064-8/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix: memory leak in ProtocolHandlerImpl::NotifySessionStarted() This is detected by valgrind. * fix: buffer over-run in ProtocolPayloadTest This is an issue of unit test implementation and only affects unit testing. * fix: invalid memory accesses in ProtocolHandlerImplTest These affect unit testing only. Invalid memory accesses occurred because the mock class didn't configure protocol version field properly. * fix: valgrind warning in IncomingDataHandlerTest This is not actually an issue. valgrind reported a warning since the memory area was not initialized. Co-authored-by: Collin <iCollin@users.noreply.github.com>
* | fix: memory leaks in data_resumption_test (#2273)Sho Amano2020-06-062-1/+37
| | | | | | | | | | This issue only affects unit testing. Co-authored-by: Collin <iCollin@users.noreply.github.com>
* | Remove defunct `appID` parameter from UnsubscribeVehicleData (#2331)Jacob Keeler2020-06-051-3/+0
| |
* | trigger ptu on startup only in proprietary policy mode (#3415)Collin2020-06-051-1/+1
| |
* | Fix: handle network interface removal (#2759)Sho Amano2020-06-045-101/+200
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix: IP address isn't notified when network interface is removed When RTM_DELLINK is notified, corresponding network interface has been already destroyed so if_indextoname() won't return a valid name. To handle such case, status_table_ is updated to use network interface index as key. * fix: generate DeviceDisconnected event when network interface is removed * fix a few comments * Reflect review comments - Add const where applicable - Add more descriptions * Reflect code review Replace FindInterfaceStatus() with std::find_if() call. * Reflect code review Remove unnecessary include. * fix style issue
* | Add workaround to handle problem, caused by gcc compiler (#3418)Yana Chernysheva (GitHub)2020-06-041-1/+8
| |
* | fix: memory leak in DynamicApplicationDataImpl class (#2259)Sho Amano2020-06-031-0/+15
| |
* | Fix defect: revoked app still be resumed as FULL. (#2546)Igor Gapchuk (GitHub)2020-05-294-8/+51
| | | | | | | | | | | | App was resumed with Full HMI Level when it has been flagged as revoked in the policy table. This PR adds a check that app is not revoked to continue with HMI Level resumption.
* | Fix onsystemrequest retry logic (#3400)JackLivio2020-05-2811-25/+100
| | | | | | | | | | | | | | | | | | * Fix onsystemrequest retry logic * Cache policy update file from HMI for retries * Fix issues with previous commit Co-authored-by: jacobkeeler <jacob.keeler@livioradio.com>
* | ensure delegates are destroyed before their threads (#3388)Collin2020-05-282-1/+4
| |
* | Fix style (#3398)Mykhailo Vorobiov (GitHub)2020-05-274-27/+36
| | | | | | The style has been broken on develop branch - commit f5cfd151d82
* | Increase policy unit-tests speed (#2315)Pavel Zhdanov (GitHub)2020-05-228-271/+755
| | | | | | | | | | | | | | | | | | | | | | * Increase speed for Policy unit-tests: External policy Increase speed for unit-tests: - External policy tests - Regular policy tests * Increase speed for Policy unit-tests: Regular policy Co-authored-by: Collin <iCollin@users.noreply.github.com>
* | Rewrite unit tests for PolicyManagerImpl from External flow (#3353)Yana Chernysheva (GitHub)2020-05-2149-61422/+5217
| | | | | | | | | | | | | | | | | | | | | | | | | | * Add new file with unit tests for CacheManager * Add new tests for PolicyManagerImpl * Add new preloaded tables * Add changes in CMakeLists.txt * Delete unused files * Re-add file with test
* | Rewrite unit tests for PolicyManagerImpl from Proprietary flow (#3352)Yana Chernysheva (GitHub)2020-05-2123-24251/+6389
| | | | | | | | | | | | | | | | | | | | | | | | | | * Add new file with unit tests for CacheManager * Add new tests for PolicyManagerImpl * Add changes in CMakeLists.txt * Add new preloaded tables * Delete unused files * Re-add test files
* | Fix messages protocol version check (#2771)Stanislav Kobziev (GitHub)2020-05-217-17/+62
|/ | | | | | | | | | | | * Fix messages protocol version check Fixed the messages protocol version check and name of ProcessData method parameter 'result' was changed to 'out_result' for more clarity * fix copy-paste error and fix style Co-authored-by: Collin <iCollin@users.noreply.github.com> Co-authored-by: collin <collin+i@collinmcqueen.com>
* unsubscripe app from waypoints when it is unregistered (#3373)Collin2020-05-214-19/+71
| | | | | | | * unsubscripe app from waypoints when it is unregistered * address review comments * fix style
* Forward OnButtonPressNotifications where HMI specifies appID to apps in ↵Collin2020-05-202-3/+5
| | | | | | | | | LIMITED HMI Level (#3378) * initialize app in non custom button path of on_button_press run * fixup! initialize app in non custom button path of on_button_press run fix unit tests with changes in run
* Fix/gufm check syntax (#3369)Collin2020-05-206-28/+72
| | | | | | | | | | | * Fix implementation for SDLGetUserFriendlyMessage * Unit test for SDLGetUserFriendlyMessage: checking syntax * Changes according to comments * add invalid syntax error response info Co-authored-by: ZhdanovP <pzhdanov@luxoft.com>
* fix memory error in TcpDevice::IsSameAs (#3386)Collin2020-05-201-2/+2
|
* Release delegate only after full thread stopping (#2534)Pavel Zhdanov (GitHub)2020-05-201-1/+1
|
* Fix SendUpdateAppList and SendOnAppUnreg order (#2517)Kostiantyn Grygoriev2020-05-191-5/+7
| | | Co-authored-by: collin <collin@livio.io>
* Add RPCService unit tests (#3250)Yana Chernysheva (GitHub)2020-05-184-0/+900
| | | | | | | | | | | * Add RPCService unit tests * Add prerequisites for execution of unit tests * Rearrange and delete unused includes * Add minor changes Co-authored-by: JackLivio <jack@livio.io>
* Change reason to IGNITION_OFF only if reason wasn't defined before (#3274)Yana Chernysheva (GitHub)2020-05-181-2/+6
|
* Fix/issue 1951 (#3371)Collin2020-05-184-19/+56
| | | | | | | | | | | | | | | * remove deadlock danger when removing Connections from a ConnectionMap * fix style, use log debug instead of trace for singular messages * Revert "fix style, use log debug instead of trace for singular messages" This reverts commit c83288ec9fc9007e48044b09ad038edbf612e038. * fix style, use log debug instead of trace for singular messages * restore condition updates lost with merge of old versions Co-authored-by: Frank <fronneburg@xevo.com>
* Fix/Add Startup PTU trigger check (#3363)Shobhit Adlakha2020-05-189-0/+43
| | | | | | | * Add Startup PTU check * Reset Stop PTU etry when first app is registered * Address review comment
* Merge pull request #2553 from smartdevicelink/fix/start_stream_retry_countJacob Keeler2020-05-182-8/+8
|\ | | | | Fix Start Stream retry count
| * Fix Start Stream retry countfix/start_stream_retry_countjacobkeeler2018-08-292-8/+8
| |
* | ResetGlobalProperties doesn't reset HELPPROMPT and VRHELPITEMS to default ↵Collin2020-05-185-8/+100
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | values (#3357) * Fix bug `ResetGlobalProperties` (#2337) Fix bug `ResetGlobalProperties` does't reset `HELPPROMPT` and `VRHELPITEMS` to default values Fix UT's after bugfixing - Fix `ResetGlobalPropertiesRequestTest` - Add mock function into `MockApplicationManagerSettings` Github issue #1306 Co-authored-by: v-malko4 <vmalkov@luxoft.com> * fix vr_help_title SO type, rename help_prompt_ in unit tests to so_help_prompt * fix style Co-authored-by: v-malko4 <vmalkov@luxoft.com>
* | only return if we are sure we are out of space (#3361)Collin2020-05-131-5/+0
| |
* | Revert "RAI reply DUPLICATE_NAME when app has same id and name as existing app"collin2020-05-133-69/+8
| | | | | | | | This reverts commit 236bdbaa283147044a6144f8c96cc43b82ff1014.
* | RAI reply DUPLICATE_NAME when app has same id and name as existing appcollin2020-05-133-8/+69
| |
* | Fix Compile Warnings (#3356)Collin2020-05-112-2/+5
| | | | | | | | | | | | | | * fix warning:maybe-uninitialized in CStringToEnum * fix out of buffer write warning in unit tests * Apply suggestions from code review
* | [SDL4.0]SDL does not create icons folder in case it was removed (#2649) (#3359)Collin2020-05-111-0/+8
| | | | | | | | | | | | | | | | The check for existence directory of app icons folder has been added to SetAppIconRequest. Co-authored-by: ValeriiMalkov <vmalkov@luxoft.com> Co-authored-by: ValeriiMalkov <vmalkov@luxoft.com>
* | Add missing ssl_context.h include in security_manager_listener (#2775)Stanislav Kobziev (GitHub)2020-05-111-0/+1
| |
* | Fixing problem with validation of array (#2669)Oleksandr Nyzhnyk (GitHub)2020-05-083-17/+47
| | | | | | * Fixing problem with validation of array
* | SDL crash during cleanup. (#2621)Mykola Korniichuk (GitHub)2020-05-081-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | * SDL crash during StopComponents() if StartComponents() failed. If resumption_storage_ in ResumeCtrlImpl is failed to initialize, SDL will try to stop components and exit with error code. At this time application manager tries to unregister all applications and call ResumeCtrlImpl::OnIgnitionOff() where resumption_storage_ is used. As a result, a crash will occur. As an example, if AppStorageFolder have no read/write access, resumption_storage_ fails to initialize. Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
* | Update HMI_API.xml (#3354)Collin2020-05-071-1/+1
| | | | | | | | | | | | | | | | | | * Update HMI_API.xml * Update src/components/interfaces/HMI_API.xml Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com> Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>