aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSteve Naroff <snaroff@apple.com>2008-01-17 00:36:28 +0000
committerSteve Naroff <snaroff@apple.com>2008-01-17 00:36:28 +0000
commit1b4a6227f3ee6fca074d0f521757d1ed4f95ef38 (patch)
tree0459d9fcf7e2e31da75319423dbdbd614555f6cc
parent494783a1f900366c86895a15870a04119e3e66a6 (diff)
Declarator::clear(): Null out variable after it's been deleted.
This avoids a double free (which is good:-) Bug submitted by Eli. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@46105 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--include/clang/Parse/DeclSpec.h1
-rw-r--r--test/Sema/declspec.c3
2 files changed, 4 insertions, 0 deletions
diff --git a/include/clang/Parse/DeclSpec.h b/include/clang/Parse/DeclSpec.h
index bf7920fa2d..307959f83a 100644
--- a/include/clang/Parse/DeclSpec.h
+++ b/include/clang/Parse/DeclSpec.h
@@ -566,6 +566,7 @@ public:
}
DeclTypeInfo.clear();
delete AttrList;
+ AttrList = 0;
}
/// mayOmitIdentifier - Return true if the identifier is either optional or
diff --git a/test/Sema/declspec.c b/test/Sema/declspec.c
index d9e02ec42e..05f492d67a 100644
--- a/test/Sema/declspec.c
+++ b/test/Sema/declspec.c
@@ -3,3 +3,6 @@ typedef char T[4];
T foo(int n, int m) { } // expected-error {{cannot return array or function}}
+void foof(const char *, ...) __attribute__((__format__(__printf__, 1, 2))), barf (void);
+
+