riley/uniproc
1
0
Fork 0
Code Issues 2 Pull Requests Packages Projects Releases Wiki Activity

Added error checking to ensure that we remain within the maximum of 64 objects.

Browse Source 
Reformatted a memcpy to have sizeof come before count.
This commit is contained in:
Riley King-Saunders
2026-07-19 22:08:10 +10:00
parent fe9ed46859
commit 71846beec5
1 changed files with 16 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/uniproc_win64.c
View File

@@ -158,6 +158,11 @@ void uniproc_await_processes(const uniproc_process* p, int* return_codes, const
h[num_valid_handles++] = (HANDLE)p[i]._proc_hdl; h[num_valid_handles++] = (HANDLE)p[i]._proc_hdl;
} }
if (num_valid_handles == 0) return; if (num_valid_handles == 0) return;
if (num_valid_handles >= MAXIMUM_WAIT_OBJECTS)
{
__debugbreak();
exit(-1);
}
WaitForMultipleObjects(num_valid_handles, h, 1, INFINITE); WaitForMultipleObjects(num_valid_handles, h, 1, INFINITE);
free(h); free(h);
@@ -190,6 +195,11 @@ int uniproc_are_processes_finished(const uniproc_process* p, const size_t num_pr
h[i] = (HANDLE)p[i]._proc_hdl; h[i] = (HANDLE)p[i]._proc_hdl;
num_valid_handles++; num_valid_handles++;
} }
if (num_valid_handles >= MAXIMUM_WAIT_OBJECTS)
{
__debugbreak();
exit(-1);
}
if (num_valid_handles == 0) return 1; if (num_valid_handles == 0) return 1;
DWORD ret = WaitForMultipleObjects(num_valid_handles, h, 1, 0); DWORD ret = WaitForMultipleObjects(num_valid_handles, h, 1, 0);
free(h); free(h);
@@ -197,7 +207,7 @@ int uniproc_are_processes_finished(const uniproc_process* p, const size_t num_pr
} }
void uniproc_nullify_processes(uniproc_process* p, const size_t num_processes) void uniproc_nullify_processes(uniproc_process* p, const size_t num_processes)
{ memset(p, 0, num_processes * sizeof(uniproc_process)); } { memset(p, 0, sizeof(uniproc_process) * num_processes); }
int uniproc_is_process_null(const uniproc_process* p) int uniproc_is_process_null(const uniproc_process* p)
{ return p->_proc_hdl == NULL; } { return p->_proc_hdl == NULL; }
@@ -212,6 +222,11 @@ void uniproc_await_any_processes(const uniproc_process* p, const size_t num_proc
h[i] = (HANDLE)p[i]._proc_hdl; h[i] = (HANDLE)p[i]._proc_hdl;
num_valid_handles++; num_valid_handles++;
} }
if (num_valid_handles >= MAXIMUM_WAIT_OBJECTS)
{
__debugbreak();
exit(-1);
}
if (num_valid_handles == 0) return; if (num_valid_handles == 0) return;
WaitForMultipleObjects(num_valid_handles, h, 0, INFINITE); WaitForMultipleObjects(num_valid_handles, h, 0, INFINITE);
free(h); free(h);