From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Howells Date: Fri, 20 Mar 2020 08:20:19 +0000 Subject: Re: [PATCH v5 1/2] KEYS: Don't write out to userspace while holding key semaphore Message-Id: <3251035.1584692419@warthog.procyon.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit List-Id: References: <20200318221457.1330-2-longman@redhat.com> <20200318221457.1330-1-longman@redhat.com> In-Reply-To: <20200318221457.1330-2-longman@redhat.com> To: Waiman Long Cc: dhowells@redhat.com, Jarkko Sakkinen , James Morris , "Serge E. Hallyn" , Mimi Zohar , "David S. Miller" , Jakub Kicinski , keyrings@vger.kernel.org, linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org, linux-integrity@vger.kernel.org, netdev@vger.kernel.org, linux-afs@lists.infradead.org, Sumit Garg , Jerry Snitselaar , Roberto Sassu , Eric Biggers , Chris von Recklinghausen Waiman Long wrote: > + if ((ret > 0) && (ret <= buflen)) { That's a bit excessive on the bracketage, btw, but don't worry about it unless you respin the patches. David