commit 75288afe42db31072d5ea32912b708b87d35ecce
author Greg Kaiser <gkaiser@google.com> Tue Sep 07 09:29:58 2021 -0700
committer Treehugger Robot <treehugger-gerrit@google.com> Wed Sep 08 07:47:37 2021 +0000
tree 96eef6fba44d42574dc6ca100629cca3eec05133
parent ea6944589f1e7812e50d111638d33b4c7d07e294

Initialize root_ member by default

In case of a future coding error where SetRoot() doesn't get
called prior to GetRoot(), we at least assure consistent behavior
and avoid using uninitialized memory as a pointer.

Test: TreeHugger
Bug: 197828948
Change-Id: I858a54a706283fb0f81ad0c9f60d1adfa1e89eca

permission/parser.h

diff --git a/permission/parser.h b/permission/parser.h
index 0de9952..49da2a3 100644
--- a/permission/parser.h
+++ b/permission/parser.h
@@ -95,7 +95,7 @@
 
   void* scanner_ = nullptr;
   YY_BUFFER_STATE buffer_;
-  Expression* root_;
+  Expression* root_ = nullptr;
   std::vector<std::string> errors_;
 };