Re: [PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-13 Thread Tim Gardner
On 02/12/2013 02:22 PM, J. Bruce Fields wrote: > On Tue, Feb 12, 2013 at 12:48:58PM -0700, Tim Gardner wrote: >> Dynamically allocate the NLM host structure in order to avoid stack overflow. >> nlmsvc_mark_resources() is several call levels deep in a stack >> that has a number of large variables.

Re: [PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-13 Thread Tim Gardner
On 02/12/2013 02:22 PM, J. Bruce Fields wrote: On Tue, Feb 12, 2013 at 12:48:58PM -0700, Tim Gardner wrote: Dynamically allocate the NLM host structure in order to avoid stack overflow. nlmsvc_mark_resources() is several call levels deep in a stack that has a number of large variables. 512

Re: [PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-12 Thread J. Bruce Fields
On Tue, Feb 12, 2013 at 12:48:58PM -0700, Tim Gardner wrote: > Dynamically allocate the NLM host structure in order to avoid stack overflow. > nlmsvc_mark_resources() is several call levels deep in a stack > that has a number of large variables. 512 bytes seems like a lot > on the stack at this

[PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-12 Thread Tim Gardner
Dynamically allocate the NLM host structure in order to avoid stack overflow. nlmsvc_mark_resources() is several call levels deep in a stack that has a number of large variables. 512 bytes seems like a lot on the stack at this point. smatch analysis: fs/lockd/svcsubs.c:366

[PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-12 Thread Tim Gardner
Dynamically allocate the NLM host structure in order to avoid stack overflow. nlmsvc_mark_resources() is several call levels deep in a stack that has a number of large variables. 512 bytes seems like a lot on the stack at this point. smatch analysis: fs/lockd/svcsubs.c:366

Re: [PATCH linux-next] lockd: nlmsvc_mark_resources(): avoid stack overflow

2013-02-12 Thread J. Bruce Fields
On Tue, Feb 12, 2013 at 12:48:58PM -0700, Tim Gardner wrote: Dynamically allocate the NLM host structure in order to avoid stack overflow. nlmsvc_mark_resources() is several call levels deep in a stack that has a number of large variables. 512 bytes seems like a lot on the stack at this point.