commit 269bb496c56dcf268831e02df035040162f30590
author Ryan Prichard <rprichard@google.com> Thu Oct 04 14:45:55 2018 -0700
committer Ryan Prichard <rprichard@google.com> Mon Oct 08 13:27:16 2018 -0700
tree 822f2f0cd22cfc397273ccca827414679e842b77
parent 0adf09b3705c54733e46ace3b6cf08b78533729e

Fix normalize_path's handling of "/.."

Currently it normalizes the path to a string with a single uninitialized
byte. It should instead normalize it to "/".

Bug: none
Test: /data/nativetest/linker-unit-tests/linker-unit-tests32
Test: /data/nativetest64/linker-unit-tests/linker-unit-tests64
Change-Id: I06e0f7598d16acfa21875dad53efbc293cfeb44d

linker/tests/linker_utils_test.cpp

diff --git a/linker/tests/linker_utils_test.cpp b/linker/tests/linker_utils_test.cpp
index dce223a..e406af5 100644
--- a/linker/tests/linker_utils_test.cpp
+++ b/linker/tests/linker_utils_test.cpp
@@ -58,6 +58,9 @@
   ASSERT_TRUE(normalize_path("/a/../../b", &output));
   ASSERT_EQ("/b", output);
 
+  ASSERT_TRUE(normalize_path("/..", &output));
+  ASSERT_EQ("/", output);
+
   output = "unchanged";
   ASSERT_FALSE(normalize_path("root///dir/.///dir2/somedir/../zipfile!/dir/dir9//..///afile", &output));
   ASSERT_EQ("unchanged", output);