mirrors/binutils-gdb
0
0
Fork 0
mirror of https://sourceware.org/git/binutils-gdb.git synced 2024-11-30 21:44:19 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix compile time warning messages about variables being used before they are initialised.

Browse Source 
PR 18313
bin	* ieee.c (ieee_read_cxx_class): Initialise the varargs variable.
	* readelf.c (uncompress_section_contents): Zero initialise the
	zstream structure.

bfd	* compress.c (decompress_contents): Zero initialse the z_stream
	structure.
This commit is contained in:
Nick Clifton 2015-04-24 17:13:22 +01:00
parent 7a5c933c7c
commit de13ef81f0
5 changed files with 26 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
bfd/ChangeLog
View File

@ -1,3 +1,9 @@
2015-04-24 Nick Clifton <nickc@redhat.com>
PR 18313
* compress.c (decompress_contents): Zero initialse the z_stream
structure.
2015-04-24 Nick Clifton <nickc@redhat.com> 2015-04-24 Nick Clifton <nickc@redhat.com>
* elf.c (_bfd_elf_is_local_label_name): Extend test for assembler * elf.c (_bfd_elf_is_local_label_name): Extend test for assembler

9
bfd/compress.c
View File

@ -37,9 +37,12 @@ decompress_contents (bfd_byte *compressed_buffer,
/* It is possible the section consists of several compressed /* It is possible the section consists of several compressed
buffers concatenated together, so we uncompress in a loop. */ buffers concatenated together, so we uncompress in a loop. */
strm.zalloc = NULL; /* PR 18313: The state field in the z_stream structure is supposed
strm.zfree = NULL; to be invisible to the user (ie us), but some compilers will
strm.opaque = NULL; still complain about it being used without initialisation. So
we first zero the entire z_stream structure and then set the fields
that we need. */
memset (& strm, 0, sizeof strm);
strm.avail_in = compressed_size - 12; strm.avail_in = compressed_size - 12;
strm.next_in = (Bytef*) compressed_buffer + 12; strm.next_in = (Bytef*) compressed_buffer + 12;
strm.avail_out = uncompressed_size; strm.avail_out = uncompressed_size;

7
binutils/ChangeLog
View File

@ -1,3 +1,10 @@
2015-04-24 Nick Clifton <nickc@redhat.com>
PR 18313
* ieee.c (ieee_read_cxx_class): Initialise the varargs variable.
* readelf.c (uncompress_section_contents): Zero initialise the
zstream structure.
2015-04-23 H.J. Lu <hongjiu.lu@intel.com> 2015-04-23 H.J. Lu <hongjiu.lu@intel.com>
PR binutils/18209 PR binutils/18209

2
binutils/ieee.c
View File

@ -2954,7 +2954,7 @@ ieee_read_cxx_class (struct ieee_info *info, const bfd_byte **pp,
{ {
debug_type return_type; debug_type return_type;
const debug_type *arg_types; const debug_type *arg_types;
bfd_boolean varargs; bfd_boolean varargs = FALSE;
if (debug_get_type_kind (dhandle, pv->type) if (debug_get_type_kind (dhandle, pv->type)
!= DEBUG_KIND_FUNCTION) != DEBUG_KIND_FUNCTION)

9
binutils/readelf.c
View File

@ -12067,9 +12067,12 @@ uncompress_section_contents (unsigned char **buffer,
/* It is possible the section consists of several compressed /* It is possible the section consists of several compressed
buffers concatenated together, so we uncompress in a loop. */ buffers concatenated together, so we uncompress in a loop. */
strm.zalloc = NULL; /* PR 18313: The state field in the z_stream structure is supposed
strm.zfree = NULL; to be invisible to the user (ie us), but some compilers will
strm.opaque = NULL; still complain about it being used without initialisation. So
we first zero the entire z_stream structure and then set the fields
that we need. */
memset (& strm, 0, sizeof strm);
strm.avail_in = compressed_size - header_size; strm.avail_in = compressed_size - header_size;
strm.next_in = (Bytef *) compressed_buffer + header_size; strm.next_in = (Bytef *) compressed_buffer + header_size;
strm.avail_out = uncompressed_size; strm.avail_out = uncompressed_size;