usage of "FR_TOKEN op" parameter to pairmake ?

[Arran Cudbard-Bell]

> 
> Your macro is wrong it should be
> 
> #define pairmove_reply(_vp, _a, _b, _c) do {\
> 	_vp = pairmake(request, NULL, _a, _b, _c);\
> 	pairmove(request, &(request->reply->vps), &_vp); \
> 	pairfree(&_vp); \
> while (0)

And Alan says, passing NULL to pairmake and then calling pairmove is weird,
even though pairmake will do the right thing in this case.


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: 881 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freeradius.org/pipermail/freeradius-devel/attachments/20140204/319bd89e/attachment.pgp>