On Thu, 2016-09-29 at 13:26 +0200, Vegard Nossum wrote: > On 08/29/2016 02:21 AM, Michael Kerrisk (man-pages) wrote: > > This is a preparatory patch for following work. Move the F_SETPIPE_SZ > > limit-checking logic from pipe_fcntl() into pipe_set_size(). This > > simplifies the code a little, and allows for reworking required in > > a later patch that fixes the limit checking in pipe_set_size() > > > + if ((too_many_pipe_buffers_hard(pipe->user) || > > + too_many_pipe_buffers_soft(pipe->user)) && > > + capable(CAP_SYS_RESOURCE) && !capable(CAP_SYS_ADMIN)) > > + return -EPERM; > Some might say the indentation is off here. Not sure why checkpatch > didn't care.
Parenthesis alignment is a CHECK not a WARNING. It's only reported with --strict or if the code is in drivers/staging, net/, or drivers/net/