commit 2613110ed7894360427d23d027643df8ebcab2ab
author Bram Bonné <brambonne@google.com> Tue Nov 24 13:25:23 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue Nov 24 13:25:23 2020 +0000
tree e02aaf0a6a609ca0fb348f48d1193cae2f49b9fe
parent 27a10d501b5a91d70fa5a6e7d9f3e183281230c4
parent 0281142093d7eb19f99689e58818b6c5c4fd351d

Null-terminate libselinux seinfo in fuzzer. am: f68a36b119 am: 0281142093

Original change: https://android-review.googlesource.com/c/platform/external/selinux/+/1509588

Change-Id: I14ee61d57ad5a0c8a7e882d3152286bb6f41bf00

libselinux/fuzzers/AndroidSetcontextFuzzer.cpp

diff --git a/libselinux/fuzzers/AndroidSetcontextFuzzer.cpp b/libselinux/fuzzers/AndroidSetcontextFuzzer.cpp
index 98ee15e..995b0d9 100644
--- a/libselinux/fuzzers/AndroidSetcontextFuzzer.cpp
+++ b/libselinux/fuzzers/AndroidSetcontextFuzzer.cpp
@@ -26,9 +26,9 @@
   uid_t uid = fdp.ConsumeIntegral<int>();
   bool isSystemServer = fdp.ConsumeBool();
   std::string pkgname = fdp.ConsumeRandomLengthString();
-  std::vector<char> seinfo = fdp.ConsumeRemainingBytes<char>();
+  std::string seinfo = fdp.ConsumeRemainingBytesAsString();
 
-  selinux_android_setcontext(uid, isSystemServer, seinfo.data(), pkgname.c_str());
+  selinux_android_setcontext(uid, isSystemServer, seinfo.c_str(), pkgname.c_str());
 
   return 0;
 }