fork/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2026-10-07 09:38:11 +08:00
Code Issues Packages Projects Releases Wiki Activity

Guard attribute writes against an uninstalled layout offset

Browse Source
This commit is contained in:
Dennis Kobert
2026-09-07 16:12:01 +00:00
parent 0fc4424a09
commit df99ab3a72
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
node-graph/libraries/core-types/src/record/serve.rs
+7 -1
View File
@@ -130,8 +130,14 @@ impl<'e, 'l> FrameClaim<'e, 'l> {
/// Writes a field at its wiring-resolved offset. /// Writes a field at its wiring-resolved offset.
/// ///
/// # Safety /// # Safety
/// `offset` must be this layout's resolved offset for a field of `T`. /// `offset` must be this layout's resolved offset for a field of `T`. A
/// generated write offset defaults to 0 until `set_layout` installs it, so
/// the debug assertion catches a serve that ran before the install.
pub unsafe fn attr_at<T>(&mut self, offset: usize, value: T) { pub unsafe fn attr_at<T>(&mut self, offset: usize, value: T) {
debug_assert!(
self.layout.fields.iter().any(|field| field.offset == offset),
"attribute write at an offset this layout does not carry; the layout must install before serving"
);
unsafe { write_field(self.dst(), offset, value) }; unsafe { write_field(self.dst(), offset, value) };
self.filled_fields = true; self.filled_fields = true;
} }