[CP] Change Class PortManager to be inheritable and declare SetPortProperty as virtual function.

By default, port property is set through drm ioctl call. On Android,
port property is set by Hardware Composer. On ClearLinux, port property
is set by IAS. Hence, use polymorphism to override SetPortProperty to
enhance code readability.

[Internal]
Platforms: All
OS: Linux
Feature impact: HDCP
Related-to: VSMGWL-21388
Resolves:
Klockwork: PASS

Change-Id: I21e592e11fd80d72733b6c47921cc5ffb632760f
Signed-off-by: Yu Shiqiang <shiqiang.yu@intel.com>

Author: shiqiangyu

daemon/portmanager.cpp

@@ -303,7 +303,11 @@ int32_t PortManagerInit(HdcpDaemon& socket)
         return EEXIST;
     }
 
+#ifdef ANDROID
+    portMgr = new (std::nothrow) PortManagerHWComposer(socket);
+#else
     portMgr = new (std::nothrow) PortManager(socket);
+#endif
     if (nullptr == portMgr)
     {
         HDCP_ASSERTMESSAGE("Failed to allocate the port manager!");
@@ -857,16 +861,16 @@ int32_t PortManager::DisablePort(const uint32_t portId, const uint32_t appId)
     if (SUCCESS != ret)
     {
         HDCP_ASSERTMESSAGE(
-                    "Failed to enable port with id %d, set property faild",
+                    "Failed to disable port with id %d, set property faild",
                     portId);
         return EBUSY;
     }
 
-    // Check whether Content Protection property is CP_OFF or not 
+    // Check whether Content Protection property is CP_OFF or not
     drmObject->CpTypeAtomicBegin();
     uint8_t cpType = CP_TYPE_INVALID;
     ret = GetProtectionInfo(drmObject, &cpValue, &cpType);
-    if (SUCCESS != ret) 
+    if (SUCCESS != ret)
     {
         drmObject->CpTypeAtomicEnd();
         HDCP_ASSERTMESSAGE("Failed to get protection info");
@@ -1144,7 +1148,7 @@ int32_t PortManager::SetPortProperty(
                             int32_t propId,
                             int32_t size,
                             const uint8_t *value,
-                            const uint32_t numRetry)
+                            uint32_t numRetry)
 {
     HDCP_FUNCTION_ENTER;
 
@@ -1348,3 +1352,96 @@ DrmObject *PortManager::GetDrmObjectByDrmId(const uint32_t drmId)
     return nullptr;
 }
 
+#ifdef ANDROID
+PortManagerHWComposer::PortManagerHWComposer(HdcpDaemon& daemonSocket) :
+                                        PortManager(daemonSocket)
+{
+    HDCP_FUNCTION_ENTER;
+
+    HDCP_FUNCTION_EXIT(SUCCESS);
+}
+
+int32_t PortManagerHWComposer::SetPortProperty(
+                            int32_t drmId,
+                            int32_t propId,
+                            int32_t size,
+                            const uint8_t *value,
+                            uint32_t numRetry)
+{
+    HDCP_FUNCTION_ENTER;
+
+    int32_t ret = EINVAL;
+
+    DrmObject *drmObject = GetDrmObjectByDrmId(drmId);
+    if (nullptr == drmObject)
+    {
+        HDCP_ASSERTMESSAGE("Port drm object found to be nullptr");
+        return ENOENT;
+    }
+
+    if (nullptr == value)
+    {
+        HDCP_ASSERTMESSAGE("Prop value found to be nullptr");
+        return ENOENT;
+    }
+
+    uint8_t propValue = *value;
+
+    android::ProcessState::initWithDriver(BINDER_IPC);
+
+    // Connect to HWC service
+    HWCSHANDLE hwcs = HwcService_Connect();
+    if (nullptr == hwcs)
+    {
+        HDCP_ASSERTMESSAGE("Could not connect to hwcservice");
+        return EINVAL;
+    }
+
+    while (numRetry--)
+    {
+        if (propId == drmObject->GetPropertyId(CONTENT_PROTECTION) &&
+            propValue == CP_OFF)
+        {
+            HDCP_NORMALMESSAGE("Set content protection property (off) via HWC");
+            ret = HwcService_Video_DisableHDCPSession_ForDisplay(hwcs, drmId);
+        }
+        else if (propId == drmObject->GetPropertyId(CONTENT_PROTECTION))
+        {
+            HDCP_NORMALMESSAGE("Set content protection property (on) via HWC");
+            ret = HwcService_Video_EnableHDCPSession_ForDisplay(hwcs, drmId,
+                                                (EHwcsContentType)propValue);
+        }
+        else if (propId == drmObject->GetPropertyId(CP_CONTENT_TYPE))
+        {
+            // This is only for HDCP2.2
+            HDCP_NORMALMESSAGE("Set content type property via HWC");
+            ret = HwcService_Video_EnableHDCPSession_ForDisplay(hwcs, drmId,
+                                                (EHwcsContentType)propValue);
+        }
+        else if(propId == drmObject->GetPropertyId(CP_SRM))
+        {
+            HDCP_NORMALMESSAGE("Set SRM for Display");
+            ret = HwcService_Video_SetHDCPSRM_ForDisplay(hwcs, drmId,
+                                                (const int8_t *)&value, size);
+        }
+        else
+        {
+            HDCP_ASSERTMESSAGE("Property id not supported");
+            ret = EINVAL;
+        }
+
+        if (SUCCESS == ret)
+            break;
+    }
+
+    HwcService_Disconnect(hwcs);
+
+    if (SUCCESS != ret)
+    {
+        HDCP_ASSERTMESSAGE("Failed to set port property via HWC");
+    }
+
+    HDCP_FUNCTION_EXIT(ret);
+    return ret;
+}
+#endif

daemon/portmanager.h

@@ -35,6 +35,16 @@
 #include "hdcpapi.h"
 #include "port.h"
 
+#ifdef ANDROID
+#include <hwcserviceapi.h>
+#include <log/log.h>
+#include <binder/IServiceManager.h>
+#include <binder/ProcessState.h>
+#include <iservice.h>
+
+#define BINDER_IPC          "/dev/vndbinder"
+#endif
+
 #define THREAD_STARTUP_BACKOFF_DELAY_US     100
 #define AUTH_CHECK_DELAY_MS                 1000
 #define INTEGRITY_CHECK_DELAY_MS            500
@@ -109,14 +119,14 @@ class PortManager
     PortManager(HdcpDaemon& daemonSocket);
 
     ///////////////////////////////////////////////////////////////////////////
-    /// \brief  Destructor for PortManager
+    /// \brief  Virtual destructor for PortManager
     ///
     /// \return     nothing -- It's a destructor
     ///
     /// This will cleanup any ports currently in the port list
     /// Any dynamic memory controlled by the PortManager should be released
     ///////////////////////////////////////////////////////////////////////////
-    ~PortManager(void);
+    virtual ~PortManager(void);
 
     ///////////////////////////////////////////////////////////////////////////
     /// \brief  Checks if the portmanager was successfully created
@@ -215,7 +225,7 @@ class PortManager
     void DisableAllPorts();
 
     // Declare private functions
-private:
+protected:
 
     ///////////////////////////////////////////////////////////////////////////
     /// \brief  Get DrmObject by port Id
@@ -247,7 +257,10 @@ class PortManager
                         uint8_t *cpType);
 
     ///////////////////////////////////////////////////////////////////////////
-    /// \brief  Set Port Property
+    /// \brief  Virtual function set port property. By default, port property is
+    ///         set through drm ioctl call. On ClearLinux, port property is set
+    ///         through IAS. On Andorid, port property is set through Hardware
+    ///         Composer.
     ///
     /// \param[in]  drmId,      Id of the drm object
     /// \param[in]  propertyId, Id of property
@@ -256,7 +269,7 @@ class PortManager
     /// \param[in]  numRetry,   the retry times
     /// \return     int32_t     Function return status
     ///////////////////////////////////////////////////////////////////////////
-    int32_t SetPortProperty(
+    virtual int32_t SetPortProperty(
                         int32_t drmId,
                         int32_t propertyId,
                         int32_t size,
@@ -278,10 +291,47 @@ class PortManager
     /// \return     int32_t     Function return status
     ///////////////////////////////////////////////////////////////////////////
     int32_t InitDrmObjects();
-    
+
     static void SigCatcher(int sig);
 };
 
+#ifdef ANDROID
+class PortManagerHWComposer : public PortManager
+{
+public:
+
+    ///////////////////////////////////////////////////////////////////////////
+    /// \brief  Constructor for the PortManagerHWComposer class,
+    ///         this is for Android platform.
+    ///
+    /// \return     Nothing
+    ///
+    /// Callers must check the newly created manager against IsValid.
+    ///////////////////////////////////////////////////////////////////////////
+    PortManagerHWComposer(HdcpDaemon& daemonSocket);
+
+    ///////////////////////////////////////////////////////////////////////////
+    /// \brief  Virtual function set port property. By default, port property is
+    ///         set through drm ioctl call. On ClearLinux, port property is set
+    ///         through IAS. On Andorid, port property is set through Hardware
+    ///         Composer.
+    ///
+    /// \param[in]  drmId,      Id of the drm object
+    /// \param[in]  propertyId, Id of property
+    /// \param[in]  size,       Length of value, it's an array
+    /// \param[in]  value,      Pointer of the array
+    /// \param[in]  numRetry,   the retry times
+    /// \return     int32_t     Function return status
+    ///////////////////////////////////////////////////////////////////////////
+    virtual int32_t SetPortProperty(
+                        int32_t drmId,
+                        int32_t propertyId,
+                        int32_t size,
+                        const uint8_t *value,
+                        uint32_t numRetry);
+};
+#endif
+
 ///////////////////////////////////////////////////////////////////////////////
 /// \brief  Create the local instance of the PortManager
 ///