public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/52955] New: Missing warning on wrong sizeof usage
@ 2012-04-12 21:57 xinliangli at gmail dot com
  2012-04-13  0:09 ` [Bug c++/52955] " pinskia at gcc dot gnu.org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: xinliangli at gmail dot com @ 2012-04-12 21:57 UTC (permalink / raw)
  To: gcc-bugs

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

             Bug #: 52955
           Summary: Missing warning on wrong sizeof usage
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: xinliangli@gmail.com


Example:


 #include <memory.h>
 struct S { int a, b, c; float vec[16]; };
 void test(S *s) {
  // ...
  memset(s, 0, sizeof(s));
  // ...
 }

 /*

memset.cpp:5:23: warning: argument to 'sizeof' in 'memset' call is the
same expression as the destination; did you mean to dereference it? [-
Wsizeof-pointer-memaccess]
 memset(s, 0, sizeof(s));
        ~            ^
1 warning generated.

*/


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug c++/52955] Missing warning on wrong sizeof usage
  2012-04-12 21:57 [Bug c++/52955] New: Missing warning on wrong sizeof usage xinliangli at gmail dot com
@ 2012-04-13  0:09 ` pinskia at gcc dot gnu.org
  2012-10-11 22:33 ` paolo.carlini at oracle dot com
  2012-10-11 22:45 ` jakub at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: pinskia at gcc dot gnu.org @ 2012-04-13  0:09 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-04-13 00:09:12 UTC ---
http://gcc.gnu.org/ml/gcc-patches/2006-12/msg00158.html


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug c++/52955] Missing warning on wrong sizeof usage
  2012-04-12 21:57 [Bug c++/52955] New: Missing warning on wrong sizeof usage xinliangli at gmail dot com
  2012-04-13  0:09 ` [Bug c++/52955] " pinskia at gcc dot gnu.org
@ 2012-10-11 22:33 ` paolo.carlini at oracle dot com
  2012-10-11 22:45 ` jakub at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: paolo.carlini at oracle dot com @ 2012-10-11 22:33 UTC (permalink / raw)
  To: gcc-bugs


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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-10-11
         AssignedTo|unassigned at gcc dot       |jakub at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #2 from Paolo Carlini <paolo.carlini at oracle dot com> 2012-10-11 22:32:52 UTC ---
This is Jakub's. Close to be done!


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Bug c++/52955] Missing warning on wrong sizeof usage
  2012-04-12 21:57 [Bug c++/52955] New: Missing warning on wrong sizeof usage xinliangli at gmail dot com
  2012-04-13  0:09 ` [Bug c++/52955] " pinskia at gcc dot gnu.org
  2012-10-11 22:33 ` paolo.carlini at oracle dot com
@ 2012-10-11 22:45 ` jakub at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-10-11 22:45 UTC (permalink / raw)
  To: gcc-bugs


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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-10-11 22:44:59 UTC ---
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=190467
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192141

Another patch is still pending, but about memset (s, 0, sizeof(s)) above we
already warn.


^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-11 22:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-12 21:57 [Bug c++/52955] New: Missing warning on wrong sizeof usage xinliangli at gmail dot com
2012-04-13  0:09 ` [Bug c++/52955] " pinskia at gcc dot gnu.org
2012-10-11 22:33 ` paolo.carlini at oracle dot com
2012-10-11 22:45 ` jakub at gcc dot gnu.org

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).