sql_set_user bug (was Re: Possible bug in unlang?)

[Alan DeKok]

Phil Mayers wrote:
> It's also appending a 2nd "w" on the end, almost as if something is
> re-using the original string buffer:
> 
> "IC\rmtw"
> 
> ...and writing "IC<cr>mtw" into it, giving:
> 
> "IC<cr>mtww"

  That looks like a separate bug.  Try the "valuepair.diff" patch first.

> I can work up a patch for the sql_set_user functions in rlm_sql and
> rlm_sql_log; I take it then this is un-related to the unlang issue?

  Yes.

  Try the sql.diff file for the \r bug.

  Alan DeKok.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sql.diff
Type: text/x-patch
Size: 834 bytes
Desc: not available
URL: <http://lists.freeradius.org/pipermail/freeradius-users/attachments/20080814/15720812/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: valuepair.diff
Type: text/x-patch
Size: 342 bytes
Desc: not available
URL: <http://lists.freeradius.org/pipermail/freeradius-users/attachments/20080814/15720812/attachment-0001.bin>