Re: pipex(4) fix: check session existence before creation

2020-04-06 Thread YASUOKA Masahiko
ok yasuoka On Mon, 6 Apr 2020 19:54:20 +0300 Vitaliy Makkoveev wrote: > Deny to create pipex_session which is already exist. Newly created > session will be placed to list head so the caller of > pipex_*_lookup_session() will receive wrong session. > > Index: sys/net/if_pppx.c >

Re: pipex(4) fix: check session existence before creation

2020-04-06 Thread Claudio Jeker
On Mon, Apr 06, 2020 at 06:32:01PM +0300, Vitaliy Makkoveev wrote: > > > > On 6 Apr 2020, at 17:37, Claudio Jeker wrote: > > > > On Mon, Apr 06, 2020 at 07:54:20PM +0300, Vitaliy Makkoveev wrote: > >> Deny to create pipex_session which is already exist. Newly created > >> session will be placed

Re: pipex(4) fix: check session existence before creation

2020-04-06 Thread Vitaliy Makkoveev
> On 6 Apr 2020, at 17:37, Claudio Jeker wrote: > > On Mon, Apr 06, 2020 at 07:54:20PM +0300, Vitaliy Makkoveev wrote: >> Deny to create pipex_session which is already exist. Newly created >> session will be placed to list head so the caller of >> pipex_*_lookup_session() will receive wrong se

Re: pipex(4) fix: check session existence before creation

2020-04-06 Thread Claudio Jeker
On Mon, Apr 06, 2020 at 07:54:20PM +0300, Vitaliy Makkoveev wrote: > Deny to create pipex_session which is already exist. Newly created > session will be placed to list head so the caller of > pipex_*_lookup_session() will receive wrong session. I think in the pppx(4) case the code is already doin

pipex(4) fix: check session existence before creation

2020-04-06 Thread Vitaliy Makkoveev
Deny to create pipex_session which is already exist. Newly created session will be placed to list head so the caller of pipex_*_lookup_session() will receive wrong session. Index: sys/net/if_pppx.c === RCS file: /cvs/src/sys/net/if_pp