Automatic report from sources (radiusd) between 13.09.2006 - 14.09.2006 GMT
CVS log entries from 13.09.2006 (Wed) 08:00:00 - 14.09.2006 (Thu) 08:00:00 GMT ===================================================== Summary by authors ===================================================== Author: aland File: radiusd/src/main/modules.c; Revisions: 1.109 Author: bjordanov File: radiusd/src/modules/rlm_perl/rlm_perl.c; Revisions: 1.39 Author: pnixon File: radiusd/raddb/sqlippool.conf; Revisions: 1.3 ===================================================== Log entries ===================================================== Description: Major cleanup of the queries (To match what we are running in production) Modified files: File: radiusd/raddb/sqlippool.conf; Revision: 1.3; Date: 2006/09/13 12:49:37; Author: pnixon; Lines: (+40 -26) ------------------------------- Description: When walking over a section, handle the case of CONF_DATA existing Modified files: File: radiusd/src/main/modules.c; Revision: 1.109; Date: 2006/09/13 18:55:23; Author: aland; Lines: (+5 -3) ------------------------------- Description: Fixed bug id #390 Removed pairfree and Updated to use pairmove. Modified files: File: radiusd/src/modules/rlm_perl/rlm_perl.c; Revision: 1.39; Date: 2006/09/13 08:36:53; Author: bjordanov; Lines: (+7 -12) ===================================================== Summary of modified files ===================================================== File: radiusd/raddb/sqlippool.conf Revisions: 1.3 Authors: pnixon (+40 -26) ------------------------------- File: radiusd/src/main/modules.c Revisions: 1.109 Authors: aland (+5 -3) ------------------------------- File: radiusd/src/modules/rlm_perl/rlm_perl.c Revisions: 1.39 Authors: bjordanov (+7 -12) -- Automatic cron job from /web/pages/us.freeradius.org/bin/new_makelog.pl
Fixed bug id #390 Removed pairfree and Updated to use pairmove.
No, this is wrong. "pairmove" uses the operators to determine which attributes to move, and which to leave behind. Since Perl can update all of the attributes via it's own logic, it does not need the operators. The operators are ONLY for the "users" file, which is a hack. In addition, pairmove() MIGHT leave VALUE_PAIR's in the source list. The change you made didn't free them, so there is a memory leak in the code. I'm going to revert this change, and commit a different fix for bug #390. The "Operator" code added in revision 1.38 is not needed. Again, the *only* purpose of the operator in the VALUE_PAIR is a hack, to make the "users" file work. Since the Perl script can make it's own decisions about which attributes to over-write and why, it's completey unnecessary to support operators here. All of the code dealing with "Operator" in rlm_perl.c should be deleted. The attributes used in rlm_perl should just be names and values, without operators. I'll be updating the module to remove the "Operator" stuff. It's confusing, and not necessary. Alan DeKok. -- http://deployingradius.com - The web site of the book http://deployingradius.com/blog/ - The blog
On Thursday 14 September 2006 20:46, Alan DeKok wrote:
Fixed bug id #390 Removed pairfree and Updated to use pairmove.
No, this is wrong. "pairmove" uses the operators to determine which attributes to move, and which to leave behind. Since Perl can update all of the attributes via it's own logic, it does not need the operators. The operators are ONLY for the "users" file, which is a hack.
In addition, pairmove() MIGHT leave VALUE_PAIR's in the source list. The change you made didn't free them, so there is a memory leak in the code.
I'm going to revert this change, and commit a different fix for bug #390.
The "Operator" code added in revision 1.38 is not needed. Again, the *only* purpose of the operator in the VALUE_PAIR is a hack, to make the "users" file work. Since the Perl script can make it's own decisions about which attributes to over-write and why, it's completey unnecessary to support operators here. All of the code dealing with "Operator" in rlm_perl.c should be deleted.
The attributes used in rlm_perl should just be names and values, without operators. I'll be updating the module to remove the "Operator" stuff. It's confusing, and not necessary.
Ok I am agree with this. -- Best Regards, Boian Jordanov SNE Orbitel - Next Generation Telecom tel. +359 2 4004 723 tel. +359 2 4004 002
participants (3)
-
Alan DeKok -
Automatic cvs log generator -
Boian Jordanov