Merge pull request #51 from RedBeardLab/null_terminator

Null terminator

Author: siscia

Cargo.lock

@@ -446,7 +446,7 @@ pub unsafe fn string_ptr_len(
         rm::ffi::RedisModule_StringPtrLen.unwrap()(str, &mut len)
             as *mut u8;
     let slice = slice::from_raw_parts(base, len);
-    str::from_utf8_unchecked(slice)
+    str::from_utf8_unchecked(slice).trim_end_matches(char::from(0))
 }
 
 #[repr(C)]

redisql_lib/src/redis.rs

@@ -854,6 +854,12 @@ def test_without_file(self):
     shutil.rmtree(path)
     self.assertTrue(True)
 
+class TestNullTerminatedStrings(TestRediSQLWithExec):
+  def test_null_terminated(self):
+      with DB(self, "NULL"):
+          one = self.exec_naked("REDISQL.EXEC", "NULL", "SELECT 1" + b'\x00')
+          self.assertEquals(one, [[1]])
+
 if __name__ == '__main__':
    unittest.main()