Skip to content

Commit

Permalink
Don't ignore the fact we've hit vulnerable versions in previous checks
Browse files Browse the repository at this point in the history
  • Loading branch information
arr2036 committed Sep 26, 2016
1 parent 3d6b628 commit ca367b0
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions src/main/tls.c
Original file line number Diff line number Diff line change
Expand Up @@ -2483,17 +2483,22 @@ int tls_global_version_check(char const *acknowledged)
/*
* If the CVE is acknowledged, allow it.
*/
if (strcmp(acknowledged, defect->id) == 0) return 0;
if (!bad && (strcmp(acknowledged, defect->id) == 0)) return 0;

ERROR("Refusing to start with libssl version %s (in range %s)",
ssl_version(), ssl_version_range(defect->low, defect->high));
ERROR("Security advisory %s (%s)", defect->id, defect->name);
ERROR("%s", defect->comment);

INFO("Once you have verified libssl has been correctly patched, "
"set security.allow_vulnerable_openssl = '%s'", defect->id);
/*
* Only warn about the first one...
*/
if (!bad) {
INFO("Once you have verified libssl has been correctly patched, "
"set security.allow_vulnerable_openssl = '%s'", defect->id);

bad = true;
bad = true;
}
}
}

Expand Down

0 comments on commit ca367b0

Please sign in to comment.