Project

General

Profile

Bug #6732

interfaces_ppps_edit.php: L2TP and PPTP WAN-type interface editing has broken input validation

Added by Jim Pingle 6 months ago. Updated 9 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Interfaces
Target version:
Start date:
08/19/2016
Due date:
% Done:

100%

Affected version:
2.3.2
Affected Architecture:

Description

On interfaces_ppps_edit.php when an L2TP or PPTP interface is created or edited, it cannot be saved.

Both types have an expanding list of "IP/Gateway" inputs for each selected interface, but input validation is not checking the arrays properly and it is rejecting any input. When the input validation is removed, the values are not stored in the config either.

Going the other way seems to work, starting at interfaces.php and then making an L2TP interface there, it works. If you edit that working entry at interfaces_ppps_edit.php it shows the values entered on the other page, but on save they are lost, as above.

Associated revisions

Revision 9db8959d
Added by Steve Beaver 6 months ago

Fixed #6732

(cherry picked from commit e73770057872aa1f78dfff885f89916e446e2e4c)

History

#1 Updated by Steve Beaver 6 months ago

  • Status changed from New to Feedback
  • Assignee changed from Steve Beaver to Jim Pingle

Element names and validation code fixed to resolve issue.

localip, submnt, gateway, mtu, mru & mrru tested

#2 Updated by Steve Beaver 6 months ago

  • % Done changed from 0 to 100

#3 Updated by Jim Pingle 6 months ago

  • Status changed from Feedback to Resolved

Looks good now

#4 Updated by Jim Pingle 9 days ago

  • Target version changed from 2.4.0 to 2.3.3

Also available in: Atom PDF