From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcus =?utf-8?B?SMO8d2U=?= Date: Tue, 28 Jul 2020 11:06:35 +0000 Subject: Re: [PATCH] keys: Do not check the retval of commit_creds in join_session_keyring Message-Id: <20200728110635.rlvzizbsrcddhzj2@linux> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit List-Id: References: <044b34c4388405e995e38fb57fc120361bf962db.1568124150.git.suse-tux@gmx.de> In-Reply-To: <044b34c4388405e995e38fb57fc120361bf962db.1568124150.git.suse-tux@gmx.de> To: keyrings@vger.kernel.org On 2020-06-17 04:46:24 +0300, Jarkko Sakkinen wrote: > On Fri, Jun 12, 2020 at 01:25:33PM +0200, Marcus Huewe wrote: > > Do not check the return value of the commit_creds call in > > join_session_keyring in order to make the code more concise. > > Since commit_creds always returns 0, the if-statement is not needed > > and the session keyring's serial can be directly assigned to the > > "ret" variable. > > > > Signed-off-by: Marcus Huewe > > Reviewed-by: Jarkko Sakkinen > > David, can you pick this up to your tree (give that a generic > keyring change)? > Just a gentle ping - any news on this?:) (I'm perfectly fine if you don't want to take it... (since it is just a minor cleanup and no bugfix)) Marcus