Skip to content

Commit

Permalink
Fix Jira 925 KW errors found in PR534
Browse files Browse the repository at this point in the history
1. Dismiss the warnings.
2. Rebuild the library

Changed files:
  BLEProfileManager.cpp - Init member variable
  rpc_deserialize.c - Delete unreachable code
  dtm.c - Init the variables
  gap.c - Init the variables
  gatt.c - Init the variables
  • Loading branch information
sgbihu committed May 5, 2017
1 parent 171227f commit 66cc3a7
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions libraries/CurieBLE/src/internal/BLEProfileManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ BLEProfileManager* BLEProfileManager::instance()

BLEProfileManager::BLEProfileManager ():
_start_discover(false),
_discover_gap_gatt(false),
_discovering(false),
_discover_rsp_timestamp(0),
_cur_discover_service(NULL),
Expand Down
2 changes: 0 additions & 2 deletions system/libarc32_arduino101/drivers/rpc/rpc_deserialize.c
Original file line number Diff line number Diff line change
Expand Up @@ -336,8 +336,6 @@ static void deserialize_control(uint8_t fn_index, const uint8_t *buf,
rpc_init_cb(struct_data.version, true);
}
break;
break;
panic(-1);
}
}

Expand Down
2 changes: 2 additions & 0 deletions system/libarc32_arduino101/framework/src/services/ble/dtm.c
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
*/
#include <stdint.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>

#include <bluetooth/dtm.h>
Expand All @@ -40,6 +41,7 @@ int ble_dtm_cmd(struct dtm_cmd_params *params) {
return -EINVAL;
}

memset(&req, 0, sizeof(req));
switch(params->cmd_type) {
case DTM_START_RX:
req.tx_rx_freq = params->rx.freq;
Expand Down
2 changes: 2 additions & 0 deletions system/libarc32_arduino101/framework/src/services/ble/gap.c
Original file line number Diff line number Diff line change
Expand Up @@ -385,6 +385,8 @@ static int hci_le_create_conn(struct bt_conn *conn)
{
struct nble_gap_connect_req conn_params;

memset(&conn_params, 0, sizeof(conn_params));

conn_params.bda = conn->le.dst;
conn_params.conn_params.interval_min = conn->le.interval_min;
conn_params.conn_params.interval_max = conn->le.interval_max;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -900,6 +900,7 @@ void on_nble_gatts_write_exec_evt(const struct nble_gatts_write_exec_evt *evt)

req.conn_handle = evt->conn_handle;
req.status = 0;
req.offset = 0;

BT_DBG("write_exec_evt");

Expand Down
Binary file modified variants/arduino_101/libarc32drv_arduino101.a
Binary file not shown.

0 comments on commit 66cc3a7

Please sign in to comment.