From: Brian Gix <brian.gix@intel.com>
To: linux-bluetooth@vger.kernel.org
Cc: johan.hedberg@gmail.com, inga.stotland@intel.com,
marcel@holtmann.org, brian.gix@intel.com
Subject: [PATCH BlueZ v2 22/30] mesh: restructure App Key storage
Date: Fri, 14 Dec 2018 16:01:20 -0800 [thread overview]
Message-ID: <20181215000128.928-23-brian.gix@intel.com> (raw)
In-Reply-To: <20181215000128.928-1-brian.gix@intel.com>
From: Inga Stotland <inga.stotland@intel.com>
---
mesh/appkey.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/mesh/appkey.c b/mesh/appkey.c
index 3d445d217..c4ebf16b6 100644
--- a/mesh/appkey.c
+++ b/mesh/appkey.c
@@ -31,7 +31,7 @@
#include "mesh/node.h"
#include "mesh/net.h"
#include "mesh/crypto.h"
-#include "mesh/display.h"
+#include "mesh/util.h"
#include "mesh/model.h"
#include "mesh/storage.h"
#include "mesh/appkey.h"
@@ -206,12 +206,12 @@ bool appkey_msg_in_replay_cache(struct mesh_net *net, uint16_t idx,
}
if (seq < msg->seq) {
- l_info("Ignoring packet with lower sequence number");
+ l_debug("Ignoring packet with lower sequence number");
return true;
}
if (seq == msg->seq) {
- l_info("Message already processed (duplicate)");
+ l_debug("Message already processed (duplicate)");
return true;
}
@@ -302,6 +302,7 @@ bool appkey_key_init(struct mesh_net *net, uint16_t net_idx, uint16_t app_idx,
return false;
key->net_idx = net_idx;
+ key->app_idx = app_idx;
if (key_value && !set_key(key, app_idx, key_value, false))
return false;
@@ -392,14 +393,14 @@ int appkey_key_add(struct mesh_net *net, uint16_t net_idx, uint16_t app_idx,
return MESH_STATUS_SUCCESS;
if (!update) {
- l_info("Failed to add key: index already stored %x",
+ l_debug("Failed to add key: index already stored %x",
(net_idx << 16) | app_idx);
return MESH_STATUS_IDX_ALREADY_STORED;
}
}
if (!key) {
- if (l_queue_length(app_keys) <= MAX_APP_KEYS)
+ if (!(l_queue_length(app_keys) < MAX_APP_KEYS))
return MESH_STATUS_INSUFF_RESOURCES;
key = app_key_new();
@@ -411,7 +412,7 @@ int appkey_key_add(struct mesh_net *net, uint16_t net_idx, uint16_t app_idx,
return MESH_STATUS_INSUFF_RESOURCES;
}
- if (!storage_local_app_key_add(net, net_idx, app_idx, new_key,
+ if (!storage_app_key_add(net, net_idx, app_idx, new_key,
false)) {
appkey_key_free(key);
return MESH_STATUS_STORAGE_FAIL;
@@ -424,7 +425,7 @@ int appkey_key_add(struct mesh_net *net, uint16_t net_idx, uint16_t app_idx,
if (!set_key(key, app_idx, new_key, true))
return MESH_STATUS_INSUFF_RESOURCES;
- if (!storage_local_app_key_add(net, net_idx, app_idx, new_key,
+ if (!storage_app_key_add(net, net_idx, app_idx, new_key,
true))
return MESH_STATUS_STORAGE_FAIL;
}
@@ -457,7 +458,7 @@ int appkey_key_delete(struct mesh_net *net, uint16_t net_idx,
l_queue_remove(app_keys, key);
appkey_key_free(key);
- if (!storage_local_app_key_del(net, net_idx, app_idx))
+ if (!storage_app_key_del(net, net_idx, app_idx))
return MESH_STATUS_STORAGE_FAIL;
return MESH_STATUS_SUCCESS;
--
2.14.5
next prev parent reply other threads:[~2018-12-15 0:01 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-15 0:00 [PATCH BlueZ v2 00/30] Major rewrite for Multi-Node and DBus Brian Gix
2018-12-15 0:00 ` [PATCH BlueZ v2 01/30] mesh: Staging for Mesh DBus API rewrite Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 02/30] mesh: Delete obsolete files Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 03/30] mesh: Utilities for DBus support Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 04/30] mesh: Internal errors Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 05/30] mesh: Re-write storage for Multiple Nodes Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 06/30] mesh: Rewrite Node handling for multiple nodes Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 07/30] mesh: Rewite Network layer " Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 08/30] mesh: Direction agnostic PB-Adv implimentation Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 09/30] mesh: Acceptor side provisioning implimentation Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 10/30] mesh: Initiator " Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 11/30] mesh: Rewrite Controler interface for full init Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 12/30] mesh: Unchanged variables set to const Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 13/30] mesh: centralize generic utilities Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 14/30] mesh: re-arrange provisioning for DBus API Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 15/30] mesh: Re-architect " Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 16/30] mesh: Make config model handle multiple nodes Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 17/30] mesh: Multi node Config Server model Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 18/30] mesh: restructure I/O for multiple nodes Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 19/30] mesh: Restrusture DB to support " Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 21/30] mesh: DBUS interface for Provisioning Agent Brian Gix
2018-12-15 0:01 ` Brian Gix [this message]
2018-12-15 0:01 ` [PATCH BlueZ v2 23/30] mesh: Clean-up Comment style Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 24/30] mesh: Update for DBus API and multi-node support Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 25/30] mesh: Add default location for Mesh Node storage Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 26/30] mesh: Add structural changes to for mesh Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 27/30] mesh: Sample Provisioning Agent Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 28/30] mesh: Sample On/Off Client and Server Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 29/30] mesh: Sample Mesh Joiner (provision acceptor) Brian Gix
2018-12-15 0:01 ` [PATCH BlueZ v2 30/30] mesh: Enable building Mesh Daemon Brian Gix
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181215000128.928-23-brian.gix@intel.com \
--to=brian.gix@intel.com \
--cc=inga.stotland@intel.com \
--cc=johan.hedberg@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=marcel@holtmann.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).