diff options
| author | Nick Clifton <nickc@redhat.com> | 2015-04-24 17:13:22 +0100 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2015-04-24 17:13:22 +0100 |
| commit | de13ef81f041f7f51687ef1873c74e853b97e73a (patch) | |
| tree | ac4a0f49bc8e11516977363bba28319767a5fd31 /bfd | |
| parent | 7a5c933c7c2c3926b13a047c64f2af1d3ce8e1db (diff) | |
| download | fsf-binutils-gdb-de13ef81f041f7f51687ef1873c74e853b97e73a.zip fsf-binutils-gdb-de13ef81f041f7f51687ef1873c74e853b97e73a.tar.gz fsf-binutils-gdb-de13ef81f041f7f51687ef1873c74e853b97e73a.tar.bz2 | |
Fix compile time warning messages about variables being used before they are initialised.
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.
Diffstat (limited to 'bfd')
| -rw-r--r-- | bfd/ChangeLog | 6 | ||||
| -rw-r--r-- | bfd/compress.c | 9 |
2 files changed, 12 insertions, 3 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 11a74de..87e22c2 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,5 +1,11 @@ 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> + * elf.c (_bfd_elf_is_local_label_name): Extend test for assembler local labels to include local labels with a numeric suffix and fake symbols. diff --git a/bfd/compress.c b/bfd/compress.c index d0f745f..07289e5 100644 --- a/bfd/compress.c +++ b/bfd/compress.c @@ -37,9 +37,12 @@ decompress_contents (bfd_byte *compressed_buffer, /* It is possible the section consists of several compressed buffers concatenated together, so we uncompress in a loop. */ - strm.zalloc = NULL; - strm.zfree = NULL; - strm.opaque = NULL; + /* PR 18313: The state field in the z_stream structure is supposed + to be invisible to the user (ie us), but some compilers will + 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.next_in = (Bytef*) compressed_buffer + 12; strm.avail_out = uncompressed_size; |