From patchwork Mon Jul 13 13:49:50 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Abhishek Kulkarni X-Patchwork-Id: 29733 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@bilbo.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from ozlabs.org (ozlabs.org [203.10.76.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mx.ozlabs.org", Issuer "CA Cert Signing Authority" (verified OK)) by bilbo.ozlabs.org (Postfix) with ESMTPS id 115B3B7066 for ; Mon, 13 Jul 2009 23:51:52 +1000 (EST) Received: by ozlabs.org (Postfix) id 05945DDDF6; Mon, 13 Jul 2009 23:51:52 +1000 (EST) Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id 878AFDDDF4 for ; Mon, 13 Jul 2009 23:51:51 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755915AbZGMNuR (ORCPT ); Mon, 13 Jul 2009 09:50:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755862AbZGMNuQ (ORCPT ); Mon, 13 Jul 2009 09:50:16 -0400 Received: from rv-out-0506.google.com ([209.85.198.230]:39641 "EHLO rv-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755815AbZGMNuP (ORCPT ); Mon, 13 Jul 2009 09:50:15 -0400 Received: by rv-out-0506.google.com with SMTP id f6so638578rvb.1 for ; Mon, 13 Jul 2009 06:50:15 -0700 (PDT) Received: by 10.140.127.13 with SMTP id z13mr2690017rvc.235.1247493015124; Mon, 13 Jul 2009 06:50:15 -0700 (PDT) Received: from localhost.localdomain (c-68-35-150-144.hsd1.nm.comcast.net [68.35.150.144]) by mx.google.com with ESMTPS id g31sm23849436rvb.0.2009.07.13.06.50.12 (version=TLSv1/SSLv3 cipher=RC4-MD5); Mon, 13 Jul 2009 06:50:14 -0700 (PDT) From: Abhishek Kulkarni To: linux-kernel@vger.kernel.org Cc: ericvh@gmail.com, Abhishek Kulkarni , v9fs-developer@lists.sourceforge.net, netdev@vger.kernel.org Subject: [PATCH] [net/9p] Possible regression in p9_client_stat Date: Mon, 13 Jul 2009 07:49:50 -0600 Message-Id: <1247492991-26009-1-git-send-email-adkulkar@umail.iu.edu> X-Mailer: git-send-email 1.6.0.4 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Fix a possible regression with p9_client_stat where it can try to kfree an ERR_PTR after an erroneous p9pdu_readf. Also remove an unnecessary data buffer increment in p9_client_read. Signed-off-by: Abhishek Kulkarni --- :100644 100644 dd43a82... cc25e63... M net/9p/client.c net/9p/client.c | 7 ++----- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/net/9p/client.c b/net/9p/client.c index dd43a82..cc25e63 100644 --- a/net/9p/client.c +++ b/net/9p/client.c @@ -1098,7 +1098,6 @@ p9_client_read(struct p9_fid *fid, char *data, char __user *udata, u64 offset, if (data) { memmove(data, dataptr, count); - data += count; } if (udata) { @@ -1192,9 +1191,9 @@ struct p9_wstat *p9_client_stat(struct p9_fid *fid) err = p9pdu_readf(req->rc, clnt->dotu, "wS", &ignored, ret); if (err) { - ret = ERR_PTR(err); p9pdu_dump(1, req->rc); - goto free_and_error; + p9_free_req(clnt, req); + goto error; } P9_DPRINTK(P9_DEBUG_9P, @@ -1211,8 +1210,6 @@ struct p9_wstat *p9_client_stat(struct p9_fid *fid) p9_free_req(clnt, req); return ret; -free_and_error: - p9_free_req(clnt, req); error: kfree(ret); return ERR_PTR(err);