On 06/20/2014 06:50 PM, Djalal Harouni wrote:
> Use the db->entries_hash to access the policy db entries instead of the
> db->send_access_hash which is just a cache for send entries.
>
> Signed-off-by: Djalal Harouni <tix...@opendz.org>
> ---
> policy.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/policy.c b/policy.c
> index bf49f68..79d6fa4 100644
> --- a/policy.c
> +++ b/policy.c
> @@ -373,7 +373,7 @@ static void __kdbus_policy_remove_owner(struct
> kdbus_policy_db *db,
> struct hlist_node *tmp;
> int i;
>
> - hash_for_each_safe(db->send_access_hash, i, tmp, e, hentry)
> + hash_for_each_safe(db->entries_hash, i, tmp, e, hentry)
Hmm, we need to do both, right? If an owner goes away, we need to kill
both the entries it created *and* the cached entries it is related to.
Now that I look at the code, I see that we miss an owner assignment for
the cache entries. I can fix that up later.
> @@ -482,7 +483,7 @@ int kdbus_policy_set(struct kdbus_policy_db *db,
> * At the same time, the lookup mechanism won't find any collisions
> * when looking for already exising names.
> */
> - hash_for_each_safe(db->send_access_hash, i, tmp, e, hentry)
> + hash_for_each_safe(db->entries_hash, i, tmp, e, hentry)
> if (e->owner == owner) {
> struct kdbus_policy_list_entry *l;
This change looks right though.
Daniel
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
