Is there any interest in a -* operator that always matches and deletes all attributes from the reply list of the LHS name?

It looks to me that -= is designed to only delete LHS attributes if they match the RHS (with the exception if the LHS is ""), and that := replaces the attribute named in LHS and = just adds if the LHS is not already in the reply list.

What was not clear to me from src/main/valuepair.c pairxlatmove() was that it seems that -= will delete ALL LHS named attributes from the reply list if the RHS from the current loop matches.

If I am on track here shouldnt this be done like this:

remove the special 'NULL' clause of -=?
-= loops through the found list to find the matching RHS, seperates that out, sets ->next to NULL and sends it to pairdelete
-* Replaces NULL clause of -= and deletes all LHS named attrs from the reply list. (functions most like current -= with NULL RHS)


-= is not documented in the users.5 as shipped with the server.

Joe


- List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to