summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Greg Kaiser <gkaiser@google.com> 2016-03-28 16:01:04 -0700
committer Greg Kaiser <gkaiser@google.com> 2016-07-13 14:24:44 -0700
commiteb46ce54a843e4ba13e97c4cfea83bcdc693684b (patch)
tree39fcd6a33e399cb399865538f51d21d929c35560
parent1d0a16cdda9c4969fbf5c73d9cde6738291731e6 (diff)
accessorytest: Remove bad code
This code was calling 'free' on uninitialized stack memory. However, we never reached this code due to the while(1) loop, so we weren't experiencing bad behavior. Still, we remove this code to avoid the temptation of copy/paste for someone in the future who might leave this loop. Change-Id: I7f4d9528ddc2f92f3118ec1ade630f98bfbacd0c
-rw-r--r--libs/usb/tests/accessorytest/audio.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/libs/usb/tests/accessorytest/audio.c b/libs/usb/tests/accessorytest/audio.c
index d23d9b3fa5d3..36ee6b81839c 100644
--- a/libs/usb/tests/accessorytest/audio.c
+++ b/libs/usb/tests/accessorytest/audio.c
@@ -164,7 +164,6 @@ static void* capture_thread(void* arg)
static void* play_thread(void* arg)
{
struct pcm *pcm = arg;
- char *buffer;
int index, err;
fprintf(stderr, "play_thread start\n");
@@ -181,7 +180,6 @@ static void* play_thread(void* arg)
fprintf(stderr, "play_thread done\n");
pcm_close(pcm);
- free(buffer);
return NULL;
}