Skip to content

Commit c82226b

Browse files
authored
Use std::vector for allocations here. (#37714)
Unsure why clang-tidy complains here, however the allocation is effectively a dynamic array, so used vector here instead.
1 parent a373044 commit c82226b

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/setup_payload/tests/TestAdditionalDataPayload.cpp

+5-3
Original file line numberDiff line numberDiff line change
@@ -93,10 +93,12 @@ CHIP_ERROR ParseAdditionalDataPayload(const char * additionalDataPayload, size_t
9393
return CHIP_ERROR_INVALID_STRING_LENGTH;
9494
}
9595
size_t additionalDataPayloadBytesLength = additionalDataPayloadLength / 2;
96-
std::unique_ptr<uint8_t[]> additionalDataPayloadBytes(new uint8_t[additionalDataPayloadBytesLength]);
96+
97+
std::vector<uint8_t> additionalDataPayloadBytes;
98+
additionalDataPayloadBytes.resize(additionalDataPayloadBytesLength);
9799
size_t bufferSize = chip::Encoding::HexToBytes(additionalDataPayload, additionalDataPayloadLength,
98-
additionalDataPayloadBytes.get(), additionalDataPayloadBytesLength);
99-
return AdditionalDataPayloadParser(additionalDataPayloadBytes.get(), bufferSize).populatePayload(outPayload);
100+
additionalDataPayloadBytes.data(), additionalDataPayloadBytesLength);
101+
return AdditionalDataPayloadParser(additionalDataPayloadBytes.data(), bufferSize).populatePayload(outPayload);
100102
}
101103

102104
class TestAdditionalDataPayload : public ::testing::Test

0 commit comments

Comments
 (0)