Merge pull request #13 from jonathanlim222/jonathanlim222-conformance-testing-fix-SliceByteString

fix: fixed SliceByteString tests 3 and 5

Author: yHSJ

crates/uplc/src/machine/runtime.rs

@@ -371,8 +371,8 @@ impl<'a> Machine<'a> {
             }
             DefaultFunction::SliceByteString => {
                 let arg1 = runtime.args[0].unwrap_integer()?;
-                let arg2 = runtime.args[1].unwrap_integer()?;
-                let arg3 = runtime.args[2].unwrap_byte_string()?;
+                let arg2: &'a num::BigInt = runtime.args[1].unwrap_integer()?;
+                let arg3: &'a [u8] = runtime.args[2].unwrap_byte_string()?;
 
                 let budget = self.costs.builtin_costs.slice_byte_string([
                     cost_model::integer_ex_mem(arg1),
@@ -384,17 +384,27 @@ impl<'a> Machine<'a> {
 
                 let skip: usize = if *arg1 < Integer::ZERO {
                     0
+                } else if *arg1 > arg3.len().into(){
+                    arg3.len()
                 } else {
                     arg1.try_into().expect("should cast to usize just fine")
                 };
 
                 let take: usize = if *arg2 < Integer::ZERO {
                     0
+                } else if *arg2 > arg3.len().into(){
+                    arg3.len()
                 } else {
                     arg2.try_into().expect("should cast to usize just fine")
                 };
 
-                let value = Value::byte_string(self.arena, &arg3[skip..(skip + take)]);
+                let skip_take: usize = if skip + take > arg3.len() {
+                    arg3.len()
+                } else {
+                    skip + take
+                };
+
+                let value = Value::byte_string(self.arena, &arg3[skip..(skip_take)]);
 
                 Ok(value)
             }