[apparmor] [PATCH 1/2] Fix the dfa-graph dump

John Johansen john.johansen at canonical.com
Thu May 19 22:32:07 UTC 2011


The dfa graph dump was broken by previous dfa cleanups so that the graph
transition target is the output of a pointer instead of the dfa state
number.

Signed-off-by: John Johansen <john.johansen at canonical.com>
---
 parser/libapparmor_re/hfa.cc |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/parser/libapparmor_re/hfa.cc b/parser/libapparmor_re/hfa.cc
index 6bc3c27..40bf6e1 100644
--- a/parser/libapparmor_re/hfa.cc
+++ b/parser/libapparmor_re/hfa.cc
@@ -610,14 +610,14 @@ void DFA::dump_dot_graph(ostream & os)
 			if (j->second == nonmatching)
 				excluded.insert(j->first);
 			else {
-				os << "\t\"" << **i << "\" -> \"" << j->second
+				os << "\t\"" << **i << "\" -> \"" << *j->second
 				   << "\" [" << "\n";
-				os << "\t\tlabel=\"" << j-> first << "\"\n";
+				os << "\t\tlabel=\"" << j->first << "\"\n";
 				os << "\t]" << "\n";
 			}
 		}
 		if (cases.otherwise && cases.otherwise != nonmatching) {
-			os << "\t\"" << **i << "\" -> \"" << cases.otherwise
+			os << "\t\"" << **i << "\" -> \"" << *cases.otherwise
 			   << "\" [" << "\n";
 			if (!excluded.empty()) {
 				os << "\t\tlabel=\"[^";
-- 
1.7.4.1




More information about the AppArmor mailing list