Modify

Opened 8 years ago

Closed 6 years ago

Last modified 4 years ago

#6892 closed enhancement (fixed)

pppd is not adding default route if one default route exist

Reported by: David Schueler <david.schueler@…> Owned by: developers
Priority: normal Milestone: Barrier Breaker 14.07
Component: packages Version:
Keywords: ppp pppd kamikaze add default route Cc:

Description

If a default route exists, pppd is not adding a second default route.
This is okay until the default route has another metric.

I'm using a UMTS stick for dial in if the line over Ethernet (ADSL in my case) breaks. But i'm running a ping over the ethernet line and if it comes back up the ppp connection gets disconnected.
So i changed the sys_linux.c file in ppp package a bit, so that ppp adds the route regardless of an existing default route.

BUT: This should be improved. There should be a check if the existing default route has another metric, and the route should only be added if the metric is diferent.

I'm running KAMIKAZE 8.09.2 r18801 with pppd 2.4.3

I've added a patch.

Regards.

David

Attachments (4)

400-add-default-route.patch (622 bytes) - added by David Schueler <david.schueler@…> 8 years ago.
patch for packages/ppp/patches/
defaultroute-metric.patch (4.3 KB) - added by Olivier Mehani <olivier.mehani@…> 8 years ago.
Check the existing default route metric before adding.
defaultroute-metric.2.patch (4.4 KB) - added by Olivier Mehani <olivier.mehani@…> 8 years ago.
This ones applies and compiles cleanly…
defaultroute-metric.3.patch (4.7 KB) - added by Olivier Mehani <olivier.mehani@…> 8 years ago.
Things always happen in threes. This patch also removes the route cleanly.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by David Schueler <david.schueler@…>

patch for packages/ppp/patches/

Changed 8 years ago by Olivier Mehani <olivier.mehani@…>

Check the existing default route metric before adding.

comment:1 Changed 8 years ago by Olivier Mehani <olivier.mehani@…>

I quickly hacked this patch, until I realised it was incompatible with another patch in the patchset I was building over. It is therefore untested, but I thought it may help.

Changed 8 years ago by Olivier Mehani <olivier.mehani@…>

This ones applies and compiles cleanly...

Changed 8 years ago by Olivier Mehani <olivier.mehani@…>

Things always happen in threes. This patch also removes the route cleanly.

comment:2 Changed 6 years ago by nbd

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

the metric option now works with netifd when configuring it in the config interface section.

comment:3 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.