Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965340AbcJ2C0n (ORCPT ); Fri, 28 Oct 2016 22:26:43 -0400 Received: from mail-pf0-f195.google.com ([209.85.192.195]:33619 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752313AbcJ2C0m (ORCPT ); Fri, 28 Oct 2016 22:26:42 -0400 From: Brian Norris X-Google-Original-From: Brian Norris To: Andy Whitcroft , Joe Perches Cc: , Brian Norris , Linus Torvalds , Andrew Morton , SF Markus Elfring Subject: [PATCH] checkpatch: fix uninitialized var when run with --no-tree Date: Fri, 28 Oct 2016 19:26:31 -0700 Message-Id: <1477707991-140274-1-git-send-email-briannorris@chromium.org> X-Mailer: git-send-email 2.8.0.rc3.226.g39d4020 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1056 Lines: 34 From: Brian Norris If checkpatch.pl gets copied out of the tree, --no-tree shouldn't start complaining: Use of uninitialized value $root in concatenation (.) or string at /path/to/checkpatch.pl line 764. Let's just give the safe answer instead -- don't warn about "obsolete" files. Fixes: 85b0ee18bbf8 ("checkpatch: see if modified files are marked obsolete in MAINTAINERS") Signed-off-by: Brian Norris --- This is a 4.9-rc1 regression scripts/checkpatch.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index a8368d1c4348..c8cd643dbc6f 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -761,6 +761,8 @@ sub seed_camelcase_file { sub is_maintained_obsolete { my ($filename) = @_; + return 0 if (!$tree); + return 0 if (!(-e "$root/scripts/get_maintainer.pl")); my $status = `perl $root/scripts/get_maintainer.pl --status --nom --nol --nogit --nogit-fallback -f $filename 2>&1`; -- 2.8.0.rc3.226.g39d4020