Bug #6732


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

Added by Jim Pingle almost 8 years ago. Updated over 7 years ago.

Target version:
Start date:
Due date:
% Done:


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


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.

Actions #1

Updated by Anonymous almost 8 years ago

  • Status changed from New to Feedback
  • Assignee changed from Anonymous to Jim Pingle

Element names and validation code fixed to resolve issue.

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

Actions #2

Updated by Anonymous almost 8 years ago

  • % Done changed from 0 to 100
Actions #3

Updated by Jim Pingle almost 8 years ago

  • Status changed from Feedback to Resolved

Looks good now

Actions #4

Updated by Jim Pingle over 7 years ago

  • Target version changed from 2.4.0 to 2.3.3
Actions #5

Updated by Thomas Rieschl over 7 years ago

It seems that it only works partially.
I just tried to paly around with the interfaces_ppps_edit.php-Settings on version 2.4.0-BETA-amd64-20170309. The values seem to get saved but are not displayed in the input fields when opening the page.
It's the same behavior in version 2.3.3-p1 btw.

Actions #6

Updated by Phillip Davis over 7 years ago

Can you give some detail of what you are entering. I can't (easily) reproduce it, so maybe there is some unusual/specific set of inputs that are a problem.


Also available in: Atom PDF