commit 9e75663c56f13d21799bbe6a67f5587659a320e3
author Glenn Kasten <gkasten@google.com> Tue Oct 03 13:01:09 2017 -0700
committer Glenn Kasten <gkasten@google.com> Thu Apr 19 11:09:57 2018 -0700
tree 6b89e9981004057393c6bea438f4357e55e67ece
parent c4b259a584be3e7dc9aad83e0ead06c913ceb998

Don't create empty .wav files

Bug: 78287983
Test: follow the tee sink documentation, and then verify there are no empty files
Change-Id: I0aa98aff5d0d47b86f36dd38a54d0daf3feb1965

services/audioflinger/AudioFlinger.cpp

diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp
index 54121cd..66520d6 100644
--- a/services/audioflinger/AudioFlinger.cpp
+++ b/services/audioflinger/AudioFlinger.cpp
@@ -3446,8 +3446,13 @@
             // FIXME not big-endian safe
             write(teeFd, &temp, sizeof(temp));
             close(teeFd);
-            if (fd >= 0) {
-                dprintf(fd, "tee copied to %s\n", teePath);
+            // TODO Should create file with temporary name and then rename to final if non-empty.
+            if (total > 0) {
+                if (fd >= 0) {
+                    dprintf(fd, "tee copied to %s\n", teePath);
+                }
+            } else {
+                unlink(teePath);
             }
         } else {
             if (fd >= 0) {