pppoe-server handles offset option incorrectly

Bug #279054 reported by Arne Bernin
4
Affects Status Importance Assigned to Milestone
rp-pppoe (Ubuntu)
New
Undecided
Unassigned

Bug Description

I try to setup a pppoe-server for test purposes.
As we need multiple servers on different
interfaces, we need the offset option to
start from different offsets for the pppX interface
on different (physical) interfaces.

The pppoe-server provided by rp-pppoe has 2 options for
this: -o gives an offset to use, and -u tells pppoe-server
to pass this offset to the pppd when starting a new one.
Unfortunatly, this does not work, as the offset seems
to be added to the session created internally, but
when passing it to the pppd daemon, the session offset
is subtracted again.

I think, this is a bug, the line

1696 sprintf(buffer, "%u", (unsigned int) (ntohs(session->sess) - 1 - SessOffset));

should be

1696 sprintf(buffer, "%u", (unsigned int) (ntohs(session->sess) - 1));

after changing it, everything works as expected.

----------------------------------------------

ii pppoe 3.8-3 PPP over Ethernet driver

Revision history for this message
Arne Bernin (ab-stollmann) wrote :
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.