Project

General

Profile

Actions

Bug #8441

closed

Manually disconnecting a captive portal user leaves the IPFW table entry

Added by Jim Pingle over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Category:
Captive Portal
Target version:
Start date:
04/06/2018
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.4.3
Affected Architecture:
All

Description

When a captive portal user is manually disconnected their entry is removed from the portal database, but the entry remains in the ipfw table (Check ipfw table <name>_auth_up list)

Some more debugging from the forum:
https://forum.pfsense.org/index.php?topic=146046.msg795216#msg795216

(pfsense 2.4.3-RELEASE (amd64))
When a user press the disconnect button, in /usr/local/captiveportal/index.php (line 151) calls the captiveportal_disconnect_client function. This function first removes the information about the session from the captive portal database (file /etc/inc/captiveportal.inc line 1098), then the function captiveportal_disconnect (line 1104) is called. In captiveportal_disconnect, before removing an ip from the ipfw tables (lines 1038-1041), it is checked (lines 1035, 1036) whether this ip is logged. Because the information about the session is removed from the database, it bypass lines 1037-1041. As a result of this disconnection from the Internet does not occur.
I commented the lines 1035, 1036, 1042. The Captiva portal began to disconnect users.

I was able to reproduce the problem by logging into the portal and then disconnecting the user from Status > Captive Portal

Actions #1

Updated by Anonymous over 6 years ago

  • Status changed from Confirmed to Feedback
  • % Done changed from 0 to 100
Actions #2

Updated by Chris Macmahon over 6 years ago

tested on today's snap: 2.4.4.a.20180420.1016
removed item from the captive portal status page.
was removed from ipfw table.

Actions #3

Updated by Jim Pingle over 6 years ago

  • Status changed from Feedback to Resolved
Actions #4

Updated by Jim Pingle over 6 years ago

  • Target version changed from 2.4.4 to 2.4.3-p1
Actions

Also available in: Atom PDF