diff options
author | AdityaK <appujee@google.com> | 2023-10-31 22:02:30 -0700 |
---|---|---|
committer | Aditya Kumar <appujee@google.com> | 2023-11-01 18:32:25 +0000 |
commit | 08534e7ff995196f3de9f225da29e6691511fb7d (patch) | |
tree | f7074c6181d86521ff3a524016747647a624f407 | |
parent | 9c9e93eccae01a93d1e8793d445366530e31dfa4 (diff) | |
download | libbrillo-08534e7ff995196f3de9f225da29e6691511fb7d.tar.gz |
Fix snprintf format error
Bug: b/308806535
Change-Id: Ice586c57934983b737c3bc812286ff62e142c21a
-rw-r--r-- | brillo/secure_blob_test.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/brillo/secure_blob_test.cc b/brillo/secure_blob_test.cc index 7242e86..871c79c 100644 --- a/brillo/secure_blob_test.cc +++ b/brillo/secure_blob_test.cc @@ -255,9 +255,10 @@ class TestSecureAllocator : public SecureAllocator<T> { TEST(SecureAllocator, ErasureOnDeallocation) { // Make sure that the contents are cleared on deallocation. TestSecureAllocator<char> e; + constexpr size_t test_string_sz = 15; - char *test_string_addr = e.allocate(15); - snprintf(test_string_addr, sizeof(test_string_addr), "Test String"); + char *test_string_addr = e.allocate(test_string_sz); + snprintf(test_string_addr, test_string_sz, "Test String"); // Deallocate memory; the mock class should check for cleared data. e.deallocate(test_string_addr, 15); |