diff options
author | Tianjie Xu | 2017-09-11 14:01:09 -0500 |
---|---|---|
committer | Tianjie Xu | 2017-09-11 16:08:32 -0500 |
commit | f9bc1b057165bb5ea89acf135f7f3a90eff579ab (patch) | |
tree | 4f3974078306f15a7076d8907eeec8aedacc5186 /base/include/android-base | |
parent | cfadedb1391f82a25bc0209aaa2e069875657776 (diff) | |
download | platform-system-core-f9bc1b057165bb5ea89acf135f7f3a90eff579ab.tar.gz platform-system-core-f9bc1b057165bb5ea89acf135f7f3a90eff579ab.tar.xz platform-system-core-f9bc1b057165bb5ea89acf135f7f3a90eff579ab.zip |
Add the Release function for TemporaryFiles
Some tests may create a File* by calling fdopen() on the temp file's
fd. We should release the ownership of fd in this case to avoid the
double close.
Bug: 65430057
Test: libbase unit tests pass
Change-Id: I54fcce2029f9a574f53afdbdda737ee58620c73a
Diffstat (limited to 'base/include/android-base')
-rw-r--r-- | base/include/android-base/test_utils.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/base/include/android-base/test_utils.h b/base/include/android-base/test_utils.h index c0bf0c1e1..07a5edda0 100644 --- a/base/include/android-base/test_utils.h +++ b/base/include/android-base/test_utils.h | |||
@@ -26,6 +26,10 @@ class TemporaryFile { | |||
26 | TemporaryFile(); | 26 | TemporaryFile(); |
27 | ~TemporaryFile(); | 27 | ~TemporaryFile(); |
28 | 28 | ||
29 | // Release the ownership of fd, caller is reponsible for closing the | ||
30 | // fd or stream properly. | ||
31 | int release(); | ||
32 | |||
29 | int fd; | 33 | int fd; |
30 | char path[1024]; | 34 | char path[1024]; |
31 | 35 | ||