debug_printf: simple validation of debugPrintf arguments

gfx-rs · Oct 16, 2023 · 72f4c6b · 72f4c6b
1 parent 8cf9c33
commit 72f4c6b
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/src/valid/function.rs b/src/valid/function.rs
@@ -139,6 +139,11 @@ pub enum FunctionError {
     InvalidRayDescriptor(Handle<crate::Expression>),
     #[error("Ray Query {0:?} does not have a matching type")]
     InvalidRayQueryType(Handle<crate::Type>),
+    #[error("Printf value argument {index} expression is invalid")]
+    InvalidPrintfArgument {
+        index: usize,
+        source: ExpressionError,
+    },
     #[error(
         "Required uniformity of control flow for {0:?} in {1:?} is not fulfilled because of {2:?}"
     )]
@@ -919,8 +924,15 @@ impl super::Validator {
                         crate::RayQueryFunction::Terminate => {}
                     }
                 }
-                S::DebugPrintf { .. } => {
-                    // FIXME
+                S::DebugPrintf { ref arguments, .. } => {
+                    for (index, &expr) in arguments.iter().enumerate() {
+                        context
+                            .resolve_type_impl(expr, &self.valid_expression_set)
+                            .map_err_inner(|source| {
+                                FunctionError::InvalidPrintfArgument { index, source }
+                                    .with_span_handle(expr, context.expressions)
+                            })?;
+                    }
                 }
             }
         }