Tighten sanity checks, part XXXIII.

- Check whether the artifacts in a child list actually appears in their
  products' artifact list.
- Do sanity checking right before storing the build graph. This makes
  sure that a corrupted build graph never gets written to disk.

Change-Id: I06b02b97c3621ecdc1f6dae8e6ce818519f9736e
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/src/lib/api/internaljobs.cpp b/src/lib/api/internaljobs.cpp
index e0b0afa0a..7a16ff2ad 100644
--- a/src/lib/api/internaljobs.cpp
+++ b/src/lib/api/internaljobs.cpp
@@ -31,6 +31,7 @@
 #include "jobs.h"
 
 #include <buildgraph/artifactcleaner.h>
+#include <buildgraph/buildgraph.h>
 #include <buildgraph/buildgraphloader.h>
 #include <buildgraph/productbuilddata.h>
 #include <buildgraph/projectbuilddata.h>
@@ -132,9 +133,10 @@ void InternalJob::shareObserverWith(InternalJob *otherJob)
     m_observer = otherJob->m_observer;
 }
 
-void InternalJob::storeBuildGraph(const TopLevelProjectConstPtr &project)
+void InternalJob::storeBuildGraph(const TopLevelProjectPtr &project)
 {
     try {
+        doSanityChecks(project, logger());
         project->store(logger());
     } catch (const ErrorInfo &error) {
         logger().printWarning(error);