Rooba
/
boinc
mirror of https://github.com/BOINC/boinc.git
1
0
Fork 0
Code Issues 1 Packages Projects Releases Wiki Activity

client: fix bug where project boolean attributes are stuck at true. 

Projects had some boolean attributes (verify_files_on_app_start,
ended, non_cpu_intensive) that are send in scheduler replies
and thereafter stored in the client state file.
If a project stopped sending one of these attributes,
it would never get cleared.

Solution: treat the absence of the flag in the scheduler reply
as meaning false.
This commit is contained in:
David Anderson 2015-06-14 13:07:54 -07:00
parent d49437b780
commit 656070aa6e
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
client/scheduler_op.cpp
View File

@ -578,6 +578,9 @@ int SCHEDULER_REPLY::parse(FILE* in, PROJECT* project) {
MIOFILE mf;
XML_PARSER xp(&mf);
std::string delete_file_name;
bool verify_files_on_app_start = false;
bool non_cpu_intensive = false;
bool ended = false;
mf.init_file(in);
bool found_start_tag = false, btemp;
@ -634,6 +637,13 @@ int SCHEDULER_REPLY::parse(FILE* in, PROJECT* project) {
if (project->dont_use_dcf) {
project->duration_correction_factor = 1;
}
// boolean project attributes.
// If the scheduler reply didn't specify them, they're not set.
//
project->verify_files_on_app_start = verify_files_on_app_start;
project->non_cpu_intensive = non_cpu_intensive;
project->ended = ended;
return 0;
}
else if (xp.parse_str("project_name", project->project_name, sizeof(project->project_name))) {
@ -834,9 +844,9 @@ int SCHEDULER_REPLY::parse(FILE* in, PROJECT* project) {
);
}
continue;
} else if (xp.parse_bool("non_cpu_intensive", project->non_cpu_intensive)) {
} else if (xp.parse_bool("non_cpu_intensive", non_cpu_intensive)) {
continue;
} else if (xp.parse_bool("ended", project->ended)) {
} else if (xp.parse_bool("ended", ended)) {
continue;
} else if (xp.parse_bool("no_cpu_apps", btemp)) {
if (!project->anonymous_platform) {
@ -861,7 +871,7 @@ int SCHEDULER_REPLY::parse(FILE* in, PROJECT* project) {
handle_no_rsc_apps(buf, project, true);
}
continue;
} else if (xp.parse_bool("verify_files_on_app_start", project->verify_files_on_app_start)) {
} else if (xp.parse_bool("verify_files_on_app_start", verify_files_on_app_start)) {
continue;
} else if (xp.parse_bool("send_full_workload", send_full_workload)) {
continue;