Modify

Opened 7 years ago

Closed 7 years ago

Last modified 4 years ago

#9275 closed defect (invalid)

PPP interface is added twice to firewall

Reported by: Dmitry Tunin <hanipouspilot@…> Owned by: developers
Priority: normal Milestone: Barrier Breaker 14.07
Component: packages Version: Trunk
Keywords: Cc:

Description

Apr 20 15:25:16 OpenWrt user.notice ifup: Enabling Router Solicitations on vpn (ppp0)
Apr 20 15:25:16 OpenWrt user.notice ifup: Enabling Router Solicitations on vpn (ppp0)
Apr 20 15:25:16 OpenWrt user.info firewall: removing vpn (ppp0) from zone wan
Apr 20 15:25:16 OpenWrt user.info firewall: removing vpn (ppp0) from zone wan
Apr 20 15:25:17 OpenWrt daemon.info dnsmasq-dhcp[1697]: DHCPINFORM(br-lan) 192.168.1.175 00:13:e8:78:33:8b 
Apr 20 15:25:17 OpenWrt daemon.info dnsmasq-dhcp[1697]: DHCPACK(br-lan) 192.168.1.175 00:13:e8:78:33:8b Natasha
Apr 20 15:25:18 OpenWrt user.info firewall: adding vpn (ppp0) to zone wan
Apr 20 15:25:18 OpenWrt user.info firewall: adding vpn (ppp0) to zone wan

It results in two copies of rules in iptables.

Attachments (0)

Change History (7)

comment:1 Changed 7 years ago by Dmitry Tunin <hanipouspilot@…>

This happens because ip-ip script does ifup again.
I fixed this by commenting out

[ -z "$PPP_IPPARAM" ] || env -i ACTION="ifup" INTERFACE="$PPP_IPPARAM" DEVICE="$PPP_IFACE" PROTO=ppp /sbin/hotplug-call "iface"

In Backfire it seems that interface was not up by uci.

comment:2 Changed 7 years ago by jow

  • Resolution set to fixed
  • Status changed from new to closed

Should be properly fixed with r26742.

comment:3 Changed 7 years ago by Dmitry Tunin <hanipouspilot@…>

It makes sense to change in ip-down too to avoid error.

comment:4 Changed 7 years ago by jow

You're right, see r26743

comment:5 Changed 7 years ago by Dmitry Tunin <hanipouspilot@…>

  • Resolution fixed deleted
  • Status changed from closed to reopened

And also add not only ppp-* but ppp*.
xl2tp and openl2tp use ppp0 interface.

comment:6 Changed 7 years ago by jow

  • Resolution set to invalid
  • Status changed from reopened to closed

You got it wrong, see See r25354. That change and the two ones form here are complements to each other.

comment:7 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.