Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

radclient reports success where the filter is empty except for Response-Packet-Type, even if the response packet from FreeRADIUS contains attributes #3086

Open
arr2036 opened this issue Oct 30, 2019 · 1 comment
Labels
defect category: a defect or misbehaviour v3.x.x meta: relates to the v3.0.x branch v4.0.x meta: relates to the v4.0.x branch

Comments

@arr2036
Copy link
Member

arr2036 commented Oct 30, 2019

Issue type

.

  • Defect - Unexpected behaviour (obvious or verified by project member).

Defect

How to reproduce the issue

Create a radclient filter containing only Response-Packet-Type == Access-Accept.

Pass it to radclient with -f <input_file>:<filter_file>.

Configure FreeRADIUS to include a Reply-Message attribute in its response.

Observe how the response packet passes the filter even though there's no line accounting for the Reply-Message.

This could just be a documentation issue, and we could allow all attributes by default, and only perform matching on the ones specified.

Unsure whether this is truly in v4.0.x. It was originally observed in v3.0.x HEAD

@arr2036 arr2036 added defect category: a defect or misbehaviour v3.x.x meta: relates to the v3.0.x branch v4.0.x meta: relates to the v4.0.x branch labels Oct 30, 2019
@alandekok
Copy link
Member

It's likely that the filter is matching things, and there's no default saying "anything which isn't in the filter is not a match"

I think it works that way by intention, even if it is surprising here. i.e. filtering packets in radsniff, even when the packets contain other attributes which aren't in the filter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect category: a defect or misbehaviour v3.x.x meta: relates to the v3.0.x branch v4.0.x meta: relates to the v4.0.x branch
Projects
None yet
Development

No branches or pull requests

2 participants