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 
>  <[EMAIL PROTECTED]>
> 
> 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
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to