[WebRTCP] Validate that the StreamUsage is in the StreamUsagePriorities list (#41776)

* Validate that the StreamUsage is in the StreamUsagePriorities list

* Address review comment

Author: yufengwangca

examples/camera-app/linux/include/clusters/webrtc-provider/webrtc-provider-manager.h

@@ -80,6 +80,8 @@ class WebRTCProviderManager : public Delegate, public WebRTCTransportProviderCon
 
     CHIP_ERROR ValidateAudioStreamID(uint16_t audioStreamId) override;
 
+    CHIP_ERROR IsStreamUsageSupported(StreamUsageEnum streamUsage) override;
+
     CHIP_ERROR IsHardPrivacyModeActive(bool & isActive) override;
 
     CHIP_ERROR IsSoftRecordingPrivacyModeActive(bool & isActive) override;

examples/camera-app/linux/src/clusters/webrtc-provider/webrtc-provider-manager.cpp

@@ -529,6 +529,30 @@ CHIP_ERROR WebRTCProviderManager::ValidateAudioStreamID(uint16_t audioStreamId)
     return avsmController.ValidateAudioStreamID(audioStreamId);
 }
 
+CHIP_ERROR WebRTCProviderManager::IsStreamUsageSupported(StreamUsageEnum streamUsage)
+{
+    if (mCameraDevice == nullptr)
+    {
+        ChipLogError(Camera, "CameraDeviceInterface not initialized");
+        return CHIP_ERROR_INCORRECT_STATE;
+    }
+
+    auto & hal                   = mCameraDevice->GetCameraHALInterface();
+    auto & streamUsagePriorities = hal.GetStreamUsagePriorities();
+
+    // Check if the streamUsage is in the StreamUsagePriorities list
+    for (const auto & usage : streamUsagePriorities)
+    {
+        if (usage == streamUsage)
+        {
+            return CHIP_NO_ERROR;
+        }
+    }
+
+    ChipLogError(Camera, "StreamUsage %u not found in StreamUsagePriorities", to_underlying(streamUsage));
+    return CHIP_ERROR_NOT_FOUND;
+}
+
 CHIP_ERROR WebRTCProviderManager::IsHardPrivacyModeActive(bool & isActive)
 {
     if (mCameraDevice == nullptr)

src/app/clusters/webrtc-transport-provider-server/webrtc-transport-provider-server.cpp

@@ -562,7 +562,13 @@ void WebRTCTransportProviderServer::HandleSolicitOffer(HandlerContext & ctx, con
         return;
     }
 
-    // TODO:#41458 If the passed in StreamUsage is not found in the StreamUsagePriorities, return DYNAMIC_CONSTRAINT_ERROR
+    // Validate that the StreamUsage is in the StreamUsagePriorities list
+    if (mDelegate.IsStreamUsageSupported(req.streamUsage) != CHIP_NO_ERROR)
+    {
+        ChipLogError(Zcl, "HandleSolicitOffer: StreamUsage %u is not in StreamUsagePriorities", to_underlying(req.streamUsage));
+        ctx.mCommandHandler.AddStatus(ctx.mRequestPath, Status::DynamicConstraintError);
+        return;
+    }
 
     // Validate VideoStreamID against AllocatedVideoStreams.
     // If present and null then a stream has to have been allocated.
@@ -845,7 +851,13 @@ void WebRTCTransportProviderServer::HandleProvideOffer(HandlerContext & ctx, con
             return;
         }
 
-        // TODO:#41458 If the passed in StreamUsage is not found in the StreamUsagePriorities, return DYNAMIC_CONSTRAINT_ERROR
+        // Validate that the StreamUsage is in the StreamUsagePriorities list
+        if (mDelegate.IsStreamUsageSupported(req.streamUsage) != CHIP_NO_ERROR)
+        {
+            ChipLogError(Zcl, "HandleProvideOffer: StreamUsage %u is not in StreamUsagePriorities", to_underlying(req.streamUsage));
+            ctx.mCommandHandler.AddStatus(ctx.mRequestPath, Status::DynamicConstraintError);
+            return;
+        }
 
         // If VideoStreamID is present and is not null and does not match a value in AllocatedVideoStreams:
         // Fail the command with the status code DYNAMIC_CONSTRAINT_ERROR

src/app/clusters/webrtc-transport-provider-server/webrtc-transport-provider-server.h

@@ -243,6 +243,23 @@ class Delegate
      */
     virtual CHIP_ERROR ValidateAudioStreamID(uint16_t audioStreamId) = 0;
 
+    /**
+     * @brief
+     *   Checks if the given StreamUsage is in the StreamUsagePriorities list.
+     *
+     *   This method is called during SolicitOffer and ProvideOffer command processing
+     *   to ensure that the requested stream usage is supported by the device according
+     *   to its configured priorities.
+     *
+     * @param[in] streamUsage  The stream usage to check.
+     *
+     * @return CHIP_ERROR
+     *   - CHIP_NO_ERROR - StreamUsage is found in StreamUsagePriorities.
+     *   - CHIP_ERROR_NOT_FOUND - StreamUsage is not in the StreamUsagePriorities list.
+     *   - Other error codes - For failures reading the attribute itself
+     */
+    virtual CHIP_ERROR IsStreamUsageSupported(Globals::StreamUsageEnum streamUsage) = 0;
+
     /**
      * @brief Check whether hard privacy mode is active.
      *