1
84%
master: 84%

Ran 05 Oct 2018 04:46PM UTC

Files 35

Run time 1s

Badge

Embed ▾

Committed 05 Oct 2018 04:31PM UTC coverage: 73.721% (-0.4%) from 74.083%

Job # 320.1

Build Type

push

travis-ci

Committed by

vanzin

Commit Message

CRYPTO-141: Avoid calling operations on an invalid context.

There are two main changes here.

The first one wraps the OpenSSL cipher context with some extra state
needed by the JNI wrapper. This allows checking whether it's safe to
call operations such as EVP_CipherUpdate, which can cause problems if
the context is not properly initialized. This more closely follows the
semantics of the Java interface.

The second change avoids cleaning up the OpenSSL context when an error
happens. Doing that would invalidate the memory used by the context,
but the Java code would still have the address of the now invalid
context, and calling another JNI method would crash the JVM. This
ties the lifecycle of the OpenSSL context to the Java cipher instance.

As part of those changes, all uses of EVP_CIPHER_CTX_cleanup were
removed; the remaining cleanup paths use EVP_CIPHER_CTX_free,
which is its replacement.

There are a few more changes that are not required for the above but
helped with testing and debugging:

- some very minor changes in Java code, especially using final fields
  where they make sense.
- enable -Wall and -Werror when compiling on linux64.
- fixed some minor issues with error handling in the native code.

Closes #80

Run Details

1282 of 1739 relevant lines covered (73.72%)

0.74 hits per line

apache / commons-crypto / 320 / 1
84%
master: 84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 320.1

apache / commons-crypto / 320 / 1 84% master: 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 320.1

apache / commons-crypto / 320 / 1
84%
master: 84%

README BADGES
x