diff options
| author | Tianjie Xu <xunchang@google.com> | 2017-09-11 12:01:09 -0700 |
|---|---|---|
| committer | Tianjie Xu <xunchang@google.com> | 2017-09-11 14:08:32 -0700 |
| commit | f9bc1b057165bb5ea89acf135f7f3a90eff579ab (patch) | |
| tree | 4f3974078306f15a7076d8907eeec8aedacc5186 /base/include/android-base/test_utils.h | |
| parent | cfadedb1391f82a25bc0209aaa2e069875657776 (diff) | |
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/test_utils.h')
| -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 c0bf0c1e1b..07a5edda0a 100644 --- a/base/include/android-base/test_utils.h +++ b/base/include/android-base/test_utils.h @@ -26,6 +26,10 @@ class TemporaryFile { TemporaryFile(); ~TemporaryFile(); + // Release the ownership of fd, caller is reponsible for closing the + // fd or stream properly. + int release(); + int fd; char path[1024]; |