From: Daniel Wagner <[email protected]>

Hi,

Let's have the next batch of patches from the 'per application routing
and staticics' series.

I have reordered a few patches so that we first have a bug fixes in
session.c and session_policy_local.c file only.

  session: Empty policy list indicated no match all

We need to express a 'no match' as well which is done by having no
bearer at all in the list. Obviously, when no bearer is in the policy
configuration than we should just return false in all cases.

  session: Reorder shutdown sequence

I saw ConnMan crash when I did a ctrl-c with an active session.
With this patch the crash is gone.

  session: Do not access initialized hash table

That one goes into the same catagory as the one above. Do not crash
when shuting down while a session is running.

  session_policy_local: Do not free policy on load error
  session_policy_local: Load policy when a new file is added

Both patches fixes corner cases how we handle/behave when the policy files
are modified/added/removed. These patches should make ConnMan more robust
when the files are invalid.

If you want to see where we are heading have a look at the
whole series here:

https://github.com/bmwcarit/connman/tree/secmark-v3

cheers,
daniel

Daniel Wagner (5):
  session: Empty policy list indicated no match all
  session: Reorder shutdown sequence
  session: Do not access initialized hash table
  session_policy_local: Do not free policy on load error
  session_policy_local: Load policy when a new file is added

 plugins/session_policy_local.c | 31 +++++++++++++++++--------------
 src/main.c                     |  2 +-
 src/session.c                  |  5 ++++-
 3 files changed, 22 insertions(+), 16 deletions(-)

-- 
1.8.2.rc3.16.gce432ca

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to