summaryrefslogtreecommitdiff
path: root/src/components
Commit message (Collapse)AuthorAgeFilesLines
* Fixed another Coverity issue involving mixed enums (CID 80028)hotfix/coverity_fix_mixed_enumsJacob Keeler2016-07-292-7/+6
|
* Fixed unnecessary mixing of enums (CID 80023, 80024, 80026, 80030)Jacob Keeler2016-07-293-19/+14
|
* Merge pull request #727 from BTPankow/hotfix/coverity_resource_leaksJustin Dickow2016-07-271-9/+6
|\ | | | | Change socket close logic to prevent double close (CID 137866)
| * Change socket close logic to prevent double close (CID 137866)Brad Pankow2016-07-271-9/+6
| | | | | | | | | | | | This simplifies the logic used to connect to bluetooth sockets. It prevents the rfcomm_socket from being closed twice or being left unclosed.
* | Merge pull request #714 from BTPankow/hotfix/coverity_resource_leaksJackLivio2016-07-262-0/+4
|\ \ | |/ | | Fix Coverity resource leaks (CIDs 80039, 80040)
| * Close socket connection before returning from function (CID 80039)Brad Pankow2016-07-211-0/+1
| |
| * Free opened file before returning from function (CID 80040)Brad Pankow2016-07-211-0/+3
| |
* | Merge pull request #715 from BTPankow/hotfix/coverity_buffer_terminationJackLivio2016-07-261-1/+3
|\ \ | | | | | | Null terminate char buffer after strncpy (CID 79979)
| * | Null terminate char buffer after strncpy (CID 79979)Brad Pankow2016-07-211-1/+3
| |/
* | Merge pull request #723 from BTPankow/hotfix/coverity_uninitializedJackLivio2016-07-2619-18/+72
|\ \ | | | | | | Fix uninitialized pointer and scalar fields (Coverity)
| * | Initialize binary_data_ to NULL (CID 80088)Brad Pankow2016-07-261-1/+2
| | |
| * | Initialize thread delegate to NULL before use (CID 80087)Brad Pankow2016-07-261-0/+1
| | |
| * | Initialize policy_table_type_ to INVALID_PT_TYPE for Optional template class ↵Brad Pankow2016-07-262-3/+6
| | | | | | | | | | | | (CID 80046, 80047, 80048)
| * | Initialize protocol_version to V2 (CID 80051)Brad Pankow2016-07-261-1/+3
| | |
| * | Initialize Profile members to defaults (CID 80053)Brad Pankow2016-07-261-0/+6
| | |
| * | Initialize state_ctrl_ to constructor parameter (CID 80054)Brad Pankow2016-07-261-1/+1
| | |
| * | Initialize RequestInfo members, add new RequestType (CID 80058, 80066, 80067)Brad Pankow2016-07-262-5/+17
| | |
| * | Initialize is_ready_ before use in functions (CID 80072)Brad Pankow2016-07-261-0/+1
| | |
| * | Initialize is_persistent, is_download_complete, and file_type before use in ↵Brad Pankow2016-07-261-1/+4
| | | | | | | | | | | | functions (CID 80073)
| * | Initialize max_block_size_ before use in functions (CID 80074)Brad Pankow2016-07-261-1/+2
| | |
| * | Initialize app_state_ and connection_id_ before use in functions (CID 80075)Brad Pankow2016-07-261-1/+2
| | |
| * | Initialize choice_set_id_ and is_timed_out_ before use in functions (CID 80076)Brad Pankow2016-07-261-1/+3
| | |
| * | Initialize cmd_id_ and successful_response_received_ before use in functions ↵Brad Pankow2016-07-261-1/+3
| | | | | | | | | | | | (CID 80077)
| * | Initialize port with default value of 0 (CID 80079).Brad Pankow2016-07-261-1/+2
| | |
| * | Add default enum values to initialize StatisticsDelegate (CID ↵Brad Pankow2016-07-262-0/+16
| | | | | | | | | | | | 80080,80081,80082,80083)
| * | Initialize offset_ before use in functions (CID 80084)Brad Pankow2016-07-251-0/+1
| | |
| * | Initialize current_application_ before use in functions (CID 80089)Brad Pankow2016-07-251-1/+2
| |/
* | Merge pull request #724 from BTPankow/hotfix/coverity_int_handlingJackLivio2016-07-261-1/+1
|\ \ | | | | | | Only send frames when POLLOUT set (CID 79995)
| * | Only send frames when POLLOUT set (CID 79995)Brad Pankow2016-07-261-1/+1
| | |
* | | Fix issues with freed string before use (CIDs 80093, 80094)Brad Pankow2016-07-216-11/+6
| |/ |/|
* | Merge pull request #710 from smartdevicelink/masterJackLivio2016-07-2133-6084/+6284
|\ \ | |/ | | Merge Master Into Coverity
| * ParsePTString to Modify string, instead of returning a copyhotfix/add_escape_characters_during_ptu_header_constructionJackLivio2016-06-202-7/+7
| |
| * Parse PT String for HTTP Request BodyJackLivio2016-06-082-2/+19
| | | | | | | | | | | | The "body" portion of the httprequest needs to be a valid string. Before this fix, the policy table content of the request did not have properly escaped double quotes and was causing the JSON received by the mobile side to be considered invalid. The new method "ParsePtString()" adds an escaped backslash during every occurance of a double quote or other escaped backslash. This method also removes any newlines located in the policy table string.
| * Reverts device certificate support.Andrey Oleynik2016-05-175-27/+4
| | | | | | | | This functionality is already present in develop and won't be moved to master hotfix.
| * Fixes policy update triggering on exceedeed kilometers.Andrey Oleynik2016-05-141-0/+1
| |
| * Fixes policy update triggering on application registrationAndrey Oleynik2016-05-141-3/+5
| | | | | | | | | | Update must be triggered only in case of new application registered. Before these changes update has been triggered every time on app registration.
| * Removes non-open-source fields support from policyAndrey Oleynik2016-05-134-112/+21
| | | | | | | | | | | | | | | | | | There were several fields in policy unsupported by open source implementation, which caused policy update/snapshot validation issues. Originally changes fixes APPLINK-17753 Closes-bug: APPLINK-21553
| * Fixes retry sequence implementationAnton Hrytsevich2016-05-138-167/+235
| | | | | | | | | | | | Adopts changes done originally regaring issue: [APPLINK-19543] Closes-bug: APPLINK-24295
| * Updates PolicyManager in order to properly support certificate fieldAleksandr Galiuzov2016-05-135-1596/+1611
| | | | | | | | | | | | | | Also adds some extra logic to support proper update of ModuleConfigs field. Originally changes resolved APPLINK-11891. Closes-bug: APPLINK-24306
| * Fixes setting of default HMI level during application registrationAndrey Oleynik2016-05-1312-4149/+4151
| | | | | | | | | | | | | | | | | | During registration default HMI level must be set before sending of any RPCs to mobile side since every RPC must be validated by policy. These changes have been taken from develop branch and were originally related to APPLINK-20111. Closes-bug: APPLINK-24294
| * Merge pull request #524 from ↵4.0.11AGaliuzov2016-05-101-0/+1
| |\ | | | | | | | | | | | | smartdevicelink/hotfix/Add_isSDLAllowed_to_initializer_list_v2 Add is sdl allowed to initializer list, in proper order
| | * Add is sdl allowed to initializer list, in proper orderhotfix/Add_isSDLAllowed_to_initializer_list_v2JackLivio2016-05-021-0/+1
| | |
| * | Merge pull request #514 from smartdevicelink/hotfix/generate_snapshot_fix_#461AGaliuzov2016-05-102-2/+17
| |\ \ | | | | | | | | Rewrite GenerateSnapshot()
| | * | Rewrite GenerateSnapshot()hotfix/generate_snapshot_fix_#461JackLivio2016-04-262-2/+17
| | |/ | | | | | | | | | | | | | | | | | | | | | This pr is to fix issue #461. SDL core was crashing due to the snapshot returned from CacheManager::GenerateSnapshot() was being ruled as invalid. When generating a snapshot, instead of copying the entire contents of the policy table, I copy all contents except for the messages in consumer friendly messages. Policy table snapshot are not supposed to include the actual messages in consumer friendly messages, just the version number of the messages. I also added an extra condition in ConsumerFriendlyMessages::Validate() that looks to see if messages is initialized. A pt snapshot can have a consumer friendly messages portion, just not the messages included in consumer fiendly messages.
| * | Merge pull request #483 from smartdevicelink/hotfix/14AGaliuzov2016-05-101-1/+2
| |\ \ | | | | | | | | Fixes #14 Wrap unused param (while logger is off) in ifdef
| | * | 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 #471 from dev-gh/hotfix/Partial_proprietary_policy_supportAGaliuzov2016-05-1014-147/+340
| |\ \ \ | | |_|/ | |/| | Adds partial support of proprietary policy flow
| | * | Fixes build warning for 32-bit UbuntuAndrey Oleynik2016-04-281-1/+1
| | | | | | | | | | | | | | | | Implements: APPLINK-22313
| | * | Changes logic of GetURLs and OnSystemRequest according to new APIAndrey Oleynik2016-04-282-9/+14
| | | | | | | | | | | | | | | | Implements: APPLINK-24247, APPLINK-24248
| | * | Changes HMI API for SystemRequest, OnSystemRequestAndrey Oleynik2016-04-281-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Parameter policyAppId should be replaced with appID and new parameter type must be changed to integer. Implements: APPLINK-24247. APPLINK-24248