Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp4351772ybe; Mon, 9 Sep 2019 08:07:01 -0700 (PDT) X-Google-Smtp-Source: APXvYqzPKLoDxkjX3nofg+sT9USJXLvK5tSJZzljXq2mcly9BMna/BGrOcVoC9TGluFb9i3XCq6u X-Received: by 2002:a17:906:694:: with SMTP id u20mr16432669ejb.83.1568041621097; Mon, 09 Sep 2019 08:07:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1568041621; cv=none; d=google.com; s=arc-20160816; b=akNe30P5X6tQBQjq8vrLwytKtLZGfoyhh2ML4d3hHuqGYCLoOtXu4LbadJWVVb7i2M vFtBjzwZFphbXsQFe/9W3vSQdjgnNwxelCc/W6YCjfNPjiLtxUHgKxxXxKqr30DrlJRj CIWUyk4AFj94tNWMIjm5KlB2WPJY4kTufR2KDp+czrTY6jWHefzdeqlcAYp37l8PCtoY DBTEpkCO1TAvxrdL4sS/rRmK1Pr4GV+f+osqFn3EJDq5Qw6iMxrUc1o5dQZSOqv4RrLR TxQnbixA0xqq5H6ygt0EXxJqadi9bRXP25gGy4ro07EW1DOCNICmwtmlRJAkKSfgHGk4 oGdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from; bh=QkEh6/L8DwxbtNlgyrp7arcx2GMoVRK8RibnktgBxuI=; b=PgKpTJyPs8wShCRxL7HJkFR8vJ2ajO/Jj39v8sQZKy9n9aIfA0loiMKLQWqxdnQKjR zr2/7M75j7nuzeyoFRdwe7q6Z8ItRg8wKKVUK0pcJTsiAFORO0+4p4cT36ERIlqKEvtD e93hw+OUXEHAZlvCEVr/NxboEN9x8y07Fhvvdq1ZmqNaR5j8p2Ud7kd3ouNNFRzd0Uv8 D17TnsR+IRJpHMnrxzHfADBZ9E8/UxQ9Zafa5ZImlOZt7JTaJ+/a2F45SFTAYpa0Tk4m mbSYtY5rseGc91u9RrAzc9Mbs8uKCW+tvpGCZQD5jvqjSH2j//kgNrW7ccQr8UpkONHy o7Tg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l17si9529904edd.35.2019.09.09.08.06.36; Mon, 09 Sep 2019 08:07:01 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389395AbfIIIjs (ORCPT + 99 others); Mon, 9 Sep 2019 04:39:48 -0400 Received: from cmccmta1.chinamobile.com ([221.176.66.79]:2384 "EHLO cmccmta1.chinamobile.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726587AbfIIIjs (ORCPT ); Mon, 9 Sep 2019 04:39:48 -0400 X-Greylist: delayed 551 seconds by postgrey-1.27 at vger.kernel.org; Mon, 09 Sep 2019 04:39:47 EDT Received: from spf.mail.chinamobile.com (unknown[172.16.121.19]) by rmmx-syy-dmz-app02-12002 (RichMail) with SMTP id 2ee25d760d96739-c508a; Mon, 09 Sep 2019 16:30:14 +0800 (CST) X-RM-TRANSID: 2ee25d760d96739-c508a X-RM-TagInfo: emlType=0 X-RM-SPAM-FLAG: 00000000 Received: from localhost.localdomain (unknown[223.105.0.243]) by rmsmtp-syy-appsvr10-12010 (RichMail) with SMTP id 2eea5d760d93853-071fb; Mon, 09 Sep 2019 16:30:13 +0800 (CST) X-RM-TRANSID: 2eea5d760d93853-071fb From: Ding Xiang To: miklos@szeredi.hu, linux-unionfs@vger.kernel.org Cc: linux-kernel@vger.kernel.org Subject: [PATCH] ovl: Fix dereferencing possible ERR_PTR() Date: Mon, 9 Sep 2019 16:29:56 +0800 Message-Id: <1568017796-27513-1-git-send-email-dingxiang@cmss.chinamobile.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org if ovl_encode_real_fh() fails, no memory was allocated and the error in the error-valued pointer should be returned. Fixes: 9b6faee0747 ("ovl: check ERR_PTR() return value from ovl_encode_fh()") Signed-off-by: Ding Xiang --- fs/overlayfs/export.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/overlayfs/export.c b/fs/overlayfs/export.c index cb8ec1f..50ade19 100644 --- a/fs/overlayfs/export.c +++ b/fs/overlayfs/export.c @@ -229,7 +229,7 @@ static int ovl_d_to_fh(struct dentry *dentry, char *buf, int buflen) ovl_dentry_upper(dentry), !enc_lower); err = PTR_ERR(fh); if (IS_ERR(fh)) - goto fail; + return err; err = -EOVERFLOW; if (fh->len > buflen) -- 1.9.1