tree d9632b1264748a0a4005a5ebc84535740a7675c9
parent 8dd44e907ada675a5767bf2f4108e8c591e2781f
author Tao Bao <tbao@google.com> 1479843433 -0800
committer Tao Bao <tbao@google.com> 1479861881 -0800

Add ota_close(unique_fd&) and ota_fclose(std::unique_ptr<FILE>&).

We were using the below sequence prior to the CL in [1].

unique_fd fd(ota_open(...));
ota_close(fd);
fd.reset(ota_open(...));

fd.reset() may unintentionally close the newly opened FD if it
has the same value as the early ota_open. The CL in [1] changed to
"ota_close(fd.release())" to avoid the issue. This CL adds a new
overloaded function ota_close(unique_fd&) to handle the release
automatically.

Similarly add ota_fclose(std::unique_ptr<FILE>&).

[1] commit 48cf770471ef53fbf0a1837196220862a0bdb18d.

Bug: 33034669
Test: recovery_component_test passes.
Change-Id: Ief91edc590e95a7426e33364b28754173efb1056