|
Message-ID: <54B8F9B3.4030809@dd-wrt.com> Date: Fri, 16 Jan 2015 12:44:51 +0100 From: Sebastian Gottschall <s.gottschall@...wrt.com> To: Natanael Copa <ncopa@...inelinux.org> CC: musl@...ts.openwall.com Subject: Re: pthreads broken (freeradius testcase) Am 16.01.2015 um 08:20 schrieb Natanael Copa: > On Thu, 15 Jan 2015 23:52:36 +0100 > Sebastian Gottschall <s.gottschall@...wrt.com> wrote: > >> following test case >> >> configure freeradius with --with-threads (which is on by default) >> if you start radiusd with your radius configuration you will see that >> radius does not listen on any ports. it will hang in the listener thread >> which creates the socket. >> if you configure it as --without-threads, it works >> >> >> tested with musl 1.1.6 on a mips (big endian) system using kernel 3.10 >> >> Sebastian > What version of freeradius is it? > > I have had some interesting threading issues with freeradius 2.2.x. > Some modules are marked as non-thread safe but will still run in a > separate thread. It runs main thread + a single non-thread-safe thread. > > They used getgrnam and getpwnam in both main thread and in the > non-thread-safe module so memory got corrupted. (IMHO this should get a > CVE but upstream disagrees because it only happens on a non-recommended > config) > > They fixed that in 3.x.x but AFAIK they didn't fix it in 2.x.x. > > Patches: > http://git.alpinelinux.org/cgit/aports/tree/main/freeradius/0001-Use-threadsafe-wrapper-for-getpwnam-getgrnam.patch > http://git.alpinelinux.org/cgit/aports/tree/main/freeradius/0001-use-threadsafe-rad_getgrnam.patch > > (upstream patched it differently in 3.x.x branch) > > When backporting the fix to 2.x.x I also found that the TLS configure > test is completely broke in 2.x.x branch too. IIRC it will say "TLS > found" but behind the scenes it will still disable TLS support. > > patch: > http://git.alpinelinux.org/cgit/aports/tree/main/freeradius/fix-tls-test.patch > > This is probably not the related the issue you have have at hand, but > I'm would not be surprised if musl libc has unmasked another bug in > freeradius. i applied all these fixes. the first thread for port 1812 works, the second thread for internal tunnel 18120 doesnt work an hangs again. even if setuid support is disabled > > > -nc >
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.