From 8875abd9ab191a8cc49d3620c64d1dbead2ccdac Mon Sep 17 00:00:00 2001
From: David Anderson <dvander@google.com>
Date: Wed, 1 Dec 2021 18:31:56 -0800
Subject: [PATCH] libsnapshot: Fix crash in MakeXorBlockString test.

std::string doesn't like being initialized with negative values, so use
memset() instead.

Bug: 208944665
Test: vts_libsnapshot_test
Change-Id: I42a1a01fd9e59ffc9913c9df9f4bb8ab85113356
---
 fs_mgr/libsnapshot/snapshot_reader_test.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs_mgr/libsnapshot/snapshot_reader_test.cpp b/fs_mgr/libsnapshot/snapshot_reader_test.cpp
index 078f16e4d..9adc6551b 100644
--- a/fs_mgr/libsnapshot/snapshot_reader_test.cpp
+++ b/fs_mgr/libsnapshot/snapshot_reader_test.cpp
@@ -155,8 +155,8 @@ class OfflineSnapshotTest : public ::testing::Test {
     }
 
     std::string MakeXorBlockString() {
-        std::string data(100, -1);
-        data.resize(kBlockSize, 0);
+        std::string data(kBlockSize, 0);
+        memset(data.data(), 0xff, 100);
         return data;
     }