hjnormey/siloqy
Public Access
1
0
Fork 0
Code Issues 7 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

PINK: fix last c_char_p temporary in set_slot_json

Browse Source 
Completes the ctypes lifetime audit. All eight FFI call sites now
assign _to_rust_bytes() to a local var before passing to c_char_p,
ensuring the bytes object lives for the full duration of the Rust call.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Codex
2026-06-04 17:00:16 +02:00
parent acffc783e6
commit f78cc0d3f9
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
prod/clean_arch/dita_v2/rust_backend.py
View File

@@ -219,7 +219,8 @@ class _RustKernelLib:
return json.loads(self._take_string(raw)) return json.loads(self._take_string(raw))
def set_slot_json(self, handle: ctypes.c_void_p, slot_id: int, payload: Dict[str, Any]) -> None: def set_slot_json(self, handle: ctypes.c_void_p, slot_id: int, payload: Dict[str, Any]) -> None:
rc = self.lib.dita_kernel_set_slot_json(handle, ctypes.c_size_t(slot_id), ctypes.c_char_p(_to_rust_bytes(payload))) _pb = _to_rust_bytes(payload)
rc = self.lib.dita_kernel_set_slot_json(handle, ctypes.c_size_t(slot_id), ctypes.c_char_p(_pb))
if rc != 0: if rc != 0:
raise RuntimeError(f"dita_kernel_set_slot_json failed rc={rc}") raise RuntimeError(f"dita_kernel_set_slot_json failed rc={rc}")