From: Rich Felker <dalias@aerifal.cx>
Date: Sat, 22 Jun 2013 21:23:45 +0000 (-0400)
Subject: fix scanf %c conversion wrongly storing a terminating null byte
X-Git-Tag: v0.9.11~27
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ef5507867b59d19f21437970e87b5d0415c07b2e;p=musl

fix scanf %c conversion wrongly storing a terminating null byte

this seems to have been a regression from the refactoring which added
the 'm' modifier.
---

diff --git a/src/stdio/vfscanf.c b/src/stdio/vfscanf.c
index 6bea6ad8..bb928480 100644
--- a/src/stdio/vfscanf.c
+++ b/src/stdio/vfscanf.c
@@ -265,8 +265,10 @@ int vfscanf(FILE *restrict f, const char *restrict fmt, va_list ap)
 				if (size == SIZE_l) *(wchar_t **)dest = wcs;
 				else *(char **)dest = s;
 			}
-			if (wcs) wcs[i] = 0;
-			if (s) s[i] = 0;
+			if (t != 'c') {
+				if (wcs) wcs[i] = 0;
+				if (s) s[i] = 0;
+			}
 			break;
 		case 'p':
 		case 'X':
diff --git a/src/stdio/vfwscanf.c b/src/stdio/vfwscanf.c
index b1eb7939..760864ff 100644
--- a/src/stdio/vfwscanf.c
+++ b/src/stdio/vfwscanf.c
@@ -281,8 +281,10 @@ int vfwscanf(FILE *restrict f, const wchar_t *restrict fmt, va_list ap)
 				if (size == SIZE_l) *(wchar_t **)dest = wcs;
 				else *(char **)dest = s;
 			}
-			if (wcs) wcs[i] = 0;
-			if (s) s[i] = 0;
+			if (t != 'c') {
+				if (wcs) wcs[i] = 0;
+				if (s) s[i] = 0;
+			}
 			break;
 
 		case 'd': case 'i': case 'o': case 'u': case 'x':