[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#836910: jessie-pu: package kamailio/4.2.0-2+deb8u1



On Sun, Sep 11, 2016 at 20:48:07 +0200, Julien Cristau wrote:

> > +diff --git a/modules/tls/tls_init.c b/modules/tls/tls_init.c
> > +index a381be1..7bfc10f 100644
> > +--- a/modules/tls/tls_init.c
> > ++++ b/modules/tls/tls_init.c
> > +@@ -543,8 +543,10 @@ int init_tls_h(void)
> > + #endif
> > + 	ssl_version=SSLeay();
> > + 	/* check if version have the same major minor and fix level
> > +-	 * (e.g. 0.9.8a & 0.9.8c are ok, but 0.9.8 and 0.9.9x are not) */
> > +-	if ((ssl_version>>8)!=(OPENSSL_VERSION_NUMBER>>8)){
> > ++	 * (e.g. 0.9.8a & 0.9.8c are ok, but 0.9.8 and 0.9.9x are not)
> > ++	 * - values is represented as 0xMMNNFFPPS: major minor fix patch status
> > ++	 *   0x00090705f == 0.9.7e release */
> > ++	if ((ssl_version>>12)!=(OPENSSL_VERSION_NUMBER>>12)){
> > + 		LOG(L_CRIT, "ERROR: tls: init_tls_h: installed openssl library "
> > + 				"version is too different from the library the ser tls module "
> > + 				"was compiled with: installed \"%s\" (0x%08lx), compiled "
> 
> TBH, this seems just as wrong; libssl has a SONAME for a reason, no need
> to reinvent broken checks in each user.
> 
If I'm reading it right, the new check will still be unhappy with
libssl1.0.0 1.0.2h-1~bpo8+2 from jessie-backports, whereas that should
be ABI-compatible with libssl1.0.0 1.0.1t-1+deb8u2 from stable.

Cheers,
Julien


Reply to: