Skip to content

Commit

Permalink
Merge updates from branch 'master' into github
Browse files Browse the repository at this point in the history
  • Loading branch information
rwbaumg committed Jul 2, 2019
2 parents f8983b5 + 57b91f0 commit 2622f56
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion TODO.md
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
# pki-lint TODO list

- [ ] Fix linking for ASN1_TIME_to_tm() function (currently breaking TravisCI build)
- [ ] Create new development branch for ```gs-certlint``` module to fix multiple bugs and/or contact upstream devs
- [ ] Support custom error levels for certain messages depending on context?
- [ ] Add support for optional downloading of AIA certificates to attempt to build missing chains.
- [ ] Add support for OpenSSL CRL validation along the entire chain (use ```-crl_check_all```).
- [ ] Add basic test PKI structure for post-build unit testing.
- [ ] Refactor LLVM / clang++ installation to use upstream source.
- [x] Fix linking for ASN1_TIME_to_tm() function (currently breaking TravisCI build)
- [x] Fix remaining shellcheck warnings and remove exclusions from ```make check``` target.
- [x] Check/fix zlint output processing for accuracy.
- [x] Fix cases where output is null (eg. certain GlobalSign lint output not processed).
Expand Down
12 changes: 9 additions & 3 deletions lint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1325,22 +1325,28 @@ CERTTOOL_VERSION=$(certtool --version | head -n1 | grep -Po '(?<=\s)[0-9\.]+$')
if version_gt "$CERTTOOL_VERSION" "$CERTTOOL_MIN_VER"; then
CERTTOOL_CAN_VERIFY="true"
fi
if [ -z "${CERTTOOL_VERSION}" ]; then
exit_script 1 "Failed to detect installed certtool version."
fi

if [ $VERBOSITY -gt 1 ]; then
print_info "Detected certtool version ${CERTTOOL_VERSION}"
fi

OPENSSL_IS_OLD="true"
OPENSSL_VERSION_NUM=$(openssl version | grep -Po '(?<=OpenSSL\s)\d\.\d\.\d(?=[a-z]\s)')
OPENSSL_VERSION_NUM=$(openssl version | grep -Po '(?<=OpenSSL\s)\d\.\d\.\d(?=[a-z]\s)?')
OPENSSL_VERSION_EXT=$(openssl version | grep -Po '(?<=OpenSSL\s\d\.\d\.\d)[a-z](?=\s)')
OPENSSL_FULLVERSION="${OPENSSL_VERSION_NUM}${OPENSSL_VERSION_EXT}"
if [ "$OPENSSL_VERSION_NUM" == "$OPENSSL_MIN_VERSION_NUM" ] || version_gt "$OPENSSL_VERSION_NUM" "$OPENSSL_MIN_VERSION_NUM"; then
if version_gt "$OPENSSL_VERSION_NUM" "$OPENSSL_MIN_VERSION_NUM"; then
REQ_EXT_NUMBER=$(printf '%d' "'$OPENSSL_MIN_VERSION_EXT")
CUR_EXT_NUMBER=$(printf '%d' "'$OPENSSL_VERSION_EXT")
if [ "${CUR_EXT_NUMBER}" -ge "${REQ_EXT_NUMBER}" ]; then
if [ -z "${OPENSSL_VERSION_EXT}" ] || [ "${CUR_EXT_NUMBER}" -ge "${REQ_EXT_NUMBER}" ]; then
OPENSSL_IS_OLD="false"
fi
fi
if [ -z "${OPENSSL_FULLVERSION}" ]; then
exit_script 1 "Failed to detect installed OpenSSL version."
fi

if [ $VERBOSITY -gt 1 ]; then
print_info "Detected OpenSSL version ${OPENSSL_FULLVERSION}"
Expand Down
2 changes: 1 addition & 1 deletion lints/x509lint
Submodule x509lint updated 3 files
+155 −0 asn1_time.c
+22 −0 asn1_time.h
+3 −2 checks.c

0 comments on commit 2622f56

Please sign in to comment.