nik/bitchat
1
0
Fork 0
mirror of https://github.com/permissionlesstech/bitchat.git synced 2026-08-29 07:27:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix Periphery assign-only findings on QuotaByteReservation.

Browse Source 
Keep scope and bytes only in the coordination map; the reservation token is an opaque id so the dead-code scan stays clean under --strict.
This commit is contained in:
Ayush7614 2026-08-02 00:35:22 +05:30
parent dacc733153
commit a6ad4f6539
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
bitchat/Services/BLE/BLEIncomingFileStore.swift
View File

@ -112,10 +112,10 @@ struct BLEIncomingFileStore: @unchecked Sendable {
/// to `enforceQuota`, this stays charged against the quota until
/// `releaseQuotaReservation` — so cancel / encode failure / panic must
/// release it or later writers see a permanently tighter budget.
/// Scope and byte count live in `PayloadCoordination.byteReservations`
/// keyed by `id`; the token itself is only an opaque handle.
struct QuotaByteReservation: Sendable {
fileprivate let id: UUID
fileprivate let scope: MediaQuotaScope
fileprivate let bytes: Int64
}
private final class PayloadCoordination: @unchecked Sendable {
@ -627,11 +627,7 @@ struct BLEIncomingFileStore: @unchecked Sendable {
let clamped = Int64(max(0, bytes))
enforceQuotaLocked(reservingBytes: Int(clamped), scope: scope)
let reservation = QuotaByteReservation(
id: UUID(),
scope: scope,
bytes: clamped
)
let reservation = QuotaByteReservation(id: UUID())
payloadCoordination.byteReservations[reservation.id] = (
scope: scope,
bytes: clamped