[PATCH] ubsan TLC + PR59306 fix

Jakub Jelinek jakub@redhat.com
Wed Nov 27 12:32:00 GMT 2013


On Wed, Nov 27, 2013 at 11:29:21AM +0100, Marek Polacek wrote:
> This was meant only as a TLC, but it also fixes PR59306.  Using
> walk_gimple_op was an overkill; gimple_{store,assign_load}_p is
> enough.  As a side effect, it also fixes the bug because now we
> better restrict what goes into instrument_member_call.
> 
> Bootstrapped, ran ubsan testsuite on x86_64-linux, ok for trunk?

Ok, thanks.

> 2013-11-27  Marek Polacek  <polacek@redhat.com>
> 
> 	PR sanitizer/59306
> 	* ubsan.c (instrument_null): Use gimple_store_p/gimple_assign_load_p
> 	instead of walk_gimple_op.
> 	(ubsan_pass): Adjust.  Call instrument_null only if SANITIZE_NULL.
> testsuite/
> 	* g++.dg/ubsan/pr59306.C:

Missing " New test." ;)

	Jakub



More information about the Gcc-patches mailing list