Merge pull request #1722 from ignatz/fix_empty_blob

Fix panic in unsafe code when unpacking empty blobs.

Author: MarinPostma

libsql/src/value.rs

@@ -244,10 +244,11 @@ impl From<libsql_sys::Value> for Value {
                 assert!(len >= 0, "unexpected negative bytes value from sqlite3");
 
                 let mut v = Vec::with_capacity(len as usize);
-
-                let slice: &[u8] =
-                    unsafe { std::slice::from_raw_parts(blob as *const u8, len as usize) };
-                v.extend_from_slice(slice);
+                if !blob.is_null() {
+                    let slice: &[u8] =
+                        unsafe { std::slice::from_raw_parts(blob as *const u8, len as usize) };
+                    v.extend_from_slice(slice);
+                }
                 Value::Blob(v)
             }
         }

libsql/tests/integration_tests.rs

@@ -526,6 +526,17 @@ async fn blob() {
 
     let out = row.get::<Vec<u8>>(1).unwrap();
     assert_eq!(&out, &bytes);
+
+    let empty: Vec<u8> = vec![];
+    let mut rows = conn
+        .query(
+            "INSERT INTO bbb (data) VALUES (?1) RETURNING *",
+            [Value::Blob(empty.clone())],
+        )
+        .await
+        .unwrap();
+    let row = rows.next().await.unwrap().unwrap();
+    assert_eq!(row.get::<Vec<u8>>(1).unwrap(), empty);
 }
 
 #[tokio::test]