Merge pull request #63 from monikasoni/master

Fix mem leak when failure is set
graphql · May 16, 2018 · 9d55a3c · 9d55a3c
2 parents 28808fd + f17c26c
commit 9d55a3c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/GraphQLParser.cpp b/GraphQLParser.cpp
@@ -19,9 +19,12 @@ namespace graphql {
 // Given properly-configured yylex, run the parser and return the
 // result.
 static std::unique_ptr<ast::Node> doParse(const char **outError, yyscan_t scanner, bool enableSchema) {
-  Node *outAST;
+  Node *outAST = NULL;
   yy::GraphQLParserImpl parser(enableSchema, &outAST, outError, scanner);
   int failure = parser.parse();
+  if (failure) {
+    delete outAST;
+  }
   return !failure ? std::unique_ptr<ast::Node>(outAST) : nullptr;
 }