Skip to content

Commit 71e2688

Browse files
npoltorapavloanand-kyapatel859
authored
Delia 69034 delia 68911 25q4 (#6406)
* Delia 68911 main (#6396) * DELIA-68911 : Send no "clutSize" in modifyPalettedWatermark (#6389) Reason for change: Skip palette size parameter. Because of watermark plugin, which by default having logic that works for palette size of 256*4. When same watermark plugin is interacted from aamp instead of content protection, palette size parameter is not passed and defaulted to 256*4 inside watermark plugin. Test Procedure: None Risks: None Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> Co-authored-by: Anand Kandasamy <37086488+anand-ky@users.noreply.github.com> * update changelog and api version --------- Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> Co-authored-by: Anand Kandasamy <37086488+anand-ky@users.noreply.github.com> Co-authored-by: apatel859 <48992974+apatel859@users.noreply.github.com> * Delia 69034 main (#6404) * DELIA-69034 : handling errors for content protection service (#6387) Reason for change: Add mapping for Invalid aspect dimension, and Watermark perceptibility errors. Fix mapping for DRM entitlement error. Test Procedure: None Risks: None Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> Co-authored-by: Anand Kandasamy <37086488+anand-ky@users.noreply.github.com> * RDKEMW-8549 : handling errors for content protection service (#6392) Reason for change: Typo fix. Test Procedure: None Risks: None Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> * update changelog and api version --------- Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> Co-authored-by: Anand Kandasamy <37086488+anand-ky@users.noreply.github.com> --------- Signed-off-by: Nikita Poltorapavlo <npoltorapavlo@productengine.com> Co-authored-by: Anand Kandasamy <37086488+anand-ky@users.noreply.github.com> Co-authored-by: apatel859 <48992974+apatel859@users.noreply.github.com>
1 parent c5c5184 commit 71e2688

3 files changed

Lines changed: 29 additions & 5 deletions

File tree

ContentProtection/CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,14 @@ All notable changes to this RDK Service will be documented in this file.
1616

1717
* For more details, refer to [versioning](https://github.com/rdkcentral/rdkservices#versioning) section under Main README.
1818

19+
## [3.1.0] - 2025-10-03
20+
### Added
21+
- Update the interface for handling Invalid aspect dimension, Watermark perceptibility, and DRM entitlement errors
22+
23+
## [3.0.3] - 2025-09-30
24+
### Fixed
25+
- Skip palette size parameter in watermark plugin call, similarly to aamp
26+
1927
## [3.0.2] - 2025-08-08
2028
### Fixed
2129
- JSON-RPC calls changed from sync to a-sync

ContentProtection/ContentProtection.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020
#include "ContentProtection.h"
2121

2222
#define API_VERSION_NUMBER_MAJOR 3
23-
#define API_VERSION_NUMBER_MINOR 0
24-
#define API_VERSION_NUMBER_PATCH 2
23+
#define API_VERSION_NUMBER_MINOR 1
24+
#define API_VERSION_NUMBER_PATCH 0
2525

2626
namespace WPEFramework {
2727
namespace Plugin {

ContentProtection/ContentProtection.h

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -311,8 +311,9 @@ namespace Plugin {
311311
uint16_t classification, uint16_t reason)
312312
{
313313
Core::OptionalType<uint32_t> result;
314-
// https://github.com/comcast-contentsecurity/spec
314+
// https://github.com/comcast-contentsecurity/spec/blob/master/device-security/SecManagerResultStatus.md
315315
static std::map<std::tuple<uint16_t, uint16_t>, uint32_t> map{
316+
{ { 100, 2 }, 21002 },
316317
{ { 100, 3 }, 21003 },
317318
{ { 100, 4 }, 21004 },
318319
{ { 100, 5 }, 21005 },
@@ -330,10 +331,15 @@ namespace Plugin {
330331
{ { 200, 11 }, 22011 },
331332
{ { 200, 12 }, 22012 },
332333
{ { 200, 13 }, 22013 },
333-
{ { 200, 16 }, 22016 },
334+
{ { 200, 102 }, 22016 },
334335
{ { 300, 1 }, 23001 },
335336
{ { 300, 3 }, 23003 },
336-
{ { 300, 12 }, 23012 }
337+
{ { 300, 12 }, 23012 },
338+
{ { 300, 1002 }, 23014 },
339+
{ { 300, 1003 }, 23014 },
340+
{ { 300, 1004 }, 23014 },
341+
{ { 300, 1007 }, 23014 },
342+
{ { 300, 1008 }, 23014 }
337343
};
338344
auto it = map.find(std::make_tuple(classification, reason));
339345
if (it != map.end()) {
@@ -732,6 +738,16 @@ namespace Plugin {
732738
out.ClutKey = params.WatermarkClutBufferKey;
733739
out.ClutSize = palette.Value().clutSize;
734740
out.ClutType = palette.Value().clutType;
741+
// Skip palette size parameter.
742+
// Because of watermark plugin, which
743+
// by default having logic that works for
744+
// palette size of 256*4. When same watermark
745+
// plugin is interacted from aamp instead of
746+
// content protection, palette size parameter
747+
// is not passed and defaulted to 256*4 inside
748+
// watermark plugin.
749+
out.Remove(_T("clutSize"));
750+
735751
Callback1 callback =
736752
[&, id](const JsonObject& in) {
737753
if (!in["success"].Boolean()) {

0 commit comments

Comments
 (0)