Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixed another Coverity issue involving mixed enums (CID 80028)hotfix/coverity_fix_mixed_enums | Jacob Keeler | 2016-07-29 | 2 | -7/+6 |
| | |||||
* | Fixed unnecessary mixing of enums (CID 80023, 80024, 80026, 80030) | Jacob Keeler | 2016-07-29 | 3 | -19/+14 |
| | |||||
* | Merge pull request #727 from BTPankow/hotfix/coverity_resource_leaks | Justin Dickow | 2016-07-27 | 1 | -9/+6 |
|\ | | | | | Change socket close logic to prevent double close (CID 137866) | ||||
| * | Change socket close logic to prevent double close (CID 137866) | Brad Pankow | 2016-07-27 | 1 | -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_leaks | JackLivio | 2016-07-26 | 2 | -0/+4 |
|\ \ | |/ | | | Fix Coverity resource leaks (CIDs 80039, 80040) | ||||
| * | Close socket connection before returning from function (CID 80039) | Brad Pankow | 2016-07-21 | 1 | -0/+1 |
| | | |||||
| * | Free opened file before returning from function (CID 80040) | Brad Pankow | 2016-07-21 | 1 | -0/+3 |
| | | |||||
* | | Merge pull request #715 from BTPankow/hotfix/coverity_buffer_termination | JackLivio | 2016-07-26 | 1 | -1/+3 |
|\ \ | | | | | | | Null terminate char buffer after strncpy (CID 79979) | ||||
| * | | Null terminate char buffer after strncpy (CID 79979) | Brad Pankow | 2016-07-21 | 1 | -1/+3 |
| |/ | |||||
* | | Merge pull request #723 from BTPankow/hotfix/coverity_uninitialized | JackLivio | 2016-07-26 | 19 | -18/+72 |
|\ \ | | | | | | | Fix uninitialized pointer and scalar fields (Coverity) | ||||
| * | | Initialize binary_data_ to NULL (CID 80088) | Brad Pankow | 2016-07-26 | 1 | -1/+2 |
| | | | |||||
| * | | Initialize thread delegate to NULL before use (CID 80087) | Brad Pankow | 2016-07-26 | 1 | -0/+1 |
| | | | |||||
| * | | Initialize policy_table_type_ to INVALID_PT_TYPE for Optional template class ↵ | Brad Pankow | 2016-07-26 | 2 | -3/+6 |
| | | | | | | | | | | | | (CID 80046, 80047, 80048) | ||||
| * | | Initialize protocol_version to V2 (CID 80051) | Brad Pankow | 2016-07-26 | 1 | -1/+3 |
| | | | |||||
| * | | Initialize Profile members to defaults (CID 80053) | Brad Pankow | 2016-07-26 | 1 | -0/+6 |
| | | | |||||
| * | | Initialize state_ctrl_ to constructor parameter (CID 80054) | Brad Pankow | 2016-07-26 | 1 | -1/+1 |
| | | | |||||
| * | | Initialize RequestInfo members, add new RequestType (CID 80058, 80066, 80067) | Brad Pankow | 2016-07-26 | 2 | -5/+17 |
| | | | |||||
| * | | Initialize is_ready_ before use in functions (CID 80072) | Brad Pankow | 2016-07-26 | 1 | -0/+1 |
| | | | |||||
| * | | Initialize is_persistent, is_download_complete, and file_type before use in ↵ | Brad Pankow | 2016-07-26 | 1 | -1/+4 |
| | | | | | | | | | | | | functions (CID 80073) | ||||
| * | | Initialize max_block_size_ before use in functions (CID 80074) | Brad Pankow | 2016-07-26 | 1 | -1/+2 |
| | | | |||||
| * | | Initialize app_state_ and connection_id_ before use in functions (CID 80075) | Brad Pankow | 2016-07-26 | 1 | -1/+2 |
| | | | |||||
| * | | Initialize choice_set_id_ and is_timed_out_ before use in functions (CID 80076) | Brad Pankow | 2016-07-26 | 1 | -1/+3 |
| | | | |||||
| * | | Initialize cmd_id_ and successful_response_received_ before use in functions ↵ | Brad Pankow | 2016-07-26 | 1 | -1/+3 |
| | | | | | | | | | | | | (CID 80077) | ||||
| * | | Initialize port with default value of 0 (CID 80079). | Brad Pankow | 2016-07-26 | 1 | -1/+2 |
| | | | |||||
| * | | Add default enum values to initialize StatisticsDelegate (CID ↵ | Brad Pankow | 2016-07-26 | 2 | -0/+16 |
| | | | | | | | | | | | | 80080,80081,80082,80083) | ||||
| * | | Initialize offset_ before use in functions (CID 80084) | Brad Pankow | 2016-07-25 | 1 | -0/+1 |
| | | | |||||
| * | | Initialize current_application_ before use in functions (CID 80089) | Brad Pankow | 2016-07-25 | 1 | -1/+2 |
| |/ | |||||
* | | Merge pull request #724 from BTPankow/hotfix/coverity_int_handling | JackLivio | 2016-07-26 | 1 | -1/+1 |
|\ \ | | | | | | | Only send frames when POLLOUT set (CID 79995) | ||||
| * | | Only send frames when POLLOUT set (CID 79995) | Brad Pankow | 2016-07-26 | 1 | -1/+1 |
| | | | |||||
* | | | Fix issues with freed string before use (CIDs 80093, 80094) | Brad Pankow | 2016-07-21 | 6 | -11/+6 |
| |/ |/| | |||||
* | | Merge pull request #710 from smartdevicelink/master | JackLivio | 2016-07-21 | 33 | -6084/+6284 |
|\ \ | |/ | | | Merge Master Into Coverity | ||||
| * | ParsePTString to Modify string, instead of returning a copyhotfix/add_escape_characters_during_ptu_header_construction | JackLivio | 2016-06-20 | 2 | -7/+7 |
| | | |||||
| * | Parse PT String for HTTP Request Body | JackLivio | 2016-06-08 | 2 | -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 Oleynik | 2016-05-17 | 5 | -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 Oleynik | 2016-05-14 | 1 | -0/+1 |
| | | |||||
| * | Fixes policy update triggering on application registration | Andrey Oleynik | 2016-05-14 | 1 | -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 policy | Andrey Oleynik | 2016-05-13 | 4 | -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 implementation | Anton Hrytsevich | 2016-05-13 | 8 | -167/+235 |
| | | | | | | | | | | | | Adopts changes done originally regaring issue: [APPLINK-19543] Closes-bug: APPLINK-24295 | ||||
| * | Updates PolicyManager in order to properly support certificate field | Aleksandr Galiuzov | 2016-05-13 | 5 | -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 registration | Andrey Oleynik | 2016-05-13 | 12 | -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.11 | AGaliuzov | 2016-05-10 | 1 | -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_v2 | JackLivio | 2016-05-02 | 1 | -0/+1 |
| | | | |||||
| * | | Merge pull request #514 from smartdevicelink/hotfix/generate_snapshot_fix_#461 | AGaliuzov | 2016-05-10 | 2 | -2/+17 |
| |\ \ | | | | | | | | | Rewrite GenerateSnapshot() | ||||
| | * | | Rewrite GenerateSnapshot()hotfix/generate_snapshot_fix_#461 | JackLivio | 2016-04-26 | 2 | -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/14 | AGaliuzov | 2016-05-10 | 1 | -1/+2 |
| |\ \ | | | | | | | | | Fixes #14 Wrap unused param (while logger is off) in ifdef | ||||
| | * | | Wrap unused param (while logger is off) in ifdefhotfix/14 | Justin Dickow | 2016-04-12 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | Have to do this because compiler treating warnings as errors | ||||
| * | | | Merge pull request #471 from dev-gh/hotfix/Partial_proprietary_policy_support | AGaliuzov | 2016-05-10 | 14 | -147/+340 |
| |\ \ \ | | |_|/ | |/| | | Adds partial support of proprietary policy flow | ||||
| | * | | Fixes build warning for 32-bit Ubuntu | Andrey Oleynik | 2016-04-28 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | Implements: APPLINK-22313 | ||||
| | * | | Changes logic of GetURLs and OnSystemRequest according to new API | Andrey Oleynik | 2016-04-28 | 2 | -9/+14 |
| | | | | | | | | | | | | | | | | Implements: APPLINK-24247, APPLINK-24248 | ||||
| | * | | Changes HMI API for SystemRequest, OnSystemRequest | Andrey Oleynik | 2016-04-28 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Parameter policyAppId should be replaced with appID and new parameter type must be changed to integer. Implements: APPLINK-24247. APPLINK-24248 |