commit 51207514eb834ee1a3c1d3218a9615116dc15bab
author Jiyong Park <jiyong@google.com> Tue Dec 03 16:24:48 2019 +0900
committer Jiyong Park <jiyong@google.com> Fri Dec 13 09:27:38 2019 +0900
tree 1af90c549976b7d12f5b776daf1345853f2ade06
parent fa12f52ba2c0f528277bfe7d7543f5f53361850c

unique_fd is passed by value in AIDL interfaces

FileDescriptor type in AIDL was translated into const unique_fd& in C++.
Now, it is unique_fd, i.e. passed by value, to make it easier to keep it
beyond the scope of the call.

Bug: 144943748
Test: m
Change-Id: Ic6b1a4cba71c0fedb206b5ca3fb65b9944bbd69f

services/mediaextractor/MediaExtractorService.h

diff --git a/services/mediaextractor/MediaExtractorService.h b/services/mediaextractor/MediaExtractorService.h
index 1b61c4b..1b40bf9 100644
--- a/services/mediaextractor/MediaExtractorService.h
+++ b/services/mediaextractor/MediaExtractorService.h
@@ -37,7 +37,7 @@
             ::android::sp<::android::IMediaExtractor>* _aidl_return);
 
     virtual ::android::binder::Status makeIDataSource(
-            const base::unique_fd &fd,
+            base::unique_fd fd,
             int64_t offset,
             int64_t length,
             ::android::sp<::android::IDataSource>* _aidl_return);