Input scanner: Prevent resolving directories as included files

This can otherwise easily happen, in particular with C++ standard
library headers that don't have file extensions.

Change-Id: I30529271278f567e6cb26054ea3b01df7efcfbfa
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>

1 files changed, 4 insertions, 1 deletions

diff --git a/src/lib/corelib/buildgraph/inputartifactscanner.cpp b/src/lib/corelib/buildgraph/inputartifactscanner.cpp
index e0d0726c3..e24d603fe 100644
--- a/src/lib/corelib/buildgraph/inputartifactscanner.cpp
+++ b/src/lib/corelib/buildgraph/inputartifactscanner.cpp
@@ -103,7 +103,10 @@ static void resolveWithIncludePath(const QString &includePath,
     }
 
     QString absFilePath = absDirPath + QLatin1Char('/') + dependency.fileName();
-    if (FileInfo::exists(absFilePath))
+
+    // TODO: We probably need a flag that tells us whether directories are allowed.
+    const FileInfo fi(absFilePath);
+    if (fi.exists(absFilePath) && !fi.isDir())
         result->filePath = absFilePath;
 }