Thread (13 messages) flat view 13 messages, 5 authors, 2007-03-27

Re: [PATCH 1/5 2.6.21-rc4] l2tp: pppol2tp core

From: Florian Zumbiehl <hidden>
Date: 2007-03-24 18:07:50

Hi,
Florian Zumbiehl wrote:
quoted
Hi,
quoted
+ * 251003 :	Copied from pppoe.c version 0.6.9.
you might want to have a look at the patches to the PPPoE code that were
posted to netdev recently, as some of them seem to apply to code that's
left over from pppoe.c.
Do you mean this change?

 * 070228 : Fix to allow multiple sessions with same remote MAC and same
 *	    session id by including the local device ifindex in the
 *	    tuple identifying a session. This also ensures packets can't
 *          be injected into a session from interfaces other than the
 *          one	specified by userspace. Florian Zumbiehl 
 [off-list ref]

The pppoe session hash table isn't used in the pppol2tp code so the 
above change doesn't affect pppol2tp. I rechecked the latest pppoe.c and 
I don't see other changes that haven't already been incorporated in 
pppol2tp. Which specific code in pppol2tp do you think needs to be updated?
I meant those that haven't been committed yet (AFAIK):

http://www.spinics.net/lists/netdev/msg26926.html
http://www.spinics.net/lists/netdev/msg26927.html
http://www.spinics.net/lists/netdev/msg26928.html
http://www.spinics.net/lists/netdev/msg26751.html
http://www.spinics.net/lists/netdev/msg26752.html
http://www.spinics.net/lists/netdev/msg26753.html
http://www.spinics.net/lists/netdev/msg26754.html
http://www.spinics.net/lists/netdev/msg26787.html

(Plus the respective followup postings, of course.)

Some probably don't apply. I didn't have a very detailed look at your
code, but some certainly do apply.

Florian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help