Skip to content

Conversation

@MozirDmitriy
Copy link

@MozirDmitriy MozirDmitriy commented Sep 8, 2025

This change replaces two calls to write(...) with write_all(...) when writing fixed-size payloads:

  • CustomGatesUsedSection: parameters (field_size bytes each)
  • CustomGatesAppliedSection: signal indices (8 bytes each)

The paired reader uses read_exact for these fields, so write(...) risks short writes and corrupting the bytestream. Using write_all(...) guarantees the full buffer is written, ensuring consistency with the reader and with the rest of the module, which already uses write_all for fixed-size data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant