curses: dialog broken with wide characters
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	chyen
	Jul 1 2019, 3:23 PM

Description

If the description or the prompt contains non-ASCII characters, the dialog is broken. Looks like the dialog width is not correctly calculated. For example, the following commands:

$ ./curses/pinentry-curses
OK Pleased to meet you
setdesc 中文中文中文中文中文中文中文中文中文中文中文中文中文中文中文
OK
setprompt 中文中文中文中文中文
OK
getpin

Generates such a dialog:

┌─────────────────────────────────────────────────────┐
│ 中文中文中文中文中文中文中文中文中文中文中文中文中文中文中文
│                                                     │
│ 中文中文中文中文中文 ________________________________________
│                                                     │
│       <OK>                             <Cancel>     │
└─────────────────────────────────────────────────────┘

I gave it a try to fix the issue and got a patch (attached below). For the description it looks good:

┌──────────────────────────────────────────────────────────────┐
│ 中文中文中文中文中文中文中文中文中文中文中文中文中文中文中文 │
│                                                              │
│ 中文中文中文中文中文 _________________________________________________
│                                                              │
│         <OK>                                  <Cancel>       │
└──────────────────────────────────────────────────────────────┘

But the dialog is still broken with non-ASCII prompts.

pinentry-curses-chinese.patch1 KBDownload

Details

Version: master

Revisions and Commits

rP Pinentry
	rPdad35d65f05e curses: Fix more.
	rPa03f39207148 curses: Fix handling of newline in description.
	rP9298cca67742 curses: Fix calculation of columns for CJK text.

Related Objects
Search...

		Status	Assigned	Task
		Resolved	• gniibe	T4598 curses: dialog broken with wide characters
		Resolved	gouttegd	T4659 Release Pinentry-1.1.1

Event Timeline

chyen created this task.Jul 1 2019, 3:23 PM

chyen created this object in space S1 Public.

Well in my browser (Firefox) the dialogs are not rendered correctly. Here are the two dialogs in the terminal:

• werner triaged this task as Normal priority.Jul 1 2019, 9:59 PM

• gniibe claimed this task.Jul 9 2019, 7:20 AM

I pushed the change to master.
Please test.

• gniibe added a commit: rP9298cca67742: curses: Fix calculation of columns for CJK text..Jul 9 2019, 7:41 AM

Thanks for the update! With git-master, the toy example above works fine. However, pinentry-curses seems to hang with real commands from gpg. Here is an example:

$ ./curses/pinentry-curses 
OK Pleased to meet you
SETDESC 請輸入密語來解鎖 OpenPGP 私鑰:%0A%22Chih-Hsuan Yen <yan12125@gmail.com>%22%0A3072 位元長的 DSA 金鑰, ID F98EF2A7B0A098AE,%0A建立於 2018-04-25 (主要金鑰 ID 3FDDD575826C5C30).%0A
OK
SETPROMPT 密語:
OK
GETPIN

(CPU usage of ./curses/pinentry-curses goes > 90%)

My terminal info:

$ stty size
36 146

• gniibe added a commit: rPa03f39207148: curses: Fix handling of newline in description..Jul 9 2019, 9:30 AM

Thanks for the further fix! With that only a minor hiccup remains:

A segment in the left border is not drawn.

Thanks for further testing.
I realized that it's not the left border drawing problem in fact, but the newline should be between the description and passphrase line.
I'm going to fix this.

BTW, I envy somehow that you have good concise words in Chinese for technical terms of cryptography. Even non-native speaker like me can understand them.

In Japanese, by easier normal words, it is difficult to distinguish between key and lock, and the concept of keyring is not popular, while people can understand when given good explanations.

I pushed the fix. Thanks for your cooperation.

• gniibe added a commit: rPdad35d65f05e: curses: Fix more..Jul 10 2019, 2:16 AM

• gniibe closed this task as Resolved.Jul 11 2019, 3:53 AM

Thanks for all the fixes! I can confirm commit dad35d65f05eb1c15589a7e4755dcae6aed2d6cf works just fine on all my machines (Linux & macOS).

BTW, I envy somehow that you have good concise words in Chinese for technical terms of cryptography.

Words like 金鑰 or 私鑰 are all newly created specifically for translating cryptographical terms. I guess those who speaking Japanese can do something similar.

That reminds me that there are two different usages for the term 密鑰:

Refer to private keys
Refer to general keys, either public or private

Sometimes it's easier to create terms than to keep the same usage everywhere :)

• gniibe added a subtask: T4659: Release Pinentry-1.1.1.Dec 5 2019, 7:35 AM

gouttegd closed subtask T4659: Release Pinentry-1.1.1 as Resolved.Jan 23 2021, 11:22 PM

curses: dialog broken with wide charactersClosed, ResolvedPublicActions