denozord/strongswan-ext
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

processor: Move priority threads assignment to set_threads()

Browse Source 
This avoids the evaluation of %N even if the thread pool is never used.
We need to avoid as many custom printf specifiers as possible when
fuzzing our code to avoid excessive log messages.
This commit is contained in:
Tobias Brunner
2017-05-23 18:29:12 +02:00
parent 05eeffb2f2
commit 83eacc448c
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/libstrongswan/processing/processor.c
+9 -4
View File
@@ -429,7 +429,15 @@ METHOD(processor_t, execute_job, void,
METHOD(processor_t, set_threads, void,
private_processor_t *this, u_int count)
{
int i;
this->mutex->lock(this->mutex);
for (i = 0; i < JOB_PRIO_MAX; i++)
{
this->prio_threads[i] = lib->settings->get_int(lib->settings,
"%s.processor.priority_threads.%N", 0, lib->ns,
job_priority_names, i);
}
if (count > this->total_threads)
{ /* increase thread count */
worker_thread_t *worker;
@@ -551,13 +559,10 @@ processor_t *processor_create()
.job_added = condvar_create(CONDVAR_TYPE_DEFAULT),
.thread_terminated = condvar_create(CONDVAR_TYPE_DEFAULT),
);
for (i = 0; i < JOB_PRIO_MAX; i++)
{
this->jobs[i] = linked_list_create();
this->prio_threads[i] = lib->settings->get_int(lib->settings,
"%s.processor.priority_threads.%N", 0, lib->ns,
job_priority_names, i);
}
return &this->public;
}