Error in passwordparsing

[Arran Cudbard-Bell]

> On 24 Jul 2015, at 05:25, Herwin Weststrate <herwin at quarantainenet.nl> wrote:
> 
> On 24-07-15 10:39, Stabla, Daniel wrote:
>> Hello,
>> 
>> don't know if it's a bug or a change (doesn't found anything in the
>> release notes), but if
>> you use commas for the password in e.g. clients.conf, then the password
>> itself gets malformed.
> 
> I assume you mean a shared secret here, not a password. The problem here
> is the config parser, that threats a comma as a split character. If I
> change the default from "testing123" to "testing123,45", I get the
> following message:
> 
> Warning: etc/raddb/clients.conf[100]: The item '45' is defined, but is
> unused by the configuration
> 
> And the shared secret "testing123" works just fine.
> 
> Adding some quotes around the shared secret solves the problem, so IMHO
> it's not a bug.

I know the change and the reason for it, and it's not a bug.  Wrap the string in quotes.

Arran Cudbard-Bell <a.cudbardb at freeradius.org>
FreeRADIUS development team

FD31 3077 42EC 7FCD 32FE 5EE2 56CF 27F9 30A8 CAA2

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 872 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freeradius.org/pipermail/freeradius-users/attachments/20150724/e68ce8bc/attachment.sig>