Unlock mutex when we find a matching connection

Free newfd on error

Free correct VP

One more check for size > 253

Add empty field to struct

Fix for coverity (missed mutex free)

More debug macros

DEBUGE, DEBUGW, for errors / warnings.  RDEBUG*, too.

This lets the errors and warnings have a common format, so that
the messages don't need to use an explicit "WARNING" string.

It also means that they are now colorized:
warning = bold + yellow
error = bold + red

This should hopefully help people who have issues reading the
debug output

Turn colorize off only if it was one

Check return code of pairmark_xlat

Whitespace fixes

value.xlat should be set to NULL after being freed

Simplify copying of xlat value

Remove L_CONS

Remove redundant checks for colourisation

Check whether it's appropriate to output colourised log messages

Quick hack for colourising log output to stderr/stdout

Write newline on buffer truncation too

Be explicit about why were not building rlm_rest

Update autoconf.h.in

Check for libcurl >= 17.9.1

Disable definitions for ancient code depending on libcurl

dir can't be NULL

Check pointers before using them

Check length of ascend secrets

Remove finger functionality from radwho.  No one uses it (or should)

Check for NULL before incrementing

Check array[0], not array

Error if the dictionaries weren't initialized

Correct args to memset

Use strlcpy instead of strcpy

Use sizeof instead of strlen

Return on error

EOL is EOL

EOL is EOL

Fix build error

d_namlen is BSD, not posix

Fix for coverity (printf of non null terminated string)

Clean up and sanitize pairread()

Should fix a number of coverity issues

Fix for coverity (potential buffer overflow in radlog_request)

Fix race condition in conf file

Closes Coverity #720457

Fix typo

Check for =~ and !~

Fixes Coverity #982075

Check for no challenge/response in packet

Closes Coverity #720480

Remove unused variables

Remove unused variable

Fix for coverity (redundant default statement)

Fix for coverity (redundant condition)

Fix for coverity (duplicate case statements)

Fix for coverity (remove redundant default statement)

Output proper error messages for prthread_setspecific failure in rlm_perl.c and lib/log.c

Fix for coverity (verify we parsed the proxy config section ok)

Fix for covertiy

Fix for coverity (check setsockopt in listen.c)

Fix for coverity (check return of ip_hton in detail.c)

Check return code of dup()

Closes Coverity #720380, 81, 82

Don't check array for existence.

Closes Coverity #720384

Don't check an array for existence

Closes Coverity #720389

Fix for coverity (bail if we can't stat the file in detail_recv)

Fix for coverity (more radrecv stuff)

Fix for coverity

Use strlcpy, and check sizeof module name

Closes Covierty #720447

Use strlcpy

Fail safely if the directory exists in rad_mkdir

Maybe another thread is making the directory.  If so, it's OK

Closes Coverity #720459

Initialize IPaddr

Closes Coverity #720462

Sign error

Closes Coverity #720436

Clear memory using correct size

Closes Coverity #720439, 40, 41

Don't expand variables if cs==NULL

Closes Coverity #720373

Fix for coverity (don't need to check recvfrom return in rad_recv_discard)

Fix for coverity (check return value of setsockopt)

Fix for coverity

Remove dead code

Closes Coverity #982076

Remove dead code.

Closes Coverity #982077

Use correct flag for check

Closes Coverity #982078

Don't free data returned from gdbm_fetch

Closes Coverity #982067, 982068, 982069

Remove dead code.

Closes Coverity #982080, 982079

Check for NULL da.  Only if dictionary files are broken.

Closes Coverity #982081

value may be NULL.  Only for admin mistake.

Closes Coverity #982082

Check for NULL.

Closes Coverity #982083 (false positive)

Limit attrlen to 128K.

Closes Coverity #982085

Fix for coverity

Fix for coverity

Fix for coverity

Fix for coverity

Fix for coverity (don't segv if the ruby function returns an array during intiialisation)

Fix for coverity (assign return from recvfrom to a local signed variable so we can check for error conditions)

Unlock on error

Closes Coverity #982086

Add missing break

Closes Coverity #982087

Add missing break

Closes Coverity #982088

Note fall-through

Closes Coverity #982089

Add missing break

closes Coverity #982090

Remove unnecessary check

Closes Coverity #982093

Use sizeof(struct) not sizeof(*struct)

Closes Coverity #982094

There's no need to memset data to zero

Closes Coverity #982095

Check size of raw attributes in TTLS

Closes Coverity #982096

Free vp on error

Closes Coverity #982097

Fix for coverity (check return value of cf_section_parse)

Fix for coverity (potential segv for very long libpaths in run_mode)

Fix for coverity (potential segv for argument names over 49 bytes in jlibtool)

Fix for coverity (fix potential buffer overflow in cleanup_tmp_dir)

Fix for coverity (zero out sizeof(*fi) not sizeof(fi) in fr_fifo_free)

Fix for coverity (zero out sizeof(*packet) not sizeof(packet) in fr_dhcp_rcv)