servicemanager: Fix loop continue/break

in getVintfUpdatableApex, it can break out the outer loop if the nested
forEach find a match.

Bug: n/a
Test: TH
Change-Id: I0261a2d43156a563e9e7fd7e96f89544c0f0fd50

Author: Jooyung Han

cmds/servicemanager/ServiceManager.cpp

@@ -150,16 +150,15 @@ static std::optional<std::string> getVintfUpdatableApex(const std::string& name)
     std::optional<std::string> updatableViaApex;
 
     forEachManifest([&](const ManifestWithDescription& mwd) {
-        mwd.manifest->forEachInstance([&](const auto& manifestInstance) {
+        bool cont = mwd.manifest->forEachInstance([&](const auto& manifestInstance) {
             if (manifestInstance.format() != vintf::HalFormat::AIDL) return true;
             if (manifestInstance.package() != aname.package) return true;
             if (manifestInstance.interface() != aname.iface) return true;
             if (manifestInstance.instance() != aname.instance) return true;
             updatableViaApex = manifestInstance.updatableViaApex();
             return false; // break (libvintf uses opposite convention)
         });
-        if (updatableViaApex.has_value()) return true; // break (found match)
-        return false; // continue
+        return !cont;
     });
 
     return updatableViaApex;