pinentry-dmenu

Commit: 07b14f1e91beb96591b210bc7b1cd75a516d646d
Parent: 6a22ded419a9b22f6b05b4c9ef2655e83ecf9d35
Author: Moritz Luedecke
Date:   Sun, 24 Sep 2017 20:34:36 +0200

Don't delete the whole suffix when remove a symbol in the middle of the passphrase

Diffstat:
M
pinentry-dmenu.c
 | 
9
+++++++--

1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/pinentry-dmenu.c b/pinentry-dmenu.c
@@ -125,8 +125,9 @@ nextrune(int cursor, int inc) {
 static void
 insert(const char *str, ssize_t n) {
 	int repeat;
+	size_t len = strlen(pin);
 
-	if (strlen(pin) + n > pinentry->pin_len - 1) {
+	if (len + n > pinentry->pin_len - 1) {
 		repeat = (pin == pinentry->pin) ? 0 : 1;
 
 		if (!pinentry_setbufferlen(pinentry, 2 * pinentry->pin_len)) {
@@ -137,12 +138,16 @@ insert(const char *str, ssize_t n) {
 		pin = (repeat) ? pinentry->pin : pinentry->repeat_passphrase;
 	}
 
+	/* Move existing text out of the way, insert new text, and update cursor */
+	memmove(&pin[cursor + n], &pin[cursor],
+	        pinentry->pin_len - cursor - MAX(n, 0));
+
 	if (n > 0) {
 		memcpy(&pin[cursor], str, n);
 	}
 
 	cursor += n;
-	pin[cursor] = '\0';
+	pin[len + n] = '\0';
 }
 
 static void