diff options
| author | Andreas Schwab <schwab@linux-m68k.org> | 2011-01-11 10:46:55 +0000 |
|---|---|---|
| committer | Andreas Schwab <schwab@linux-m68k.org> | 2011-01-11 10:46:55 +0000 |
| commit | 7a88bc9c3dd8284dcd8f4222ec3524d146810884 (patch) | |
| tree | 3519bf0677ab18bd66b97b0d79629a3bf0fd8ae3 | |
| parent | 62672b10bbd1c0797eb8ff2fd71b30360d6c373e (diff) | |
| download | gdb-7a88bc9c3dd8284dcd8f4222ec3524d146810884.zip gdb-7a88bc9c3dd8284dcd8f4222ec3524d146810884.tar.gz gdb-7a88bc9c3dd8284dcd8f4222ec3524d146810884.tar.bz2 | |
* readelf.c (print_symbol): Handle symbol characters as unsigned.
Whitespace fixes.
| -rw-r--r-- | binutils/ChangeLog | 5 | ||||
| -rw-r--r-- | binutils/readelf.c | 17 |
2 files changed, 14 insertions, 8 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 593681e..9287fda 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2011-01-11 Andreas Schwab <schwab@redhat.com> + + * readelf.c (print_symbol): Handle symbol characters as unsigned. + Whitespace fixes. + 2011-01-10 Nick Clifton <nickc@redhat.com> * po/da.po: Updated Danish translation. diff --git a/binutils/readelf.c b/binutils/readelf.c index b8ab55b..a9f271d 100644 --- a/binutils/readelf.c +++ b/binutils/readelf.c @@ -1,6 +1,6 @@ /* readelf.c -- display contents of an ELF format file Copyright 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, - 2008, 2009, 2010 + 2008, 2009, 2010, 2011 Free Software Foundation, Inc. Originally developed by Eric Youngdale <eric@andante.jic.com> @@ -380,15 +380,16 @@ print_vma (bfd_vma vma, print_mode mode) Returns the number of emitted characters. */ static unsigned int -print_symbol (int width, const char * symbol) +print_symbol (int width, const char *symbol) { - const char * c; + const char *c; bfd_boolean extra_padding = FALSE; unsigned int num_printed = 0; if (do_wide) { - /* Set the width to a very large value. This simplifies the code below. */ + /* Set the width to a very large value. This simplifies the + code below. */ width = INT_MAX; } else if (width < 0) @@ -407,7 +408,7 @@ print_symbol (int width, const char * symbol) /* Look for non-printing symbols inside the symbol's name. This test is triggered in particular by the names generated by the assembler for local labels. */ - while (ISPRINT (* c)) + while (ISPRINT (*c)) c++; len = c - symbol; @@ -423,12 +424,12 @@ print_symbol (int width, const char * symbol) num_printed += len; } - if (* c == 0 || width == 0) + if (*c == 0 || width == 0) break; /* Now display the non-printing character, if there is room left in which to dipslay it. */ - if (*c < 32) + if ((unsigned char) *c < 32) { if (width < 2) break; @@ -443,7 +444,7 @@ print_symbol (int width, const char * symbol) if (width < 6) break; - printf ("<0x%.2x>", *c); + printf ("<0x%.2x>", (unsigned char) *c); width -= 6; num_printed += 6; |