diff options
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 |
commit | eb46ce54a843e4ba13e97c4cfea83bcdc693684b (patch) | |
tree | 39fcd6a33e399cb399865538f51d21d929c35560 /libs/usb | |
parent | 1d0a16cdda9c4969fbf5c73d9cde6738291731e6 (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
Diffstat (limited to 'libs/usb')
-rw-r--r-- | libs/usb/tests/accessorytest/audio.c | 2 |
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; } |