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/ -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
