From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753047AbbL2Bx7 (ORCPT ); Mon, 28 Dec 2015 20:53:59 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:43925 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752854AbbL2Bxz (ORCPT ); Mon, 28 Dec 2015 20:53:55 -0500 X-AuditID: cbfee61a-f79266d000003652-32-5681e7b15fe4 From: Chao Yu To: "'Jaegeuk Kim'" Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net References: <1451345497-65784-1-git-send-email-jaegeuk@kernel.org> In-reply-to: <1451345497-65784-1-git-send-email-jaegeuk@kernel.org> Subject: RE: [f2fs-dev] [PATCH 1/3] f2fs: early check broken symlink length in the encrypted case Date: Tue, 29 Dec 2015 09:52:49 +0800 Message-id: <00a701d141db$c2f7d480$48e77d80$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-transfer-encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-index: AQF8sL4YKOhQKJAKtMQkmbtvwkRIL5+KVWKg Content-language: zh-cn X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrOLMWRmVeSWpSXmKPExsVy+t9jAd2NzxvDDHYvk7J4sn4Ws8WlRe4W e/aeZLG4vGsOmwOLx6ZVnWweuxd8ZvL4vEkugDmKyyYlNSezLLVI3y6BK+PLm5eMBU8EK7p3 rGNsYFzL18XIwSEhYCKx+YtYFyMnkCkmceHeerYuRi4OIYGljBLTj6xmBkkICbxilPh+TBHE ZhNQkVje8Z8JxBYRUJPo3TcFzGYWyJSY0P+CHaLeSeJy939WEJtTwFli+8YnjCC2sECyxO63 r9lAbBYBVYlHP7vB5vMKWErcP/SKCcIWlPgx+R4LxEwtifU7j0PNl5fYvOYtM8ShChI7zr5m hLjBSOLi62eMEDXiEhuP3GKZwCg0C8moWUhGzUIyahaSlgWMLKsYJVILkguKk9JzDfNSy/WK E3OLS/PS9ZLzczcxgoP/mdQOxoO73A8xCnAwKvHwZkxqDBNiTSwrrsw9xCjBwawkwuu6BSjE m5JYWZValB9fVJqTWnyIUZqDRUmct/ZSZJiQQHpiSWp2ampBahFMlomDU6qBcZ+3YViQytkN P1dO/HT0vXyTfYdX06k4yZ/CXy7Kr2E/sfbkBAXJVp/4jUsbdA9kLNVI8WvzrM1rv8E1f+lN m0yZV/4LXrBMez037VgEj8zXKdEiyRPrl/b3BIje5D5lfUFgyv/rnk+i36tkdoknX2fwXf74 Tm3x7II5txO/H9h1uEBjU/pEHyWW4oxEQy3mouJEACavnrV6AgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jaegeuk, > -----Original Message----- > From: Jaegeuk Kim [mailto:jaegeuk@kernel.org] > Sent: Tuesday, December 29, 2015 7:32 AM > To: linux-kernel@vger.kernel.org; linux-fsdevel@vger.kernel.org; > linux-f2fs-devel@lists.sourceforge.net > Cc: Jaegeuk Kim > Subject: [f2fs-dev] [PATCH 1/3] f2fs: early check broken symlink length in the encrypted case > > If link is broken, its len is zero, and we don't need to move forward. > > Signed-off-by: Jaegeuk Kim > --- > fs/f2fs/namei.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c > index fb41c80..5cc4128 100644 > --- a/fs/f2fs/namei.c > +++ b/fs/f2fs/namei.c > @@ -931,7 +931,7 @@ static const char *f2fs_encrypted_follow_link(struct dentry *dentry, void > **cook > { > struct page *cpage = NULL; > char *caddr, *paddr = NULL; > - struct f2fs_str cstr; > + struct f2fs_str cstr = FSTR_INIT(NULL, 0); > struct f2fs_str pstr = FSTR_INIT(NULL, 0); > struct inode *inode = d_inode(dentry); > struct f2fs_encrypted_symlink_data *sd; > @@ -952,6 +952,12 @@ static const char *f2fs_encrypted_follow_link(struct dentry *dentry, void > **cook > /* Symlink is encrypted */ > sd = (struct f2fs_encrypted_symlink_data *)caddr; > cstr.len = le16_to_cpu(sd->len); > + > + /* this is broken symlink case */ > + if (cstr.len == 0) { This case seems rare, can we add unlikely here? > + res = -ENOENT; > + goto errout; > + } > cstr.name = kmalloc(cstr.len, GFP_NOFS); > if (!cstr.name) { > res = -ENOMEM; > @@ -960,7 +966,7 @@ static const char *f2fs_encrypted_follow_link(struct dentry *dentry, void > **cook > memcpy(cstr.name, sd->encrypted_path, cstr.len); > > /* this is broken symlink case */ > - if (cstr.name[0] == 0 && cstr.len == 0) { > + if (cstr.name[0] == 0) { ditto. Thanks, > res = -ENOENT; > goto errout; > } > -- > 2.5.4 (Apple Git-61) > > > ------------------------------------------------------------------------------ > _______________________________________________ > Linux-f2fs-devel mailing list > Linux-f2fs-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel