fix: fix the logic of encoding/decoding of the strings.

It must be first decoded to the plain UTF-8 string, and only then
bastard-encoded for the Assuan protocol.

1 files changed, 8 insertions, 4 deletions

diff --git a/pinentry-rofi.sh b/pinentry-rofi.sh
index 13e89cb..5c7dd37 100755
--- a/pinentry-rofi.sh
+++ b/pinentry-rofi.sh
@@ -82,7 +82,7 @@ rawurldecode() {
 }
 
 # Except Assuan apparently doesn't want really encoded strings, but some kind of bastard
-basturldecode () {
+basturlencode () {
     echo "$1" | sed -e 's/</\&lt;/g' -e 's/>/\&gt;/g'
 }
 
@@ -98,6 +98,7 @@ win_mesg=""
 keyinfo=""
 
 main () {
+    local temp_str
 
     while : ; do
         read -r line
@@ -167,7 +168,9 @@ main () {
             #SETDESC Please enter the passphrase for the ssh key%0A  ke:yf:in:ge:rp:ri:nt
             IFS=" " line_arr=($(split_line "$line"))
             log_debug "line_arr: ${line_arr[*]}"
-            win_mesg="$(basturldecode "${line_arr[*]}")"
+            temp_str="$(rawurldecode "${line_arr[*]}")"
+            log_debug "temp_str: ${temp_str}"
+            win_mesg="$(basturlencode "${temp_str}")"
             assuan_send "OK"
         elif [[ "$line" =~ ^SETPROMPT ]] ; then
             #SETPROMPT Passphrase:
@@ -178,8 +181,9 @@ main () {
         elif [[ "$line" =~ ^SETTITLE ]] ; then
             IFS=" " line_arr=($(split_line "$line"))
             log_debug "line_arr: ${line_arr[*]}"
-            log_debug "line_arr: ${line_arr[*]}"
-            win_title="$(basturldecode "${line_arr[*]}")"
+            temp_str="$(rawurldecode "${line_arr[*]}")"
+            log_debug "temp_str: ${temp_str}"
+            win_title="$(basturlencode "${temp_str}")"
             assuan_send "OK"
         elif [[ "$line" =~ ^GETPIN ]] ; then
             passw=None