From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757346AbcGZTJo (ORCPT ); Tue, 26 Jul 2016 15:09:44 -0400 Received: from mout.web.de ([212.227.15.3]:56474 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754198AbcGZTJm (ORCPT ); Tue, 26 Jul 2016 15:09:42 -0400 Subject: [PATCH 08/12] staging: lustre: Less checks after a failed ptlrpc_request_alloc() in mgc_process_recover_log() To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Greg Kroah-Hartman , Oleg Drokin References: <566ABCD9.1060404@users.sourceforge.net> <566D7733.1030102@users.sourceforge.net> <56784D83.7080108@users.sourceforge.net> <56784F0C.6040007@users.sourceforge.net> <20151221234857.GA27079@kroah.com> <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall , Bhumika Goyal From: SF Markus Elfring Message-ID: <4cb1e6d1-0fc5-212d-78fc-b2d7b2bb8d1e@users.sourceforge.net> Date: Tue, 26 Jul 2016 21:09:34 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2 MIME-Version: 1.0 In-Reply-To: <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:QbLbsSOOrOSJkCaqFprMeVJT1MSRQpP6HatA8hu2Gsr+5XAh/Oy klyUa2jrRdXzyvhERftcddiP4WTgMAUHhGX5RGJNv1POJUJLQe/78dbg+kLq7kx/A05KIcn O696CkFN88UkDmkPHPHMd+Cys5/XiOJxXrFe1gE0xPE1b4ffn4uGgzz4PRWGNQbgIQ6M4af v4wo7ojDHlcZERjAfsXRQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:4ZIP4QvLYmU=:JiJqxn004y9ShNDrSYW5pp SLzYKs4M6KlaSPjG49gG+zEOPR+yLrUhUnVAt9U67VyrOQ373qwupKUnJR8ShD1ZoogirBDE7 je4BhrLc4ksTOQs1Owy5wDLQjPj542G+e8ZnUM2RB00MmP/yrjBoQdLgy1ocfsvXbpl0K6+EK p7GJUsXdVmJvpTtdCN8LJPw+6J73CVdhNjbCej3e0SweQUByHGP0GVHooHthRfjG1ZbY2CZ++ a/+B8T3ScMW26d/t+5cBN639rc4bNS4dTrqrQY2xYmBdFGZDBAQznk6HrEkf9qxOwjfyKz8SR eVojyW5qJkR039WKR1AdOjDER/lyhBO8fxHAO7tS5tKTaiFcvkjWDiFWaiSi1ov/QhS3E22I3 62KRIvt2/KkejRWiA/GBEwjFQg6wf3VoOiR/Tz0ZJKCX/BYVKf0SeH+tOu/bmGByPYFdxV3f+ V5fC2rFbb+RdPTTnk04k/OZ8ufEIHO5fbmsZREekxYTY/WP72Du1p1SDhdsHB6kAdpNLmAMjR nLyhoJgjHRpYRjISnESKCAbm811ESfvBN2T11osM8iJJd0P8Gu8diqBu5fZcr8StTU3PTdpoB x8oL4Ta6CzYw72JjMY09leVX5xluqszYv/uQbK/YMtGGgYmJkz+0DqCuX2q36bylpi00LW4Vm yUclB92+iPWC8cfzQZAETpQwrKZ5j51sSlUXQLmDXOqXzgLatbgxm3+VXI5KV1F94/3DkYKmx W9CzSbKkNgNWrIrYSOuzzBhQefvX/hQ4YCs1sd9addSa4PjbLBQOPBry/MnjoHV9j35ZLQ+F/ RRbe/xi Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 26 Jul 2016 17:54:24 +0200 Release memory directly after a call of the function "ptlrpc_request_alloc" failed. Signed-off-by: Markus Elfring --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/mgc/mgc_request.c b/drivers/staging/lustre/lustre/mgc/mgc_request.c index b064bd3..f65bb45 100644 --- a/drivers/staging/lustre/lustre/mgc/mgc_request.c +++ b/drivers/staging/lustre/lustre/mgc/mgc_request.c @@ -1357,7 +1357,7 @@ again: &RQF_MGS_CONFIG_READ); if (!req) { rc = -ENOMEM; - goto out; + goto free_pages; } rc = ptlrpc_request_pack(req, LUSTRE_MGS_VERSION, MGS_CONFIG_READ); -- 2.9.2