Received: by 2002:a5d:9c59:0:0:0:0:0 with SMTP id 25csp3878iof; Wed, 8 Jun 2022 13:48:50 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxXADUNaSPPMBeV4U8xjfELAU16V9ddN2pe4J5hM1Ta793yhe3nCtuopYiDVDj/5aiDbTFo X-Received: by 2002:a17:902:d4c2:b0:166:46c9:577b with SMTP id o2-20020a170902d4c200b0016646c9577bmr32175090plg.66.1654721329746; Wed, 08 Jun 2022 13:48:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1654721329; cv=none; d=google.com; s=arc-20160816; b=pHm305HtDRmvdZjskRv2FgH5VDjTtwAtp4ftC7yrLsjsLQ2CkLHJk2GaVCG5j8FY9e 26PiYrv0qNfKGdiiBWPAOcfVM3o+s/rs8TBalalkgSCrM9emGriNJSV+WKqmYJuNe824 XRpKKZBSdTUoxEvFv6kZtekQJqMvlk1lTitEt4dkRvhooaNuKHQuGkIk4OfajNLflxAn X0AFKyECB1ZGePci4g/ngUw/GnWV9tBPK+rNWqNQLi4wxhhP/Yv6pNshm8V7lZ0IwU04 k2Djth5iwyp9alr4HVpsnAUQoftRGZyRzvVIsP3cvTv3ugKUtMD9LxeEpU3uVr6GvIRK IMGQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:reply-to :in-reply-to:references:mime-version:dkim-signature; bh=AOjdpRzw4kjXPBWg+IgjE0lEriWtzNXqibpX4C2uBGc=; b=ncyyf62f5q9dKkglJvD9/LoP6GfOjdUbxn1SCO2PWCamthMOr5qgZzhZuM+6vg/g4L faJYJ93KgdcGfa/gt90Zv8Es5otcOkfBRQhBjSMhdVOy2Y6XPB5bADVEwivh4NfpALK1 lOqON18d6H2kIRTeddH5cTXG19eu3Z9tsdCaNz4JmspmI5ESL2aVX5Du2yDcug6brlej 1JWXERxn6nB+geYliX+ikRBOia1xa+wHRLq40HmpiyECiBCy+2b16tPfi6fSQ80Hh1hC KoqHThwVkYdJKWEDohoYPKl/14Ak8bURWr+Mzr99bdF0KGCWpGbVclhBiyHPkxeLoMKs 4WdA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=jAHJfid2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h6-20020a636c06000000b003fde9094ba3si4346823pgc.757.2022.06.08.13.48.35; Wed, 08 Jun 2022 13:48:49 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=jAHJfid2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232833AbiFHTaY (ORCPT + 99 others); Wed, 8 Jun 2022 15:30:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39998 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229830AbiFHTaW (ORCPT ); Wed, 8 Jun 2022 15:30:22 -0400 Received: from mail-il1-x135.google.com (mail-il1-x135.google.com [IPv6:2607:f8b0:4864:20::135]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8F8791E734A; Wed, 8 Jun 2022 12:30:21 -0700 (PDT) Received: by mail-il1-x135.google.com with SMTP id a15so17279567ilq.12; Wed, 08 Jun 2022 12:30:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:reply-to:from:date:message-id :subject:to:cc; bh=AOjdpRzw4kjXPBWg+IgjE0lEriWtzNXqibpX4C2uBGc=; b=jAHJfid2MSi3OT1OIrtN7Gv3Z7jqBsRN41Sb5eYx0Qjy5mikfQtYnmbGDS4uiVXgW7 zHNMxIlB11TP9sMUU5jW7e5OuPn1/lBV0Tm2C+amVjedIj1dvPGXRLHZ61sGNkqTiEVN TiiQOwyIc8G0sRKr/Fik/5BCd0m/7McOMzOV7vilEJr85Dc2R4KJfQ00f+YN85b0flW7 MtX3WMntOfil+plLEXbCxOZnvKfrdAmnLZY250xVt971dFPC+GtCLuVg9TfjZGbJxnnz aPjCRVpFAOpYWXuUQNXOknXyi+1/YPh4qnS/Lpwsv42WL7UrBNUL3L6UhzFmk+qPpmux j5ZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc; bh=AOjdpRzw4kjXPBWg+IgjE0lEriWtzNXqibpX4C2uBGc=; b=2BAMnOUA4gFJgxoTfqITiKGoLzCrrV6c05WUO1aWfWnFm2HH3Cj6NprVOdFRMtGjAp nXsAM2uOKn2Nz7uqmMD0NyHbj/SEP+CFdVxDL5M810QCBHJxzb38pP9+w8aw64iSIPzr cKeqr8LningBh5nmbrq42Bzj9qUXaJ8P4BjFgxERmsq6TcYFv31z7A1IIOMkGvoD1w/V 6+PR4UoOg7aJiBh6iyZ+/nKOLL2A2Ol9wX1ifV+MbOMlr1o+N6zry0gaseR8jBC6P75z aUO5ZZvb83cUYVW1GB+OHk5bt6ZIdHZi1lgpgkwE1R1slbZEL5e0XK94FaIj0MVoB4nu 4ZJA== X-Gm-Message-State: AOAM531w2yKr4Q1Mtl+NIm+mvlsh3J8bO2KIQUrwPSAt9LKKXXzWDkNh vBmwzi25ciK+IWoUnBVdBjrBm3WVex8hGHmodUg= X-Received: by 2002:a05:6e02:16ce:b0:2d3:edec:17a5 with SMTP id 14-20020a056e0216ce00b002d3edec17a5mr18678433ilx.4.1654716620920; Wed, 08 Jun 2022 12:30:20 -0700 (PDT) MIME-Version: 1.0 References: <20220607000851.39798-1-irogers@google.com> In-Reply-To: Reply-To: sedat.dilek@gmail.com From: Sedat Dilek Date: Wed, 8 Jun 2022 21:29:44 +0200 Message-ID: Subject: Re: [PATCH] perf unwind: Fix uninitialized variable To: Ian Rogers Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, Fangrui Song , Sebastian Ullrich Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 7, 2022 at 8:12 PM Sedat Dilek wrote: > > On Tue, Jun 7, 2022 at 2:09 AM Ian Rogers wrote: > > > > ret may be uninitialized on error goto paths. > > > > Reported-by: Sedat Dilek > > Fixes: dc2cf4ca866f ("perf unwind: Fix segbase for ld.lld linked objects") > > Signed-off-by: Ian Rogers > > Thanks for the patch. > > Feel free to add my: > > Tested-by: Sedat Dilek > If it is not too late, more exactly: Tested-by: Sedat Dilek # LLVM-14 (x86-64) -Sedat- > $ LC_ALL=C ll ~/bin/perf > -rwxr-xr-x 2 dileks dileks 16M Jun 7 20:10 /home/dileks/bin/perf > > $ ~/bin/perf -vv > perf version 5.19.0-rc1 > dwarf: [ on ] # HAVE_DWARF_SUPPORT > dwarf_getlocations: [ on ] # HAVE_DWARF_GETLOCATIONS_SUPPORT > glibc: [ on ] # HAVE_GLIBC_SUPPORT > syscall_table: [ on ] # HAVE_SYSCALL_TABLE_SUPPORT > libbfd: [ on ] # HAVE_LIBBFD_SUPPORT > debuginfod: [ OFF ] # HAVE_DEBUGINFOD_SUPPORT > libelf: [ on ] # HAVE_LIBELF_SUPPORT > libnuma: [ on ] # HAVE_LIBNUMA_SUPPORT > numa_num_possible_cpus: [ on ] # HAVE_LIBNUMA_SUPPORT > libperl: [ on ] # HAVE_LIBPERL_SUPPORT > libpython: [ on ] # HAVE_LIBPYTHON_SUPPORT > libslang: [ on ] # HAVE_SLANG_SUPPORT > libcrypto: [ on ] # HAVE_LIBCRYPTO_SUPPORT > libunwind: [ on ] # HAVE_LIBUNWIND_SUPPORT > libdw-dwarf-unwind: [ on ] # HAVE_DWARF_SUPPORT > zlib: [ on ] # HAVE_ZLIB_SUPPORT > lzma: [ on ] # HAVE_LZMA_SUPPORT > get_cpuid: [ on ] # HAVE_AUXTRACE_SUPPORT > bpf: [ on ] # HAVE_LIBBPF_SUPPORT > aio: [ on ] # HAVE_AIO_SUPPORT > zstd: [ on ] # HAVE_ZSTD_SUPPORT > libpfm4: [ OFF ] # HAVE_LIBPFM > > -Sedat- > > > --- > > tools/perf/util/unwind-libunwind-local.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tools/perf/util/unwind-libunwind-local.c b/tools/perf/util/unwind-libunwind-local.c > > index 37622699c91a..6e5b8cce47bf 100644 > > --- a/tools/perf/util/unwind-libunwind-local.c > > +++ b/tools/perf/util/unwind-libunwind-local.c > > @@ -174,7 +174,7 @@ static int elf_section_address_and_offset(int fd, const char *name, u64 *address > > Elf *elf; > > GElf_Ehdr ehdr; > > GElf_Shdr shdr; > > - int ret; > > + int ret = -1; > > > > elf = elf_begin(fd, PERF_ELF_C_READ_MMAP, NULL); > > if (elf == NULL) > > -- > > 2.36.1.255.ge46751e96f-goog > >