From: Felipe Sateler <fsateler@users.noreply.github.com>
Date: Thu, 14 Sep 2017 17:51:20 +0000 (-0300)
Subject: shared: end string with % if one was found at the end of a expandible string (#6828)
X-Git-Tag: v235~106
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=038492aed3e0293fd9cf4998fd891addb597b954;p=thirdparty%2Fsystemd.git

shared: end string with % if one was found at the end of a expandible string (#6828)

Current behavior is that %X where X is an unidentified specifier, then the result is
the same %X string. This was not the case when the string ended with a stray %, where
the character would have not been output. Lets add that missing character.

Fixes: #6374
---

diff --git a/src/shared/specifier.c b/src/shared/specifier.c
index 1c17eb52514..81379041cc9 100644
--- a/src/shared/specifier.c
+++ b/src/shared/specifier.c
@@ -107,6 +107,10 @@ int specifier_printf(const char *text, const Specifier table[], void *userdata,
                         *(t++) = *f;
         }
 
+        /* if string ended with a stray %, also end with % */
+        if (percent)
+                *(t++) = '%';
+
         *t = 0;
         *_ret = ret;
         return 0;
diff --git a/src/test/test-unit-name.c b/src/test/test-unit-name.c
index b8979c0e956..dcccc9080cd 100644
--- a/src/test/test-unit-name.c
+++ b/src/test/test-unit-name.c
@@ -237,7 +237,8 @@ static int test_unit_printf(void) {
         /* general tests */
         expect(u, "%%", "%");
         expect(u, "%%s", "%s");
-        expect(u, "%", "");    // REALLY?
+        expect(u, "%,", "%,");
+        expect(u, "%", "%");
 
         /* normal unit */
         expect(u, "%n", "blah.service");