3.5.7 batch1 #939
Merged
3.5.7 batch1 #939
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Details There are connections where bound client stays VSOMEIP_CLIENT_UNSET (e.g., a local TCP connection between applications) That means that the underlying server will never be able to map this connection to a client-id, and therefore some on-connection-close handling would never succeed
Summary Unused after which plugins are loaded one-by-one based on configuration in application::init While at it, improve logging for plugin load failure
Update security offer rule for messages Details With the introduction of 0622382, an issue in the policies was detected affecting internal security in the following scenario: 1- Client A connects to the daemon and receives the policies loaded from vsomeip_security.json and <uid>_<gid>/vsomeip_security.json. 2- Client B connects to the daemon and wants to communicate with Client A. 3- The daemon sends an UPDATE_SECURITY_CREDENTIALS_ID message to Client A with Client B credentials. 4- If Client A does not yet have the credentials for Client B, it adds the credentials to any_client_policies_. Until now this worked well because it only verified if the client was allowed and we only added the UID/GID to the allowed clients list. Now since it verifies that the offer is allowed, it doesn't work because the policy load wasn't done.
This commit 39f6770 mistakenly broke the ability to hide export symbols. Here the functionality is fixed. Details The number of exported symbols increased exponentially due to the recent CMakelists update. The export symbols flag were being removed, and that was resulting in this increase. Here the unneeded optimization flag was removed
Summary Change the order how a connection is cleaned-up. Details When a server -> client connection is broken the endpoint needs to be cleaned up, before the corresponding client -> server connection is closed. Otherwise it can happen that the later is restored and the former tried to be used to acknowledge a subscription.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.