GitLab

The session_keys array is not shared with the
template so if it was allocated just free it.

The template may not have been used for the deallocated
stream, therefore the size of the session_keys array
could be different.

Should maybe contain a pointer to template from stream
so it is explicitly known that it was used.

logic in srtp_stream_free was not correct and could result
in memory access errors, srtp_stream_dealloc can safely be used
instead and is "more" correct.

mki_index is a zero based index in the sesssion_keys
array which has a max length of num_master_keys.

Reported by Guido Vranken <guidovranken@gmail.com>

session_keys may not have been allocated yet.

reported by Guido Vranken <guidovranken@gmail.com>

Use explicit clang-format version on travis

When travis upgrade's clang it breaks current
formatting that was done with 3.9 . Better to explicitly
set the version to use.

Return NULL when allocating memory of size zero

bitvector of length zero is not valid

Fixes coverity scan issues 179781

If the length resulted in l == 0 then there was no
allocation of word, which could subsequently be
dereferenced. Treat length of zero as invalid to
avoid dealing with it.

Travis fixes

Add travis and coverity badges to README.md

moved externs from crypto_kernel into its own header

fixed several compiler warnings from Firefox builds

make read only arrays const

removed variable init code which never gets used

added comments to ifdef's for easier understanding

added error in case the platform is not detected

Currently no warnings are generated on travis so by enabling
this flag we will catch when any warnings are introduced.

The host is the OS that the binary will run on,
this ensures that CPU_RISC is set to true.

Run all test with make runtest.

Add coverity scan to tavis builds