Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp4080256yba; Wed, 17 Apr 2019 04:16:49 -0700 (PDT) X-Google-Smtp-Source: APXvYqzH2jfh1lc3le/dXmwS8MrVj5vNmi+HUk2eeeJMfG4JNss3oOUX1ouViwoYF0+VJ5ju9xmU X-Received: by 2002:a63:2158:: with SMTP id s24mr35867839pgm.156.1555499809339; Wed, 17 Apr 2019 04:16:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1555499809; cv=none; d=google.com; s=arc-20160816; b=MDEwDDrmNXrHdBin0VelaQ1e0YpzwKkQCuWspXn3twhyom4KnnQc6cE+jwhz9egRzA WpX9Is6z5CTEERlWhZXZzhb+h+VnRaDVR0Ig87mbJTn/Izw1vuAI1RDXYC/1DZEGVYdD bapd1Z9xmrtif+164MAFzs75QSYG/0l9kBXBZ5OQwl58//fu4QBC79UL63FbR0LzHI6j irlZ0MUdzgqVdyYtNe9/oZh9g/WVeiSxSZT7hvPQPy3MEk2dRqSWkztOqM7arm3ayozh W82c2ykRRFzlFWLMMXFRAAolmoJde+Aay9w6lQzT9fShqiJmyOjL3fiBaU/gPuiVynjE oOsw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=p4lSDLKtMrDY/1mG9JUzZ2kNUs4KK58UcwWIxddUIsU=; b=g1j9DZmqGQGDCuqHhFMhghsd7qbcs08rDuLHcoKZFo3L6w2KxLCRgnAxGrtlBv3uZt QLrD3ULPrwmvcbs6Hxg03tC7ExUxrXKVn+IhBFVNe2Aom1T7D5pCS4m9K7/Is6Ptck8h htqEqDbo9V1lKR7FZG9cpHER4vrQnjNYgtUjNQv/MWDs6QMseL5NGNRbOxq1u20wvuLd /boYNZdfSh+qDlGwCGJR+DoSFABhZYhkgyABsbC69v6RLTbWDWm/LR4zMR/pSpa833AX 6E2/Fm0yEWmULvaPkBRH2Kuq6eDQG9M9TuNd0FN7Lo3HO9WPQDkCXJhOIc7Ox1uSPo22 yE0w== 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 n72si42137301pfi.0.2019.04.17.04.16.34; Wed, 17 Apr 2019 04:16:49 -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 S1731847AbfDQLOL (ORCPT + 99 others); Wed, 17 Apr 2019 07:14:11 -0400 Received: from mx2.suse.de ([195.135.220.15]:37706 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729522AbfDQLOL (ORCPT ); Wed, 17 Apr 2019 07:14:11 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 3EBC8AF01; Wed, 17 Apr 2019 11:14:10 +0000 (UTC) Received: by quack2.suse.cz (Postfix, from userid 1000) id 504F41E15AE; Wed, 17 Apr 2019 13:14:08 +0200 (CEST) Date: Wed, 17 Apr 2019 13:14:08 +0200 From: Jan Kara To: Wenwen Wang Cc: Jan Kara , open list Subject: Re: [PATCH v2] udf: fix an uninitialized read bug and remove dead code Message-ID: <20190417111408.GB26435@quack2.suse.cz> References: <1555351086-3935-1-git-send-email-wang6495@umn.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1555351086-3935-1-git-send-email-wang6495@umn.edu> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon 15-04-19 12:58:06, Wenwen Wang wrote: > In udf_lookup(), the pointer 'fi' is a local variable initialized by the > return value of the function call udf_find_entry(). However, if the macro > 'UDF_RECOVERY' is defined, this variable will become uninitialized if the > else branch is not taken, which can potentially cause incorrect results in > the following execution. > > To fix this issue, this patch drops the whole code in the ifdef > 'UDF_RECOVERY' region, as it is dead code. > > Signed-off-by: Wenwen Wang Thanks for the patch. I've added it to my tree. Honza > --- > fs/udf/namei.c | 15 --------------- > 1 file changed, 15 deletions(-) > > diff --git a/fs/udf/namei.c b/fs/udf/namei.c > index 58cc241..77b6d89 100644 > --- a/fs/udf/namei.c > +++ b/fs/udf/namei.c > @@ -304,21 +304,6 @@ static struct dentry *udf_lookup(struct inode *dir, struct dentry *dentry, > if (dentry->d_name.len > UDF_NAME_LEN) > return ERR_PTR(-ENAMETOOLONG); > > -#ifdef UDF_RECOVERY > - /* temporary shorthand for specifying files by inode number */ > - if (!strncmp(dentry->d_name.name, ".B=", 3)) { > - struct kernel_lb_addr lb = { > - .logicalBlockNum = 0, > - .partitionReferenceNum = > - simple_strtoul(dentry->d_name.name + 3, > - NULL, 0), > - }; > - inode = udf_iget(dir->i_sb, lb); > - if (IS_ERR(inode)) > - return inode; > - } else > -#endif /* UDF_RECOVERY */ > - > fi = udf_find_entry(dir, &dentry->d_name, &fibh, &cfi); > if (IS_ERR(fi)) > return ERR_CAST(fi); > -- > 2.7.4 > > -- Jan Kara SUSE Labs, CR