tree 9d1192939a882831a4407eaef3b35b8c5f598229
parent d1db58eb0dec6bee91cfd4244eb5fe58a869bfc1
author Jianzhou Zhao <jianzhouzh@google.com> 1611873407 +0000
committer Copybara-Service <copybara-worker@google.com> 1612516814 -0800

[dfsan] Clean TLS after sigaction callbacks

DFSan uses TLS to pass metadata of arguments and return values. When an
instrumented function accesses the TLS, if a signal callback happens, and
the callback calls other instrumented functions with updating the same TLS,
the TLS is in an inconsistent state after the callback ends. This may cause
either under-tainting or over-tainting.

This fix follows MSan's workaround.
  https://github.com/llvm/llvm-project/commit/cb22c67a21e4b5e1ade65141117a70be318be072
It simply resets TLS at restore. This prevents from over-tainting. Although
under-tainting may still happen, a taint flow can be found eventually if we
run a DFSan-instrumented program multiple times. The alternative option is
saving the entire TLS. However the TLS storage takes 2k bytes, and signal calls
could be nested. So it does not seem worth.

This diff fixes sigaction. A following diff will be fixing signal.

Reviewed-by: morehouse

Differential Revision: https://reviews.llvm.org/D95642

GitOrigin-RevId: e1a4322f8136788228d915a7384c5679b39dfeed