RDKECOREMW-893 : setVendorID API returns a success status of true when the parameter is not provided (#271)

AkshayKumar2794 · web-flow · commit 20e867c0eb7d · 2025-10-16T09:28:59.000-04:00
diff --git a/HdmiCecSource/HdmiCecSourceImplementation.cpp b/HdmiCecSource/HdmiCecSourceImplementation.cpp
@@ -1224,6 +1224,11 @@ namespace WPEFramework
         Core::hresult HdmiCecSourceImplementation::SetVendorId(const string &vendorid, HdmiCecSourceSuccess &success)
         {
             LOGINFO("SetVendorId :%s ",vendorid.c_str());
+            if (vendorid.empty()) {
+                LOGERR("SetVendorId failed: vendorid is not given");
+                success.success = false;
+                return Core::ERROR_GENERAL;
+            }
             unsigned int vendorIdInt = 0;
             try
             {

Original file line number	Diff line number	Diff line change
`@@ -1224,6 +1224,11 @@ namespace WPEFramework`
`1224`	`1224`	`Core::hresult HdmiCecSourceImplementation::SetVendorId(const string &vendorid, HdmiCecSourceSuccess &success)`
`1225`	`1225`	`{`
`1226`	`1226`	`LOGINFO("SetVendorId :%s ",vendorid.c_str());`
	`1227`	`+ if (vendorid.empty()) {`
	`1228`	`+ LOGERR("SetVendorId failed: vendorid is not given");`
	`1229`	`+ success.success = false;`
	`1230`	`+ return Core::ERROR_GENERAL;`
	`1231`	`+ }`
`1227`	`1232`	`unsigned int vendorIdInt = 0;`
`1228`	`1233`	`try`
`1229`	`1234`	`{`