minadbd: use strdup() to create argument for sideload thread.

So sideload thread will not use argument which is to be freed
in the main thread.

Bug: 23968770
Change-Id: I9d6dadc6c33cfbe4b5759382a80fe14cd0d54355

1 files changed, 4 insertions, 2 deletions

diff --git a/minadbd/services.cpp b/minadbd/services.cpp
index dd1fd7c..0eeeb20 100644
--- a/minadbd/services.cpp
+++ b/minadbd/services.cpp
@@ -44,13 +44,14 @@ void* service_bootstrap_func(void* x) {
 }
 
 static void sideload_host_service(int sfd, void* data) {
-    const char* args = reinterpret_cast<const char*>(data);
+    char* args = reinterpret_cast<char*>(data);
     int file_size;
     int block_size;
     if (sscanf(args, "%d:%d", &file_size, &block_size) != 2) {
         printf("bad sideload-host arguments: %s\n", args);
         exit(1);
     }
+    free(args);
 
     printf("sideload-host file size %d block size %d\n", file_size, block_size);
 
@@ -97,7 +98,8 @@ int service_to_fd(const char* name) {
         // sideload-host).
         exit(3);
     } else if (!strncmp(name, "sideload-host:", 14)) {
-        ret = create_service_thread(sideload_host_service, (void*)(name + 14));
+        char* arg = strdup(name + 14);
+        ret = create_service_thread(sideload_host_service, arg);
     }
     if (ret >= 0) {
         close_on_exec(ret);