Bug 8873 – Segmentation Fault on saving imported V2 Unicode files from Password Gorilla

Bug 8873 - Segmentation Fault on saving imported V2 Unicode files from Password Gorilla


Summary:	Segmentation Fault on saving imported V2 Unicode files from Password Gorilla

Status:	RESOLVED FIXED

Product:	PasswordSafe
Component:	General
Version:	unspecified
Platform:	N810 Maemo

Importance:	High major (vote)
Target Milestone:	---
Assigned To:	unassigned
QA Contact:	general-bugs

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree

Reported:	2010-02-06 21:37 UTC by tulse
Modified:	2010-02-15 11:51 UTC (History)
CC List:	1 user (show)

See Also:

Attachments
segfault file. passwd: test (766 bytes, application/x-force-download) 2010-02-14 22:13 UTC, tulse	Details
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description tulse (reporter) 2010-02-06 21:37:41 UTC

SOFTWARE VERSION:1.5.5f_armel
(Settings > General > About product)

EXACT STEPS LEADING TO PROBLEM: 
(Explain in detail what you do (e.g. tap on OK) and what you see (e.g. message
Connection Failed appears))
0. To obtain Unicode-compatible to other utils like Password Gorilla and others
I have to create dat-file in Password Gorilla
1. I create "V2 Unicode support" database in latest Password Gorilla 1.4
2. I open it with pwsafe, everything seems fine. Unicode characters look well
3. I tap Save (or Save As), and pwsafe crashes with Segmentation Fault. 
4. It leaves .new file before crashing seemingly.

EXPECTED OUTCOME:

ACTUAL OUTCOME:

REPRODUCIBILITY:always
(always, less than 1/10, 5/10, 9/10)

EXTRA SOFTWARE INSTALLED:

OTHER COMMENTS:

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US) AppleWebKit/532.8
(KHTML, like Gecko) Chrome/4.0.302.2 Safari/532.8

Comment 1 Fred Lefévère-Laoide 2010-02-14 21:49:00 UTC

Can you provide a crashing password file sample ?

Comment 2 tulse (reporter) 2010-02-14 22:13:53 UTC

Created an attachment (id=2290) [details]
segfault file. passwd: test

Comment 3 tulse (reporter) 2010-02-14 22:14:42 UTC

HE(In reply to comment #1)
> Can you provide a crashing password file sample ?
> 

Here it is with .new file.
Passwd: test.

Comment 4 Fred Lefévère-Laoide 2010-02-14 22:43:49 UTC

found the bug : the check for the version number was a little too permissive :
fixed in 1.6.0a

There are still some strange character at the end of the strings ...

Comment 5 Fred Lefévère-Laoide 2010-02-14 23:01:08 UTC

mixed with 8917

Comment 6 Fred Lefévère-Laoide 2010-02-14 23:06:52 UTC

fixed in 1.6.0a too

*** This bug has been marked as a duplicate of bug 8917 ***

Comment 7 tulse (reporter) 2010-02-15 06:52:49 UTC

I have just updated app, and it is still crashes on the same file and on file
created similar to this.

Comment 8 tulse (reporter) 2010-02-15 06:54:38 UTC

(In reply to comment #7)
> I have just updated app, and it is still crashes on the same file and on file
> created similar to this.
> 

Yes, I have double checked the ver. with "apt-cache show", and even rebooted
for sheer confidence.

Comment 9 Fred Lefévère-Laoide 2010-02-15 10:41:32 UTC

Sorry I was a little bit too quick last night about it : I did'n read well your
report and I only checked that it opened !!!

Comment 10 Fred Lefévère-Laoide 2010-02-15 11:51:23 UTC

fixed in 1.6.0b : UUID is created if not present in the V2 format
will soon be uploaded