aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFariborz Jahanian <fjahanian@apple.com>2008-01-02 18:09:46 +0000
committerFariborz Jahanian <fjahanian@apple.com>2008-01-02 18:09:46 +0000
commit0ba0aa1460ae234423609c09da1b6ad11fae24f3 (patch)
treed82fd04036c1a0201bd005f104e3af8732efdf12
parent5d5c462e416ef29ad2005daf7d39a82c6e4c61bf (diff)
Prevent crash on incorrect objc messaging expression.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@45489 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--Parse/ParseObjc.cpp4
-rw-r--r--test/Parser/objc-messaging-neg-1.m7
2 files changed, 9 insertions, 2 deletions
diff --git a/Parse/ParseObjc.cpp b/Parse/ParseObjc.cpp
index d8da0daea9..19bbfea624 100644
--- a/Parse/ParseObjc.cpp
+++ b/Parse/ParseObjc.cpp
@@ -1342,13 +1342,13 @@ Parser::ExprResult Parser::ParseObjCMessageExpression() {
} else if (!selIdent) {
Diag(Tok, diag::err_expected_ident); // missing selector name.
SkipUntil(tok::semi);
- return 0;
+ return true;
}
if (Tok.isNot(tok::r_square)) {
Diag(Tok, diag::err_expected_rsquare);
SkipUntil(tok::semi);
- return 0;
+ return true;
}
SourceLocation RBracloc = ConsumeBracket(); // consume ']'
diff --git a/test/Parser/objc-messaging-neg-1.m b/test/Parser/objc-messaging-neg-1.m
new file mode 100644
index 0000000000..23db7e28c4
--- /dev/null
+++ b/test/Parser/objc-messaging-neg-1.m
@@ -0,0 +1,7 @@
+// RUN: clang -fsyntax-only -verify %s
+
+int main()
+ {
+ id a;
+ [a bla:0 6:7]; // expected-error {{expected ']'}}
+ }