Skip to content

Commit 5c90f2b

Browse files
committed
tests: Don't access out of bounds array index: array[sizeof(array)]
1 parent 064bb94 commit 5c90f2b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/test/script_tests.cpp

+4-4
Original file line numberDiff line numberDiff line change
@@ -165,18 +165,18 @@ BOOST_AUTO_TEST_CASE(script_PushData)
165165
static const unsigned char pushdata4[] = { OP_PUSHDATA4, 1, 0, 0, 0, 0x5a };
166166

167167
vector<vector<unsigned char> > directStack;
168-
BOOST_CHECK(EvalScript(directStack, CScript(&direct[0], &direct[sizeof(direct)]), CTransaction(), 0, 0));
168+
BOOST_CHECK(EvalScript(directStack, CScript(direct, direct + sizeof(direct)), CTransaction(), 0, 0));
169169

170170
vector<vector<unsigned char> > pushdata1Stack;
171-
BOOST_CHECK(EvalScript(pushdata1Stack, CScript(&pushdata1[0], &pushdata1[sizeof(pushdata1)]), CTransaction(), 0, 0));
171+
BOOST_CHECK(EvalScript(pushdata1Stack, CScript(pushdata1, pushdata1 + sizeof(pushdata1)), CTransaction(), 0, 0));
172172
BOOST_CHECK(pushdata1Stack == directStack);
173173

174174
vector<vector<unsigned char> > pushdata2Stack;
175-
BOOST_CHECK(EvalScript(pushdata2Stack, CScript(&pushdata2[0], &pushdata2[sizeof(pushdata2)]), CTransaction(), 0, 0));
175+
BOOST_CHECK(EvalScript(pushdata2Stack, CScript(pushdata2, pushdata2 + sizeof(pushdata2)), CTransaction(), 0, 0));
176176
BOOST_CHECK(pushdata2Stack == directStack);
177177

178178
vector<vector<unsigned char> > pushdata4Stack;
179-
BOOST_CHECK(EvalScript(pushdata4Stack, CScript(&pushdata4[0], &pushdata4[sizeof(pushdata4)]), CTransaction(), 0, 0));
179+
BOOST_CHECK(EvalScript(pushdata4Stack, CScript(pushdata4, pushdata4 + sizeof(pushdata4)), CTransaction(), 0, 0));
180180
BOOST_CHECK(pushdata4Stack == directStack);
181181
}
182182

0 commit comments

Comments
 (0)