diff mbox

Fix uninitialized pp->flags

Message ID 20130304210835.GV12913@tucnak.redhat.com
State New
Headers show

Commit Message

Jakub Jelinek March 4, 2013, 9:08 p.m. UTC 
Hi!

This is something that caused a lot of testsuite failures during my
--enable-checking=yes,valgrind bootstrap.  Nothing was initializing
pp->flags.  Fixed thusly, bootstrapped/regtested on x86_64-linux and
i686-linux, ok for trunk?

2013-03-04  Jakub Jelinek  <jakub@redhat.com>

	* c-pretty-print.c (pp_c_pretty_printer_init): Clear pp->flags.


	Jakub

Comments

Jeff Law March 4, 2013, 9:27 p.m. UTC | #1 
On 03/04/2013 02:08 PM, Jakub Jelinek wrote:
> Hi!
>
> This is something that caused a lot of testsuite failures during my
> --enable-checking=yes,valgrind bootstrap.  Nothing was initializing
> pp->flags.  Fixed thusly, bootstrapped/regtested on x86_64-linux and
> i686-linux, ok for trunk?
>
> 2013-03-04  Jakub Jelinek  <jakub@redhat.com>
>
> 	* c-pretty-print.c (pp_c_pretty_printer_init): Clear pp->flags.
OK.
jeff
diff mbox

Patch

--- gcc/c-family/c-pretty-print.c.jj	2013-02-13 17:05:53.000000000 +0100
+++ gcc/c-family/c-pretty-print.c	2013-03-04 16:34:00.989736325 +0100
@@ -2311,6 +2311,8 @@  pp_c_pretty_printer_init (c_pretty_print
 {
   pp->offset_list               = 0;
 
+  pp->flags			= 0;
+
   pp->declaration               = pp_c_declaration;
   pp->declaration_specifiers    = pp_c_declaration_specifiers;
   pp->declarator                = pp_c_declarator;