From cdaa9517afc5cd85da7c92ffa272417771632a07 Mon Sep 17 00:00:00 2001
From: Colin Walters <walters@verbum.org>
Date: Sun, 20 Jul 2014 22:32:39 -0400
Subject: [PATCH] repofile: Avoid segfault if querying child in non-directory

The user might "ostree ls /usr/bin/bash/blah", which previously would
segfault.

A somewhat related future enhancement here would be for "ostree ls" to
follow symbolic links.

Reported-by: Dusty Mabe <dustymabe@gmail.com>

https://bugzilla.gnome.org/show_bug.cgi?id=733476
---
 src/libostree/ostree-repo-file.c | 7 +++++++
 tests/test-basic.sh              | 7 +++++++
 2 files changed, 14 insertions(+)

diff --git a/src/libostree/ostree-repo-file.c b/src/libostree/ostree-repo-file.c
index 454b523b..f3215c62 100644
--- a/src/libostree/ostree-repo-file.c
+++ b/src/libostree/ostree-repo-file.c
@@ -223,6 +223,13 @@ do_resolve_nonroot (OstreeRepoFile     *self,
   if (!ostree_repo_file_ensure_resolved (self->parent, error))
     goto out;
 
+  if (!self->parent->tree_contents)
+    {
+      g_set_error (error, G_IO_ERROR, G_IO_ERROR_NOT_DIRECTORY,
+                   "Not a directory");
+      goto out;
+    }
+
   i = ostree_repo_file_tree_find_child (self->parent, self->name, &is_dir, &container);
 
   if (i < 0)
diff --git a/tests/test-basic.sh b/tests/test-basic.sh
index 19e98b1e..e0783855 100755
--- a/tests/test-basic.sh
+++ b/tests/test-basic.sh
@@ -267,6 +267,13 @@ cd ${test_tmpdir}
 $OSTREE ls test2
 echo "ok ls with no argument"
 
+cd ${test_tmpdir}
+if $OSTREE ls test2 /baz/cow/notadir 2>errmsg; then
+    assert_not_reached
+fi
+assert_file_has_content errmsg "Not a directory"
+echo "ok ls of not a directory"
+
 cd ${test_tmpdir}
 $OSTREE show test2
 echo "ok show with non-checksum"