Commit report for v3.0.x branch

The git bot announce at freeradius.org
Thu Mar 7 01:00:02 CET 2019


New activity for FreeRADIUS (the high performance and highly configurable RADIUS server)

======
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJcgEcPCRBK7hj4Ov3rIwAAdHIIAAFF4l4RGE3f0U2VwqcVrKYg
 FFqxiovYFyWJfcaNLdTtCxgaoJETCZkxOkTFksV44Qf0XzkEuz4n/vK4JKAnnKc3
 4vGJ6K1m2RpxPXNG2pjoJd6BxOIRc+0dZ/N+A1iYoFDKdXFqxQMfB1xWzR2v2WFv
 HyUo49eh88EQNaR5dgEGvEik/N3hKzWUthJOdwUM4glBN2EQSJIWuZDb/jyFQAse
 HPkxNxq2BiczDECklDuJmVEZV+WvbveJDu/YqvTfYW8QqudDb/2xJAIMPyVtpeHQ
 MTX2e3UQjC0o3NAn8xHNZDIar+2pgppx+E0RD+Gyh4rIAkwwwH/fwDEPc+8kbV0=
 =pnEc
 -----END PGP SIGNATURE-----
 

Merge pull request #2557 from alejandro-perez/patch-1

Fix TrustRouter query potential deadlock

Alan DeKok (via GitHub)@2019-03-06T22:17:51Z
Files modified:
	* src/modules/rlm_realm/trustrouter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c28b0c154d9fd79b95d75201c30239d99eab0afa
====== 
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCAAdFiEEAD8iviBDz9wiyN7eC8nJ3sjiRa4FAlyAD9kACgkQC8nJ3sji
 Ra4elg//cRlHdQcdKyp380c9KuHKkyY9GlDBkFcDDZKflN9jlb7JsZgapxi8melT
 ArKYwlqkznHzo3jOU8LM8yrv827p45gFcAtWLnHi+QO8wATY0TD+maoKOnf9uVwW
 6IlYSb0S6pAExLtczMYqF3f/PbofUh+eNmLqJzAStUeZBc7hngjHksqoe2MJ1Va/
 sf69TG5yNyS1kZpZxWLXA19qXfioX/+GdWeyyFKAxi70t+6JhiGpe6V7N7y/xgel
 Seiyxy42qcVGELL2C0L2XzOx38jL25vzqMTKxubNnOwQ1iwlkGLzUwMSMBPY411m
 gWbWTD1jcVMVDPZp+U5dH7GdQwQvz42F79wHad0r+l+O7v/119k0I4742N2hyozV
 ogD6vTMeDRFKU51BveFKUl7gs91erIV958TFET3tDafsVTWAMjzUz/z454QnLwCl
 E6muCZvd5Vsq/OkbZldVMS8Kg3e6wMk9ySPSloGJrb8Na5dxPak7txJw2VvJcbn4
 f6oXwvi/1Ww5xfXLVp3THALpizOURthM5pNAjw8BEUG1DGXV91ouuF3P8kn4Iy8I
 IJWDz0GeHpGlAMlvIQ4k/uloYiHayIk7Pe4pNEYoQt7quRclHYTznby61xh2gjRs
 T7sfQCLAeOo+CKHi13Ke4+3cgVmrg76/MyWzZbPWu9jPoKIlbRE=
 =Kw0y
 -----END PGP SIGNATURE-----

Fix TrustRouter query potential deadlock

Existing code uses a single lock to protect concurrent access to the
global_tidc variable. Under certain circumstances, two FreeRadius servers
wanting to resolve each other's realms at the same time might provoke
a deadlock situation, if this query triggered in turn another realm
resolution attempt for the APC's realm on both sides.

While this is a corner case, it has been observed in stress testing
environments.

This patch gets rid of the global_tidc in favour of a local approach,
and provides a pool of mutexes that are used to serialise different queries for
a particular realm, while allowing concurrent queries for different realms.

Alejandro Perez Mendez (via Alejandro Perez)@2019-03-06T18:14:54Z
Files modified:
	* src/modules/rlm_realm/trustrouter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/8ff5e2cb70a14f2af51c6ebeba29904e47ee657d
====== 
-- 
This commit summary was generated @2019-03-07T00:00:02Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).


More information about the Freeradius-Devel mailing list