From 8c653212c0e6d15997e4217a514301f5682a41c4 Mon Sep 17 00:00:00 2001 From: Hisham Muhammad Date: Sun, 18 Feb 2018 10:38:49 -0300 Subject: Replace size_t with int/void* union I was occasionally passing negative values to size_t. Plus, this better reflects the intent of the variant argument. Reported by Coverity: https://scan8.coverity.com/reports.htm#v13253/p10402/fileInstanceId=22093891&defectInstanceId=7543346&mergedDefectId=174179&fileStart=251&fileEnd=500 --- Process.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'Process.c') diff --git a/Process.c b/Process.c index 18360802..2ff778df 100644 --- a/Process.c +++ b/Process.c @@ -536,11 +536,11 @@ bool Process_setPriority(Process* this, int priority) { return (err == 0); } -bool Process_changePriorityBy(Process* this, size_t delta) { +bool Process_changePriorityBy(Process* this, int delta) { return Process_setPriority(this, this->nice + delta); } -void Process_sendSignal(Process* this, size_t sgn) { +void Process_sendSignal(Process* this, int sgn) { CRT_dropPrivileges(); kill(this->pid, (int) sgn); CRT_restorePrivileges(); -- cgit v1.2.3