From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from one.firstfloor.org ([193.170.194.197]:44324 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753599Ab3JWNHy (ORCPT ); Wed, 23 Oct 2013 09:07:54 -0400 Date: Wed, 23 Oct 2013 15:07:53 +0200 From: Andi Kleen Subject: [PATCH 2/3] Kbuild: Handle longer symbols in kallsyms.c v2 Message-ID: <20131023130753.GO29695@two.firstfloor.org> References: <1382456783-22826-1-git-send-email-andi@firstfloor.org> <1382456783-22826-3-git-send-email-andi@firstfloor.org> <1382459794.2041.91.camel@joe-AO722> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1382459794.2041.91.camel@joe-AO722> Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Joe Perches Cc: Andi Kleen , mmarek@suse.cz, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, Andi Kleen Also warn for too long symbols v2: Add missing newline. Use 255 max (Joe Perches) Signed-off-by: Andi Kleen --- scripts/kallsyms.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c index 6940f00..c4230a4 100644 --- a/scripts/kallsyms.c +++ b/scripts/kallsyms.c @@ -27,7 +27,7 @@ #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof(arr[0])) #endif -#define KSYM_NAME_LEN 128 +#define KSYM_NAME_LEN 255 struct sym_entry { unsigned long long addr; @@ -118,6 +118,12 @@ static int read_symbol(FILE *in, struct sym_entry *s) fprintf(stderr, "Read error or end of file.\n"); return -1; } + if (strlen(str) > KSYM_NAME_LEN) { + fprintf(stderr, "Symbol %s too long for kallsyms (%lu vs %d).\n" + "Please increase KSYM_NAME_LEN both in kernel and kallsyms.c\n", + str, strlen(str), KSYM_NAME_LEN); + return -1; + } sym = str; /* skip prefix char */ -- 1.8.4 -- ak@linux.intel.com -- Speaking for myself only.