[PATCH] D28502: [compiler-rt] [Interception] Properly check for export table's size before referring to its elements.

[Marcos Pividori via Phabricator via llvm-commits]

mpividori added a comment.

@kubamracek  Thanks for your comments. Yes, I had to revert it because of the cmake code for the test. I need to review it.
As this was a bug in the interception library, I think we should include it in the interception directory, instead of asan library.
I discovered this bug when updating sanitizer coverage for windows. In https://reviews.llvm.org/D29168  , when asan dll is initialized, I analyse the exported symbols in the main executable. It was failing when the main executable doesn't export any symbol.  Then I realized there was a bug in the interception library.
As I considered it a "regression test", and I read in the documentation: http://llvm.org/docs/TestingGuide.html  that regression tests should go in the `repository/test` folder, I decided to include the test there.


Repository:
  rL LLVM

https://reviews.llvm.org/D28502