openharmony/third_party_elfutils
1
0
Fork 0
mirror of https://github.com/openharmony/third_party_elfutils.git synced 2026-07-15 07:28:18 -04:00
Code Issues Packages Projects Releases Wiki Activity

libasm: Fix one GCC7 -Wformat-truncation=2 warning.

Browse Source 
Make sure that if we have really lots of labels the tempsym doesn't get
truncated because it is too small to hold the whole name.

This doesn't enable -Wformat-truncation=2 or fix other "issues" pointed
out by enabling this warning because there are currently some issues
with it. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79448

Signed-off-by: Mark Wielaard <mark@klomp.org>
This commit is contained in:
Mark Wielaard
2017-02-12 21:51:34 +01:00
parent b1042ce216
commit 93c51144c3
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
libasm/ChangeLog
+5 -1
View File
@@ -1,8 +1,12 @@
2017-02-12 Mark Wielaard <mjw@redhat.com>
* asm_newsym.c (asm_newsym): Increase TEMPSYMLEN to 13.
2017-02-15 Ulf Hermann <ulf.hermann@qt.io>
* disasm_str.c: Include system.h.
2015-10-11 Akihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>
2016-10-11 Akihiko Odaki <akihiko.odaki.4i@stu.hosei.ac.jp>
* asm_align.c: Remove sys/param.h include.
libasm/asm_newsym.c
+4 -2
View File
@@ -1,5 +1,5 @@
/* Define new symbol for current position in given section.
Copyright (C) 2002, 2005, 2016 Red Hat, Inc.
Copyright (C) 2002, 2005, 2016, 2017 Red Hat, Inc.
This file is part of elfutils.
Written by Ulrich Drepper <drepper@redhat.com>, 2002.
@@ -44,7 +44,9 @@ AsmSym_t *
asm_newsym (AsmScn_t *asmscn, const char *name, GElf_Xword size,
int type, int binding)
{
#define TEMPSYMLEN 10
/* We don't really expect labels with many digits, but in theory it could
be 10 digits (plus ".L" and a zero terminator). */
#define TEMPSYMLEN 13
char tempsym[TEMPSYMLEN];
AsmSym_t *result;