diff options
| author | Bill Wendling <isanbard@gmail.com> | 2009-10-23 00:01:05 +0000 |
|---|---|---|
| committer | Bill Wendling <isanbard@gmail.com> | 2009-10-23 00:01:05 +0000 |
| commit | dfd85c142f184ba3ce5562f0eed32a908c59c076 (patch) | |
| tree | b4ba529697e967f13d8a999ba94f488aa2cbe86a /lib/CodeGen/StackProtector.cpp | |
| parent | 4c3715c2e5e17d7216a96ac2baf9720630f04408 (diff) | |
| download | llvm-dfd85c142f184ba3ce5562f0eed32a908c59c076.tar.gz llvm-dfd85c142f184ba3ce5562f0eed32a908c59c076.tar.bz2 llvm-dfd85c142f184ba3ce5562f0eed32a908c59c076.tar.xz | |
Neuter stack protectors by only checking character arrays. This is what GCC
does.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@84916 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/StackProtector.cpp')
| -rw-r--r-- | lib/CodeGen/StackProtector.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/CodeGen/StackProtector.cpp b/lib/CodeGen/StackProtector.cpp index 350bc6e1ad..0204969333 100644 --- a/lib/CodeGen/StackProtector.cpp +++ b/lib/CodeGen/StackProtector.cpp @@ -111,11 +111,16 @@ bool StackProtector::RequiresStackProtector() const { // protectors. return true; - if (const ArrayType *AT = dyn_cast<ArrayType>(AI->getAllocatedType())) + if (const ArrayType *AT = dyn_cast<ArrayType>(AI->getAllocatedType())) { + // We apparently only care about character arrays. + if (AT->getElementType() != Type::getInt8Ty(AT->getContext())) + continue; + // If an array has more than SSPBufferSize bytes of allocated space, // then we emit stack protectors. if (SSPBufferSize <= TD->getTypeAllocSize(AT)) return true; + } } } |