tree 64c490b3b728e5d70f4ad1085a982db82fab5343
parent 07fe8c81e3f2306f5831a0e4e018666a22d737b4
author Sarthak Kukreti <sarthakkukreti@google.com> 1552587950 -0700
committer chrome-bot <chrome-bot@chromium.org> 1553136687 -0700

libbrillo: blkdevutils: avoid passing non-null terminated data as C-string

SecureBlobs created from std::strings do not end in '\0'.
In this case, the dm code calls strlen() on the parameter string
which results in an access beyond the allocated space. This caused
the ASAN builder to fail continuously. Instead, for now, use the
string functions for SecureBlob to pass the data on.

BUG=chromium:941727
TEST=amd-generic vm boots up.

Change-Id: I6c63b37bae1204c9dca6315f2285f16f4405296a
Reviewed-on: https://chromium-review.googlesource.com/1524550
Commit-Ready: Sarthak Kukreti <sarthakkukreti@chromium.org>
Tested-by: Sarthak Kukreti <sarthakkukreti@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Cr-Mirrored-From: https://chromium.googlesource.com/chromiumos/platform2
Cr-Mirrored-Commit: 70164ed0d01338cff38777b0bbeb71a79ffe6b6c