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

[Florian Zumbiehl <florz@gmx.de>]

Hi,

> Florian Zumbiehl wrote:
> >Hi,
> >
> >>+ * 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 
>  <florz@gmx.de>
> 
> 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