Re: [net PATCH v2 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-14 Thread Neil Horman
On Mon, Aug 13, 2012 at 07:43:21PM -0700, John Fastabend wrote: > Add lock to prevent a race with a file closing and also remove > useless and ugly sscanf code. The extra code was never needed > and the case it supposedly protected against is in fact handled > correctly by sock_from_file as

Re: [net PATCH v2 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-14 Thread Neil Horman
On Mon, Aug 13, 2012 at 07:43:21PM -0700, John Fastabend wrote: Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed

[net PATCH v2 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread John Fastabend
Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed out by Al Viro. CC: Neil Horman Reported-by: Al Viro Signed-off-by:

[net PATCH v2 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread John Fastabend
Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed out by Al Viro. CC: Neil Horman nhor...@tuxdriver.com Reported-by: Al