fix(execution): forbid calling cairo0 contract with cairo1 only built…

…ins (#128)

crates/blockifier/src/execution/deprecated_entry_point_execution.rs

@@ -1,3 +1,5 @@
+use std::collections::HashSet;
+
 use cairo_vm::types::builtin_name::BuiltinName;
 use cairo_vm::types::layout_name::LayoutName;
 use cairo_vm::types::relocatable::{MaybeRelocatable, Relocatable};
@@ -29,6 +31,15 @@ pub struct VmExecutionContext<'a> {
     pub entry_point_pc: usize,
 }
 
+pub const CAIRO0_BUILTINS_NAMES: [BuiltinName; 6] = [
+    BuiltinName::range_check,
+    BuiltinName::pedersen,
+    BuiltinName::ecdsa,
+    BuiltinName::bitwise,
+    BuiltinName::ec_op,
+    BuiltinName::poseidon,
+];
+
 /// Executes a specific call to a contract entry point and returns its output.
 pub fn execute_entry_point_call(
     call: CallEntryPoint,
@@ -71,9 +82,19 @@ pub fn initialize_execution_context<'a>(
     resources: &'a mut ExecutionResources,
     context: &'a mut EntryPointExecutionContext,
 ) -> Result<VmExecutionContext<'a>, PreExecutionError> {
+    // Verify use of cairo0 builtins only.
+    let program_builtins: HashSet<&BuiltinName> =
+        HashSet::from_iter(contract_class.program.iter_builtins());
+    let unsupported_builtins =
+        &program_builtins - &HashSet::from_iter(CAIRO0_BUILTINS_NAMES.iter());
+    if !unsupported_builtins.is_empty() {
+        return Err(PreExecutionError::UnsupportedCairo0Builtin(
+            unsupported_builtins.iter().map(|&item| *item).collect(),
+        ));
+    }
+
     // Resolve initial PC from EP indicator.
     let entry_point_pc = resolve_entry_point_pc(call, &contract_class)?;
-
     // Instantiate Cairo runner.
     let proof_mode = false;
     let trace_enabled = false;

crates/blockifier/src/execution/errors.rs

@@ -1,3 +1,6 @@
+use std::collections::HashSet;
+
+use cairo_vm::types::builtin_name::BuiltinName;
 use cairo_vm::types::errors::math_errors::MathError;
 use cairo_vm::vm::errors::cairo_run_errors::CairoRunError;
 use cairo_vm::vm::errors::memory_errors::MemoryError;
@@ -42,6 +45,8 @@ pub enum PreExecutionError {
     StateError(#[from] StateError),
     #[error("Requested contract address {:#064x} is not deployed.", .0.key())]
     UninitializedStorageAddress(ContractAddress),
+    #[error("Called builtins: {0:?} are unsupported in a Cairo0 contract")]
+    UnsupportedCairo0Builtin(HashSet<BuiltinName>),
 }
 
 impl From<RunnerError> for PreExecutionError {

crates/papyrus_monitoring_gateway/src/lib.rs

@@ -313,7 +313,7 @@ async fn node_config_by_secret(
 async fn metrics(prometheus_handle: Option<PrometheusHandle>) -> Response {
     match prometheus_handle {
         Some(handle) => {
-            Collector::default().prefix(PROCESS_METRICS_PREFIX).collect();
+            Collector::new(PROCESS_METRICS_PREFIX).collect();
             handle.render().into_response()
         }
         None => StatusCode::METHOD_NOT_ALLOWED.into_response(),