Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-19 00:41:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

[wpiutil] Fix Semaphore backing vector usage (#8737)

Browse Source 
The wrong backing vector was being used (eventIds).
This commit is contained in:
Peter Johnson
2026-04-10 12:27:52 -07:00
committed by GitHub
parent 21f6bdbc1a
commit 7e86c10a28
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
wpiutil/src/main/native/cpp/Synchronization.cpp
View File

@@ -180,7 +180,7 @@ void wpi::util::DestroySemaphore(WPI_SemaphoreHandle handle) {
}
auto& manager = guard.GetManager();
std::scoped_lock lock{manager.mutex};
manager.eventIds.erase(handle & 0xffffff);
manager.semaphoreIds.erase(handle & 0xffffff);
}
bool wpi::util::ReleaseSemaphore(WPI_SemaphoreHandle handle, int releaseCount,
@@ -204,7 +204,7 @@ bool wpi::util::ReleaseSemaphore(WPI_SemaphoreHandle handle, int releaseCount,
return false;
}
auto& state = it->second;
int maxCount = manager.eventIds[index];
int maxCount = manager.semaphoreIds[index];
if (prevCount) {
*prevCount = state.signaled;
}