Fixed escapeForFileName(). [#CLICKHOUSE-2]

dbms/src/Common/escapeForFileName.cpp

@@ -39,15 +39,16 @@ std::string unescapeForFileName(const std::string & s)
     while (pos != end)
     {
         if (!(*pos == '%' && pos + 2 < end))
+        {
             res += *pos;
+            ++pos;
+        }
         else
         {
             ++pos;
             res += unhex2(pos);
             pos += 2;
         }
-
-        ++pos;
     }
     return res;
 }

dbms/src/Common/tests/gtest_unescapeForFileName.cpp

+#include <Common/escapeForFileName.h>
+#include <gtest/gtest.h>
+
+using namespace DB;
+
+
+TEST(Common, unescapeForFileName)
+{
+    EXPECT_EQ(unescapeForFileName(escapeForFileName("172.19.0.6")), "172.19.0.6");
+    EXPECT_EQ(unescapeForFileName(escapeForFileName("abcd.")), "abcd.");
+    EXPECT_EQ(unescapeForFileName(escapeForFileName("abcd")), "abcd");
+    EXPECT_EQ(unescapeForFileName(escapeForFileName("..::")), "..::");
+}