From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757821AbbIVJMb (ORCPT ); Tue, 22 Sep 2015 05:12:31 -0400 Received: from mail-pa0-f43.google.com ([209.85.220.43]:34304 "EHLO mail-pa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753196AbbIVJM1 (ORCPT ); Tue, 22 Sep 2015 05:12:27 -0400 From: Taeung Song To: Arnaldo Carvalho de Melo Cc: linux-kernel@vger.kernel.org, jolsa@redhat.com, namhyung@kernel.org, Ingo Molnar , Taeung Song Subject: [PATCH] perf report: fix owner error when reading perf.data Date: Tue, 22 Sep 2015 18:12:19 +0900 Message-Id: <1442913139-1135-1-git-send-email-treeze.taeung@gmail.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If perf.data file is owned by some user, it can't be read even if current user is root. So, add a condition (if root) to skip the owner error handling if statement. Signed-off-by: Taeung Song --- tools/perf/util/data.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/util/data.c b/tools/perf/util/data.c index 1921942..20632a5 100644 --- a/tools/perf/util/data.c +++ b/tools/perf/util/data.c @@ -67,7 +67,7 @@ static int open_file_read(struct perf_data_file *file) if (fstat(fd, &st) < 0) goto out_close; - if (!file->force && st.st_uid && (st.st_uid != geteuid())) { + if (!file->force && geteuid() && st.st_uid && (st.st_uid != geteuid())) { pr_err("File %s not owned by current user or root (use -f to override)\n", file->path); goto out_close; -- 1.9.1