]> gcc.gnu.org Git - gcc.git/commitdiff
Add missing store in emission of asan_stack_free.
authorMartin Liska <mliska@suse.cz>
Tue, 19 May 2020 14:57:56 +0000 (16:57 +0200)
committerMartin Liska <mliska@suse.cz>
Fri, 12 Jun 2020 08:04:07 +0000 (10:04 +0200)
gcc/ChangeLog:

2020-05-19  Martin Liska  <mliska@suse.cz>

PR sanitizer/94910
* asan.c (asan_emit_stack_protection): Emit
also **SavedFlagPtr(FakeStack, class_id) = 0 in order to release
a stack frame.

(cherry picked from commit 8b6731e674c76cb48a417f2eef74ced92a17f469)

gcc/asan.c

index 3b800b26b6991c5a79116a7de838751b04db496a..fdbdb6ae15acc204bf23398d78681799dacc7e49 100644 (file)
@@ -1597,8 +1597,24 @@ asan_emit_stack_protection (rtx base, rtx pbase, unsigned int alignb,
       if (use_after_return_class < 5
          && can_store_by_pieces (sz, builtin_memset_read_str, &c,
                                  BITS_PER_UNIT, true))
-       store_by_pieces (shadow_mem, sz, builtin_memset_read_str, &c,
-                        BITS_PER_UNIT, true, RETURN_BEGIN);
+       {
+         /* Emit:
+              memset(ShadowBase, kAsanStackAfterReturnMagic, ShadowSize);
+              **SavedFlagPtr(FakeStack, class_id) = 0
+         */
+         store_by_pieces (shadow_mem, sz, builtin_memset_read_str, &c,
+                          BITS_PER_UNIT, true, RETURN_BEGIN);
+
+         unsigned HOST_WIDE_INT offset
+           = (1 << (use_after_return_class + 6));
+         offset -= GET_MODE_SIZE (ptr_mode);
+         mem = gen_rtx_MEM (ptr_mode, base);
+         mem = adjust_address (mem, ptr_mode, offset);
+         rtx addr = gen_reg_rtx (ptr_mode);
+         emit_move_insn (addr, mem);
+         mem = gen_rtx_MEM (QImode, addr);
+         emit_move_insn (mem, const0_rtx);
+       }
       else if (use_after_return_class >= 5
               || !set_storage_via_setmem (shadow_mem,
                                           GEN_INT (sz),
This page took 0.058549 seconds and 5 git commands to generate.