Delia 68254 main (rdkcentral#6320)

* DELIA-68254 : handling errors for content protection service (rdkcentral#6291)

Reason for change: CPS shall map SecManager errors
to one of the CPS plug-in error codes.
Test Procedure: None
Risks: None

Signed-off-by: Nikita Poltorapavlo <[email protected]>
Co-authored-by: Anand Kandasamy <[email protected]>

* update changelog and api version

---------

Signed-off-by: Nikita Poltorapavlo <[email protected]>
Co-authored-by: Anand Kandasamy <[email protected]>

Author: npoltorapavlo

ContentProtection/CHANGELOG.md

@@ -16,6 +16,10 @@ All notable changes to this RDK Service will be documented in this file.
 
 * For more details, refer to [versioning](https://github.com/rdkcentral/rdkservices#versioning) section under Main README.
 
+## [3.0.0] - 2025-07-08
+### Changed
+- Map SecManager errors to the CPS plug-in error codes
+
 ## [2.1.0] - 2025-06-25
 ### Changed
 - Support for returning video platform data in closeDrmSession result

ContentProtection/ContentProtection.cpp

@@ -19,8 +19,8 @@
 
 #include "ContentProtection.h"
 
-#define API_VERSION_NUMBER_MAJOR 2
-#define API_VERSION_NUMBER_MINOR 1
+#define API_VERSION_NUMBER_MAJOR 3
+#define API_VERSION_NUMBER_MINOR 0
 #define API_VERSION_NUMBER_PATCH 0
 
 namespace WPEFramework {

ContentProtection/ContentProtection.h

@@ -293,6 +293,45 @@ namespace Plugin {
         using State
             = Exchange::IContentProtection::INotification::Status::State;
 
+    private:
+        enum { NoSuchSession = 21009 };
+        enum { WatermarkRenderFailed = 20001 };
+
+        static Core::OptionalType<uint32_t> SecManagerStatus(
+            uint16_t classification, uint16_t reason)
+        {
+            Core::OptionalType<uint32_t> result;
+            // https://github.com/comcast-contentsecurity/spec
+            static std::map<std::tuple<uint16_t, uint16_t>, uint32_t> map{
+                { { 100, 3 }, 21003 },
+                { { 100, 4 }, 21004 },
+                { { 100, 5 }, 21005 },
+                { { 100, 6 }, 21006 },
+                { { 100, 7 }, 21007 },
+                { { 100, 8 }, 21008 },
+                { { 100, 9 }, NoSuchSession },
+                { { 100, 12 }, 21012 },
+                { { 100, 14 }, 21014 },
+                { { 100, 15 }, 21015 },
+                { { 200, 1 }, 22001 },
+                { { 200, 3 }, 22003 },
+                { { 200, 4 }, 22004 },
+                { { 200, 8 }, 22008 },
+                { { 200, 11 }, 22011 },
+                { { 200, 12 }, 22012 },
+                { { 200, 13 }, 22013 },
+                { { 200, 16 }, 22016 },
+                { { 300, 1 }, 23001 },
+                { { 300, 3 }, 23003 },
+                { { 300, 12 }, 23012 }
+            };
+            auto it = map.find(std::make_tuple(classification, reason));
+            if (it != map.end()) {
+                result = it->second;
+            }
+            return result;
+        }
+
     private:
         class Implementation : public Exchange::IContentProtection {
         public:
@@ -361,7 +400,13 @@ namespace Plugin {
                     OpenSessionTimeout, _T("openPlaybackSession"), out, in);
                 if (result == Core::ERROR_NONE) {
                     if (!in["success"].Boolean()) {
-                        result = Core::ERROR_GENERAL;
+                        auto context = in["secManagerResultContext"].Object();
+                        auto status = SecManagerStatus(
+                            context["class"].Number(),
+                            context["reason"].Number());
+                        result = status.IsSet()
+                            ? status.Value()
+                            : Core::ERROR_GENERAL;
                     } else {
                         sessionId = in["sessionId"].Number();
                         string inStr;
@@ -380,7 +425,7 @@ namespace Plugin {
             {
                 auto session = _parent._sessionStorage.Get(sessionId);
                 if (!session.IsSet()) {
-                    return Core::ERROR_ILLEGAL_STATE; // No such session
+                    return NoSuchSession;
                 }
 
                 uint32_t result;
@@ -406,7 +451,7 @@ namespace Plugin {
             {
                 auto session = _parent._sessionStorage.Get(sessionId);
                 if (!session.IsSet()) {
-                    return Core::ERROR_ILLEGAL_STATE; // No such session
+                    return NoSuchSession;
                 }
 
                 uint32_t result;
@@ -426,7 +471,13 @@ namespace Plugin {
                     OpenSessionTimeout, _T("updatePlaybackSession"), out, in);
                 if (result == Core::ERROR_NONE) {
                     if (!in["success"].Boolean()) {
-                        result = Core::ERROR_GENERAL;
+                        auto context = in["secManagerResultContext"].Object();
+                        auto status = SecManagerStatus(
+                            context["class"].Number(),
+                            context["reason"].Number());
+                        result = status.IsSet()
+                            ? status.Value()
+                            : Core::ERROR_GENERAL;
                     } else {
                         string inStr;
                         in.ToString(inStr);
@@ -441,7 +492,7 @@ namespace Plugin {
             {
                 auto session = _parent._sessionStorage.Get(sessionId);
                 if (!session.IsSet()) {
-                    return Core::ERROR_ILLEGAL_STATE; // No such session
+                    return NoSuchSession;
                 }
 
                 uint32_t result;
@@ -453,7 +504,13 @@ namespace Plugin {
                     ClosePlaybackSessionParams, JsonObject>(
                     Timeout, _T("closePlaybackSession"), out, in);
                 if ((result == Core::ERROR_NONE) && !in["success"].Boolean()) {
-                    result = Core::ERROR_GENERAL;
+                    auto context = in["secManagerResultContext"].Object();
+                    auto status = SecManagerStatus(
+                        context["class"].Number(),
+                        context["reason"].Number());
+                    result = status.IsSet()
+                        ? status.Value()
+                        : Core::ERROR_GENERAL;
                 }
                 return result;
             }
@@ -742,7 +799,7 @@ namespace Plugin {
                                WatermarkStatusChanged(
                                    watermark.Value().SessionId,
                                    session.Value().AppId,
-                                   { State::FAILED, 20001 });
+                                   { State::FAILED, WatermarkRenderFailed });
                            }
                        })
                 == Core::ERROR_NONE);