]> git.karo-electronics.de Git - linux-beck.git/commit
pppoe: Use workqueue to die properly when a PADT is received
authorSimon Farnsworth <simon@farnz.org.uk>
Sun, 1 Mar 2015 10:54:39 +0000 (10:54 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 2 Mar 2015 05:17:31 +0000 (00:17 -0500)
commit287f3a943fef58c5c73e42545169443be379222f
treeaefaf8eb2c42acc7091a76a2f152dc8a3f62b3b1
parent26caa3469a2de617d95c29371571a9585603a023
pppoe: Use workqueue to die properly when a PADT is received

When a PADT frame is received, the socket may not be in a good state to
close down the PPP interface. The current implementation handles this by
simply blocking all further PPP traffic, and hoping that the lack of traffic
will trigger the user to investigate.

Use schedule_work to get to a process context from which we clear down the
PPP interface, in a fashion analogous to hangup on a TTY-based PPP
interface. This causes pppd to disconnect immediately, and allows tools to
take immediate corrective action.

Note that pppd's rp_pppoe.so plugin has code in it to disable the session
when it disconnects; however, as a consequence of this patch, the session is
already disabled before rp_pppoe.so is asked to disable the session. The
result is a harmless error message:

Failed to disconnect PPPoE socket: 114 Operation already in progress

This message is safe to ignore, as long as the error is 114 Operation
already in progress; in that specific case, it means that the PPPoE session
has already been disabled before pppd tried to disable it.

Signed-off-by: Simon Farnsworth <simon@farnz.org.uk>
Tested-by: Dan Williams <dcbw@redhat.com>
Tested-by: Christoph Schulz <develop@kristov.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ppp/pppoe.c
include/linux/if_pppox.h