public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug lto/47221] New: Bugus warnings issued during lto bootstrap
@ 2011-01-08  0:24 hubicka at gcc dot gnu.org
  2011-01-11 11:43 ` [Bug lto/47221] " rguenth at gcc dot gnu.org
  2011-01-11 13:04 ` hubicka at ucw dot cz
  0 siblings, 2 replies; 3+ messages in thread
From: hubicka at gcc dot gnu.org @ 2011-01-08  0:24 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47221

           Summary: Bugus warnings issued during lto bootstrap
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: lto
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: hubicka@gcc.gnu.org


../../zlib/inflate.c:577:24: warning: 'MEM[(struct z_stream_s
*)&d_stream].state' may be used uninitialized in this function
[-Wuninitialized]
../../gcc/java/jcf-io.c:204:16: note: 'MEM[(struct z_stream_s
*)&d_stream].state' was declared here


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug lto/47221] Bugus warnings issued during lto bootstrap
  2011-01-08  0:24 [Bug lto/47221] New: Bugus warnings issued during lto bootstrap hubicka at gcc dot gnu.org
@ 2011-01-11 11:43 ` rguenth at gcc dot gnu.org
  2011-01-11 13:04 ` hubicka at ucw dot cz
  1 sibling, 0 replies; 3+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-01-11 11:43 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47221

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-01-11 11:40:08 UTC ---
I don't think the warning is bogus.

      z_stream d_stream; /* decompression stream */
      d_stream.zalloc = (alloc_func) 0;
      d_stream.zfree = (free_func) 0;
...
      /* Handle NO_HEADER using undocumented zlib feature.
         This is a very common hack.  */
      inflateInit2 (&d_stream, -MAX_WBITS);
      inflate (&d_stream, Z_NO_FLUSH);

we don't check for an error of inflateInit2 in which case state is
not initialized.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug lto/47221] Bugus warnings issued during lto bootstrap
  2011-01-08  0:24 [Bug lto/47221] New: Bugus warnings issued during lto bootstrap hubicka at gcc dot gnu.org
  2011-01-11 11:43 ` [Bug lto/47221] " rguenth at gcc dot gnu.org
@ 2011-01-11 13:04 ` hubicka at ucw dot cz
  1 sibling, 0 replies; 3+ messages in thread
From: hubicka at ucw dot cz @ 2011-01-11 13:04 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47221

--- Comment #2 from Jan Hubicka <hubicka at ucw dot cz> 2011-01-11 13:03:00 UTC ---
> we don't check for an error of inflateInit2 in which case state is
> not initialized.
Probably we should fix that, but still I don't think we should leak our well
engineered
MEM_REF gimple syntax into user warnings...

Honza


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-01-11 13:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-08  0:24 [Bug lto/47221] New: Bugus warnings issued during lto bootstrap hubicka at gcc dot gnu.org
2011-01-11 11:43 ` [Bug lto/47221] " rguenth at gcc dot gnu.org
2011-01-11 13:04 ` hubicka at ucw dot cz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).