Skip to content

Commit cc059f4

Browse files
committed
tests/unit-mcdc: wrap BAD_FUNC_ARG operands in WC_NO_ERR_TRACE
check-source-text flags error-code operands compared without WC_NO_ERR_TRACE(). Wrap them in the ed25519 and cmac white-box binaries (same fix already applied to the curve25519 white-box).
1 parent 55bc029 commit cc059f4

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

tests/unit-mcdc/test_cmac_whitebox.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ static void wb_cross_combos(void)
8181
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
8282
NULL, INVALID_DEVID, CMAC_AES_INIT_ID, wb_id, (int)sizeof(wb_id),
8383
wb_label);
84-
if (ret != BAD_FUNC_ARG) {
84+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
8585
WB_NOTE("ID+label!=NULL did not return BAD_FUNC_ARG");
8686
wb_fail = 1;
8787
}
@@ -104,7 +104,7 @@ static void wb_cross_combos(void)
104104
XMEMSET(&cmac, 0, sizeof(cmac));
105105
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
106106
NULL, INVALID_DEVID, CMAC_AES_INIT_LABEL, wb_id, 0, wb_label);
107-
if (ret != BAD_FUNC_ARG) {
107+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
108108
WB_NOTE("LABEL+id!=NULL did not return BAD_FUNC_ARG");
109109
wb_fail = 1;
110110
}
@@ -114,7 +114,7 @@ static void wb_cross_combos(void)
114114
XMEMSET(&cmac, 0, sizeof(cmac));
115115
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
116116
NULL, INVALID_DEVID, CMAC_AES_INIT_LABEL, NULL, 4, wb_label);
117-
if (ret != BAD_FUNC_ARG) {
117+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
118118
WB_NOTE("LABEL+idLen!=0 did not return BAD_FUNC_ARG");
119119
wb_fail = 1;
120120
}
@@ -137,7 +137,7 @@ static void wb_cross_combos(void)
137137
XMEMSET(&cmac, 0, sizeof(cmac));
138138
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
139139
NULL, INVALID_DEVID, CMAC_AES_INIT_PLAIN, wb_id, 0, NULL);
140-
if (ret != BAD_FUNC_ARG) {
140+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
141141
WB_NOTE("PLAIN+id!=NULL did not return BAD_FUNC_ARG");
142142
wb_fail = 1;
143143
}
@@ -146,7 +146,7 @@ static void wb_cross_combos(void)
146146
XMEMSET(&cmac, 0, sizeof(cmac));
147147
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
148148
NULL, INVALID_DEVID, CMAC_AES_INIT_PLAIN, NULL, 7, NULL);
149-
if (ret != BAD_FUNC_ARG) {
149+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
150150
WB_NOTE("PLAIN+idLen!=0 did not return BAD_FUNC_ARG");
151151
wb_fail = 1;
152152
}
@@ -156,7 +156,7 @@ static void wb_cross_combos(void)
156156
XMEMSET(&cmac, 0, sizeof(cmac));
157157
ret = _InitCmac_common(&cmac, wb_key, sizeof(wb_key), WC_CMAC_AES, NULL,
158158
NULL, INVALID_DEVID, CMAC_AES_INIT_PLAIN, NULL, 0, wb_label);
159-
if (ret != BAD_FUNC_ARG) {
159+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
160160
WB_NOTE("PLAIN+label!=NULL did not return BAD_FUNC_ARG");
161161
wb_fail = 1;
162162
}

tests/unit-mcdc/test_ed25519_whitebox.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ static void wb_ed25519_hash(void)
8787

8888
/* key == NULL: operand 1 TRUE, others false. */
8989
ret = ed25519_hash(NULL, in, sizeof(in), hash);
90-
if (ret != BAD_FUNC_ARG) {
90+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
9191
WB_NOTE("ed25519_hash(key==NULL) did not return BAD_FUNC_ARG");
9292
wb_fail = 1;
9393
}
@@ -97,7 +97,7 @@ static void wb_ed25519_hash(void)
9797
* against the all-false baseline's in!=NULL, and against inLen==0
9898
* below for the inLen>0 half). */
9999
ret = ed25519_hash(&key, NULL, sizeof(in), hash);
100-
if (ret != BAD_FUNC_ARG) {
100+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
101101
WB_NOTE("ed25519_hash(in==NULL,inLen>0) did not return "
102102
"BAD_FUNC_ARG");
103103
wb_fail = 1;
@@ -114,7 +114,7 @@ static void wb_ed25519_hash(void)
114114

115115
/* hash == NULL: operand 3 TRUE, others false. */
116116
ret = ed25519_hash(&key, in, sizeof(in), NULL);
117-
if (ret != BAD_FUNC_ARG) {
117+
if (ret != WC_NO_ERR_TRACE(BAD_FUNC_ARG)) {
118118
WB_NOTE("ed25519_hash(hash==NULL) did not return BAD_FUNC_ARG");
119119
wb_fail = 1;
120120
}

0 commit comments

Comments
 (0)