Skip to content

Commit 1811194

Browse files
author
Mika Leppänen
committed
Corrected warning trace, validations and ut stubs
1 parent 69e2d19 commit 1811194

File tree

4 files changed

+38
-6
lines changed

4 files changed

+38
-6
lines changed

source/6LoWPAN/ws/ws_pae_key_storage.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -789,10 +789,10 @@ void ws_pae_key_storage_remove(void)
789789
nvm_tlv_t *tlv = ws_pae_nvm_store_generic_tlv_allocate_and_create(
790790
PAE_NVM_KEY_STORAGE_INDEX_TAG, PAE_NVM_KEY_STORAGE_INDEX_LEN);
791791

792-
ws_pae_nvm_store_tlv_file_read(KEY_STORAGE_INDEX_FILE, tlv);
793-
794792
uint64_t store_bitfield = 0;
795-
ws_pae_nvm_store_key_storage_index_tlv_read(tlv, &store_bitfield);
793+
if (ws_pae_nvm_store_tlv_file_read(KEY_STORAGE_INDEX_FILE, tlv) >= 0) {
794+
ws_pae_nvm_store_key_storage_index_tlv_read(tlv, &store_bitfield);
795+
}
796796
ws_pae_nvm_store_generic_tlv_free(tlv);
797797

798798
ws_pae_nvm_store_tlv_file_remove(KEY_STORAGE_INDEX_FILE);

source/6LoWPAN/ws/ws_pae_nvm_store.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ static int8_t ws_pae_nvm_store_create_path(char *data_path, const char *file_nam
129129

130130
static int8_t ws_pae_nvm_store_write(const char *file_name, nvm_tlv_t *tlv)
131131
{
132-
133132
if (!file_name || !tlv) {
134133
return -1;
135134
}
@@ -165,7 +164,7 @@ static int8_t ws_pae_nvm_store_read(const char *file_name, nvm_tlv_t *tlv)
165164

166165
size_t n_bytes = ns_fread(fp, tlv, tlv->len + sizeof(nvm_tlv_t));
167166
if (n_bytes != tlv->len + sizeof(nvm_tlv_t)) {
168-
tr_error("NVM write error %s", file_name);
167+
tr_warning("File not found or cannot be read: %s", file_name);
169168
ns_fclose(fp);
170169
return PAE_NVM_FILE_READ_ERROR;
171170
}

source/6LoWPAN/ws/ws_pae_supp.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -426,7 +426,9 @@ static int8_t ws_pae_supp_nvm_keys_read(pae_supp_t *pae_supp)
426426
return -1;
427427
}
428428
ws_pae_nvm_store_generic_tlv_create(tlv, PAE_NVM_KEYS_TAG, PAE_NVM_KEYS_LEN);
429-
ws_pae_nvm_store_tlv_file_read(KEYS_FILE_NAME, tlv);
429+
if (ws_pae_nvm_store_tlv_file_read(KEYS_FILE_NAME, tlv) < 0) {
430+
return -1;
431+
}
430432
ws_pae_nvm_store_keys_tlv_read(tlv, &pae_supp->entry.sec_keys);
431433
return 0;
432434
}

test/nanostack/unittest/stub/ns_file_system_stub.c

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
#include "ns_types.h"
2020
#include "nsdynmemLIB.h"
2121
#include "ns_file_system.h"
22+
#include "ns_file.h"
2223

2324
static char *file_system_root;
2425

@@ -32,3 +33,33 @@ char *ns_file_system_get_root_path(void)
3233
return file_system_root;
3334
}
3435

36+
NS_FILE ns_fopen(const char *filename, const char *mode)
37+
{
38+
return 0;
39+
}
40+
41+
int ns_fclose(NS_FILE *ns_handle)
42+
{
43+
return -1;
44+
}
45+
46+
int ns_fremove(const char *filename)
47+
{
48+
return -1;
49+
}
50+
51+
size_t ns_fwrite(NS_FILE *ns_handle, const void *buffer, size_t size)
52+
{
53+
return 0;
54+
}
55+
56+
size_t ns_fread(NS_FILE *ns_handle, void *buffer, size_t size)
57+
{
58+
return 0;
59+
}
60+
61+
int ns_fsize(NS_FILE *ns_handle, size_t *size)
62+
{
63+
return 0;
64+
}
65+

0 commit comments

Comments
 (0)