Pasaffe

Pound signs cause truncation / unicode -1 chars

Bug #1819141 reported by Adrian Wilkins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Pasaffe
Fix Released
Undecided
Unassigned

Bug Description

* Create a password that's 40 "normal" chars in gedit

e.g.

1234567890123456789012345678901234567890

* Create a new entry with it
* Close the edit window
* Copy from the entry in the tree
* Paste under the first one in gedit
* Passwords match
* Replace first 4 chars with pound signs
* Close and copy and paste into gedit
* Password is truncated at 36 chars

Pound signs anywhere in the password have this effect. Takes effect at any length.

£1 == £

If number of pound signs exceeds number of other chars, you start to get this :

1£ = 1\Uffffffff

££1 == £\Uffffffff

£££1 == £££1

££££1 == ££\Uffffffff

Workaround : copy the password from the edit window

Adrian Wilkins (adrian-wilkins)
summary: - Copying password from entry truncates at 36 chars
+ Pound signs cause truncation / unicode -1 chars
Revision history for this message
Adrian Wilkins (adrian-wilkins) wrote :

This is an encoding problem ; the clipboard receives double-byte UTF-8 chars for the pound sign but the buffer length copied is the number of single chars.

Linked branch patches this to calculate the buffer length correctly.

Revision history for this message
Christophe Kamphaus (ckamphaus) wrote :

I have implemented various improvements to pasaffe and I included your fix.
The results I have published on my ppa:
https://launchpad.net/~ckamphaus/+archive/ubuntu/pasaffe

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Thanks for reporting this bug! For some reason, it dropped off my radar. I'll review the patch this week.

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

@ckamphaus: Those look like great improvements...could you please propose a merge request against the Pasaffe tree and I'll include them in a new release?

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Oh! It looks like you did propose a merge, awesome, thanks!

Revision history for this message
Christophe Kamphaus (ckamphaus) wrote :

Before I merged @adrian-wilkins branch, I confirmed that the bug was real (it was real) and afterwards I double-checked that the bug was fixed (it is now fixed).

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

This was fixed in 0.54. Thanks!

Changed in pasaffe:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.