Merge pull request #348 from arr2036/zero_length_exp

Don't generate errors on zero length expansions

Base sql configs are in main

Set dbc and env to NULL to avoid double free

Consistently return a zero length buffer from xlat_process, except on error condition (where we free)

Don't generate errors on zero length expansions

Notes on rlm_sql_freetds/rlm_sql_sybase

Fix quoting style in debug messages, should be double to match expansion quotes

Only list one deprecated alternative to avoid users needing to change the config item twice

Set the destructor only when necessary.  Closes #347

and reset it before returning the memory to the caller

Whitespace fixes

re-align utility help output

Update README.rst

Formatting

Another missed default

Only call paircompare register once per instance of rlm_ldap

Everything else uses attribute

Last tranche of config item name standardisation (in modules)

Use EXIT_FAILURE and EXIT_SUCCESS macros

Fix return codes in radiusd

Md not dM

Memory report and check when performing config check

Fix memory check on -C -m|-M

Fix up uses of talloc

and change one malloc to talloc

Minor updates

Update ldap

More updates and formatting

Update README.rst

Add notes on portmanteau config items

Be nice, and print out the name of the new config item if it's next in the array

Remove default for depricated config item

rlm_expiration no longer takes any configuration items

Missed removing config for rlm_expiration

Be consistent in our inconsistency

Typo

Use references in cache file example config

deprecate minimum-timeout in longin time, now minimum_timeout

Remove support for reply-message from expiration, this can be set in unlang

Replace remaining instances of safe-characters with safe_characters

Replace nas_table with client_table

Use standard TLS config item names across all modules

Although this represents a lot of pain for users upgrading, it allows (with the new section references) a single TLS configuration section to be used for all modules in the server.

Attribute should be singular not plaural

Support 'children' scope

Changes to default ldap configuration which were missed

Remove reference to certs patch

nas_query is deprecated, replaced by client_query

safe-charcters is deprecated, replaced by safe_characters
deletestalesessions is deprecated, replaced by delete_stale_sessions
readclients is deprecated, replaced by read_clients

Cleanup things in modcall and fix missing closing brace

SQL functions now use ssize_t so that they can indicate actual error conditions

A module expanding to a 0 length string is no longer considered and error condition

If a module xlat returns < 0 update sections will return fail

Don't shadow a global symbol

Reorder arguments to radius_exec_program so they make more sense

Use a single function to deal with status codes and module failure messages

Have fork return 2 (translates to RLM_MODULE_FAIL)

Merge pull request #342 from fajarnugraha/master-suse-20130510

Fix suse package to build and run cleanly for current master branch

suse: enable parallel build

suse: add --disable-developer

suse: cleanup CFLAGS

Removed '-fstack-protector -fno-strict-aliasing'. Already
included by default, no need to include it manually.

suse: link to wiki page for authors credit

Add PW_TYPE_FILE_INPUT (which verifies the file exists) and PW_TYPE_FILE_OUTPUT , which is still a file, but not required to exist

Whitespace changes rlm_exec

Add PW_TYPE_EXISTS

Which requires the file specified to exist on startup

Mark more things as filenames

Remove unecessary configure arguments

May as well make it official

suse: build fixes

* Merge most changes to suse/freeradius.spec from v2.x.x branch.
* Add suse/freeradius-server-rpmlintrc and
  suse/freeradius-server-tmpfiles.conf from v2.x.x branch.
* Merge changes on suse/freeradius-server-2.1.6-suseinit.patch.
* Add BuildRequires: libtalloc-devel.
* Remove BuildRequires: apache2-devel. Dialup admin was removed.
* Remove BuildRequires: cyrus-sasl-devel, openssl. Already installed
  as requirement for other BuildRequires.
* Remove BuildRequires: gettext-devel, libtool, net-snmp-devel.
  No longer needed.
* Remove unneeded configure flags.
* Make main package more generic by using wildcard in %files. For
  example, it will put resulting oracle libs (if enabled during build)
  on the main package.

loc is not options

If anything exists don't create the link...

Update README.rst

Update README.rst

Apparently making the symlink targets as order only dependencies isnt good enough

Don't start if the configuration contains deprecated config items

Don't try and link if legacy files exist already. Fixes #341

Non-existent attrs expand to "".  Closes #308

Always fixup after deleting a black child node in rbtree

This prevents rbtree_delete from creating unbalanced/illegal rbtrees.

Use a consistent callback return code

  This is the one place in the codebase where a positive return code
  is used in an rbtree_walk callback.  Changing it negative should have no
  effect, and keeps options open as to whether to introduce new symbols
  for the next patch to follow, or to change the libfreeradius API
  slightly WRT rbtree callback return codes.

  Whether any third-party add-ons may use positive return codes is not
  known.

Fix obvious pasto in PostOrder walk.

When a socket is closed, just mark the requests as STOP

Someone else will take care of cleaning them up.  Calling
request_done() appears to cause issues

Make auto_limit_acct probabalistic

Minor cleanup to cleanup code

Make requests NULL talloc context

there are just too many cases where the socket could go away,
and the request is still referenced somewhere else in the code.

Note changes to configuration item names

Be consistent with common config_item names

If a file is being referenced or created the config item 'filename' is used.

If a file is being created, the initial permissions are set by the 'permissions' config item.

If a directory hierarchy needs to be created the permissions are set by 'dir_permissions'

If an external host is referenced in the context of a module the 'server' config item is used.

file, script_file, module, detail, detailfile, attrsfile, perm, dirperm, detailperm,
hostname are all deprecated.

Remove debug macro

More docs on recent changes

Always set reply timestamp.

This allows TCP rejects to be sent.  (oops)

Steal the packet so we don't leak memory

Remove refcount on mainconfig.  It's not used

Simplify request_free() now that we use talloc everywhere

Steal the CoA packet when we reparent it

Make request_alloc() take a TALLOC_CTX

Indentation matters

Remove unused function

Move request data to talloc

Sprinkle PW_TYPE_REQUIRED over module configuration sections

Fix -HUP

Minor formatting in modules.c

added support for IPv6 nas in postgres accounting

Cleanup whitespace in config files

Minor fixes to cui policy

Merge pull request #337 from armitasp/master

updated cui policy

change cui policy to remove warning about outer request not being available inner context

Call wait() with WNOHANG to reap any children left over by exec calls

Update radiusd.conf man pages with the properties syntax