Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Yavor Doganov
В 15:46 +0100 на 20.01.2009 (вт), Riccardo Lancellotti написа:
> I compiled and re-installed but the problem seems to be still there. I
> made available a vcf file that can be used to reproduce the error.

Yes, I get a different crash with your card.  The problem is the
presence of "\n" in some fields, which is perfectly fine for a vcard
file, AFAIK.  However, it looks like the parser is a bit dumb and counts
the lines by separating them as substrings based on "\n".  Ugh.




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Riccardo Lancellotti
On, 20/01/2009 at 15.24 +0200, Yavor Doganov wrote:
> Could you please rebuild the addresses-for-gnustep package with the
> attached patch, install libaddresses0 and report if it improves things?
I compiled and re-installed but the problem seems to be still there. I
made available a vcf file that can be used to reproduce the error.
-- 
Saluti,
   Riccardo




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Riccardo Lancellotti
On, 20/01/2009 at 08.06 -0500, Hubert Chathi wrote:

> Yes, please provide a sample file; I don't use evolution.  Feel free to
> replace any personal information with dummy data -- just make sure that
> the new file still causes addressmanager.app to fail.
Here it is.
I modified a few personal phone numbers and addresses, while leaving the
public ones untouched.
I also tried to remove the photo: the parses comments something about an
unknown encoding, but the problem is still present, so I re-inserted the
field.

-- 
Saluti,
   Riccardo
BEGIN:VCARD
VERSION:3.0
X-ICQ;TYPE=HOME;X-EVOLUTION-UI-SLOT=2:262278179
X-JABBER;TYPE=HOME;X-EVOLUTION-UI-SLOT=1:lancellotti.ricca...@jabber.org
TEL;TYPE=CELL;X-EVOLUTION-UI-SLOT=4:+39338111
TEL;TYPE=HOME,VOICE;X-EVOLUTION-UI-SLOT=3:+3905911
TEL;TYPE=WORK,FAX;X-EVOLUTION-UI-SLOT=2:+390592056129
TEL;TYPE=WORK,VOICE;X-EVOLUTION-UI-SLOT=1:+390592056256
EMAIL;TYPE=OTHER;X-EVOLUTION-UI-SLOT=4:riccardo.lancello...@provider.it
EMAIL;TYPE=OTHER;X-EVOLUTION-UI-SLOT=2:lancellotti.ricca...@provider.it
EMAIL;TYPE=WORK;X-EVOLUTION-UI-SLOT=1:lancellotti.ricca...@provider.it
ADR;TYPE=HOME:;;Via Y;Modena;;41100;Italy
URL:http://weblab.ing.unimo.it/people/riccardo/
X-EVOLUTION-FILE-AS:Lancellotti\, Riccardo
NICKNAME:nickname
FN:Riccardo Lancellotti
N:Lancellotti;Riccardo;;;
UID:pas-id-441AF7AD
PHOTO;ENCODING=b;TYPE=JPEG:/9j/4AAQSkZJRgABAQAAAQABAAD/2wBDAAEBAQEBAQEBAQEB
 AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQH/2w
 BDAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEB
 AQEBAQEBAQEBAQH/wAARCABgAFcDASIAAhEBAxEB/8QAHwAAAQUBAQEBAQEAAAECAw
 QFBgcICQoL/8QAtRAAAgEDAwIEAwUFBAQAAAF9AQIDAAQRBRIhMUEGE1FhByJxFDKBkaEII0Kx
 wRVS0fAkM2JyggkKFhcYGRolJicoKSo0NTY3ODk6Q0RFRkdISUpTVFVWV1hZWmNkZWZnaGlqc3
 R1dnd4eXqDhIWGh4iJipKTlJWWl5iZmqKjpKWmp6ipqrKztLW2t7i5usLDxMXGx8jJytLT1NXW
 19jZ2uHi4+Tl5ufo6erx8vP09fb3+Pn6/8QAHwEAAwEBAQEBAQEBAQECAwQFBgcICQ
 oL/8QAtREAAgECBAQDBAcFBAQAAQJ3AAECAxEEBSExBhJBUQdhcRMiMoEIFEKRobHBCSMzUvAV
 YnLRChYkNOEl8RcYGRomJygpKjU2Nzg5OkNERUZHSElKU1RVVldYWVpjZGVmZ2hpanN0dXZ3eH
 l6goOEhYaHiImKkpOUlZaXmJmaoqOkpaanqKmqsrO0tba3uLm6wsPExcbHyMnK0tPU1dbX2Nna
 4uPk5ebn6Onq8vP09fb3+Pn6/9oADAMBAAIRAxEAPwD5bERwdyEjIAz75qYWrMQEjJOCTjJ4Bw
 RhQxyeMZwOTk5xX5keMf2vPiXPez2Gj6ro+kQRyMskui6Ppc6kMAyQwXepW+oTNJGDueWOdAWP
 y4XCjxDWPjV8R9eFwNZ8beK7pJpt6xv4i1KwgVdrLsW1065trZUI5CLHsAyoGMV/beO+kVwtSj
 L6jlWa4ttWpVMRThhIOV07Lm9o07b3XY/j/CfR84pxF4Y/McswPLLlrQo1J4qtSd1dSUVSto7p
 63tpY/aCY29tzeXENmiqXeW7mitYgBksWlmdUXGD98qDkDOawm8V+C45DA3jXwPHPhcW8njLww
 k29sHZ5b6uj79pDeWoL84K54P4Van4laZ5J7zU98kpx88kk07kEZDSyySuxIXBdizd+O3OT+Ik
 uJVRY5JnYnb+83ZAA3MQwGVVeGY5GAQ2K+QrfSWxEZ/7Lw1hox5rSdfMsZKpKN9Y/u5RhG9lpF
 KKsklZI+qwv0c8IoWxPEuLc3BRXscHh3TU1a0nGuptvR+9fm182f0FfbLF7WTUf7Q046cmPMv0
 1CzksIhk4aW8WZraMMOVYylGwSWUDJ851b42fBfw7dnT9b+K/gPT78MFNnL4gs3ulclQqywwtM
 8JcuvlLOsTSZyqbSrN+E19qUdmL671UXT2yWUssNjp00ukafvadraAS6hIJBqLzyoRFDZwOplZ
 40nR4mMXz54lm8SaTqdzeW9+LdNSmS4lsrS4mnhGAskEU8dxczSTGMhTG8xkPm5YEEALyYr6Tm
 ZOUY4PhrLqU+aPtPbYrEVJTimnN0p3TiuW9ua7dvM6qX0bsvpKrPFcQ46v/wA+pUcNh6V7pJOr
 Gzi2m7NRSWnV3P6mNF1jSPENjBquianZ6tptypa2vrCVLm2cBmUkSRkgfMjLg88cgVqmOTkY65
 BOMjnr/kfhX8+/wR+NPjj4UWln410jW7q5t7Y/8TPwlqV7fQaDq9heMs7LbRy3jWBnlnWSWQW6
 WmoQyDfDdENLHJ+5nwp+I+nfFLwjoXimxewkGt6PaaoyaZczyw2Esryw3Wl3Md2sdzBe2N3DNb
 zI4kyqJIJCsqk/tPh54r5Lx5SqUIRWAzegoupl8pymqkNFKrRqTScordxd3e6VkrH5Dx94YZnw
 VOnXVWONynEWjSxdlGrCpZJwq0oR5U+a/LK6Ukk2dwLfHBPY5GPUH1J9c9P8aUoiqUwckDB+mc
 Z7n/P4aZiDAqRjGdpwQxxjOTuI9ecfhnFMEC7SDknB7n+tfrkFZLz1+9I/KerXZ2/L/MyjG3YE
 /h/9eitQRbAQFOCcnPP0/wAR+dFL2i7P+v6f9PQP5nvFHxA1o3moXum+E9Yg0y41OZdPSVYZZo
 4bmUtY2pEUssjSRwFYXkCAO6DGTzTdI0v4r+KpIxFoCaLE+7bceIrwaYhUNj5ISpuX2gnGIBv2
 nHOBWzqsOvW/i+70DRDokmmW2r30cOpag9xYyeVa3c0UDSJOQ0TeWgJR7dJI2z07dxLqWbW4td
 U1y6l1KDNslpoUmyw8lFTEkl3Cp1K6d23ZaTUEiUYIthuXy/8AJTBTVXCYadNuMalGMrb36Nuo
 tZPvF7H+tXGNCrh+LeI6GIjFVcPnma0ajhTVGnUlTxTipwoKMI0IqKUY0oRUIJWjpovPNV8FXe
 jS+V4m8c6VbOqq7W2jQi6uDu3Exk3slvIrOFZVaO3lLDJVSTWQ1hZ6aunjSL+81XUtffybZtRZ
 ovsqzXIs4YvIEMRBuH8yR90f+qi3BW3Kzeiw3VtBbCGw0RLbfJHLNKMCS5lhLYeQMXmZ2Z2Imc
 s7ZO5iuccT4Zt5tZ+Kuh6ZcOyi516ynVZQPlihnLSqmQOIowXUL1+8OvN1pOnSqzSk3ThKSUbX
 umu9tDwcLFVq9OmoczqSjF+6nZOUbyet/uTeux9+fAH9nPwnrfxI0rwd8SNQsLV47HSNTsZ/EU
 8lroGty6rqVtoQsLQy8SXdvrmob2sx5YXTTLesYlLOvXftHfsr+CfD/wAQNTuNA0W3tvDPi/8A
 s298D29lbtbxRaf4i0SbxHo8UCSSS7JYvsV9oNzbKS1rrFhe6dKTd20qr9v6rr3wj8MeF9NvNR
 0b4kwabqVrpltdatqHgf4f6x4fkvUMCtcaPbazqkfiq40db6OOSVb7w/q2maksTSx2aeWlzX2X
 4j+EXw18X/sj6j428S+J9euBo722reDfGHhPTJr3UdC1a+ubnW5NF8SeHNTZYrywXxTaWviGze
 XVILvS9fln1G0nisdQv9Ku/wAszTOMZQxdDEupUo06slTpwlJuPNKSjdQ5uVRs9/w7/wBDZLwb
 luY4Cph/Y4adanTVSpVbg2lyRailyupzXVrWSW7tds/lp1TwXqdlouoRHT7tPBcGpxaHNrtpDI
 66DrN1LdT2VjqrQ8q0zLLFHE5jluAGit1mZDHX1t+yn8Rrn4U3PhL4eXmp2eraZ4o1S6JtIbcM
 2lzXM1nDY31tq1xGHSO5kF4LuK0VYGiNhLcwzOk6j074dfC/xB4z1HW/Cdjrmi638Pvi3rPhpf
 i94O8P+OPhJefE62tfD3iES/8ACXeGPAFvqtybe8s49Uu9RNlJYw3kED6nsa4eG1uYPk/xL8P9
 Y+B/7TutfCfWzcTXHw88bw29vOW2XE2lvHbazp1zHFIMsl/o13a3UEa7DKkh2RklRX6

Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Yavor Doganov
reassign 512382 libaddresses0
tags 512382 + patch
thanks

В 12:47 +0100 на 20.01.2009 (вт), Riccardo Lancellotti написа:
> The program aborts when importing vcard files exported from evolution.
> The application receives a signal (I think SIGABORT or SIGSEV). 

I use Evolution (at work, at least), and I confirm this.

Could you please rebuild the addresses-for-gnustep package with the
attached patch, install libaddresses0 and report if it improves things?
Thanks.

(I'm not familiar with the code and not sure if it's correct, but with
it importing succeeds for me.)
2009-01-20  Yavor Doganov(tiny change)

	* ADVCFConverter.m ([ADVCFConverter -nextRecord]): Only call the
	-substringFromIndex method if the index is not greater than the
	length of the receiver.  Avoids raising an exception when
	importing vCard files.
	Reported by Riccardo Lancellotti.

--- addresses-for-gnustep-0.4.7/Frameworks/Addresses/ADVCFConverter.m.orig	2009-01-20 15:08:29.0 +0200
+++ addresses-for-gnustep-0.4.7/Frameworks/Addresses/ADVCFConverter.m	2009-01-20 15:09:13.0 +0200
@@ -321,8 +321,9 @@
   person = [[[ADPerson alloc] init] autorelease];
   [person setValue: [NSDate date] forProperty: ADModificationDateProperty];
   [person setValue: [NSDate date] forProperty: ADCreationDateProperty];
-  
-  str = [_str substringFromIndex: _idx];
+
+  if (_idx < [_str length])
+str = [_str substringFromIndex: _idx];
 
   lines = [str componentsSeparatedByString: @"\n"];
   


Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Hubert Chathi
On Tue, 20 Jan 2009 12:47:37 +0100 Riccardo Lancellotti
 wrote:

> The program aborts when importing vcard files exported from evolution.
> The application receives a signal (I think SIGABORT or SIGSEV). 
>
> The parsing process seems to be OK because I can read on the command
> line every token in the vcard file, but then something happens.
> I suspect the bug to be not Debian-specific, but I cannot be sure.
> If needed I can provide a sample file to reproduce the error.

Yes, please provide a sample file; I don't use evolution.  Feel free to
replace any personal information with dummy data -- just make sure that
the new file still causes addressmanager.app to fail.

Thanks

-- 
Hubert Chathi  -- Jabber: hub...@uhoreg.ca
PGP/GnuPG key: 1024D/124B61FA http://www.uhoreg.ca/
Fingerprint: 96C5 012F 5F74 A5F7 1FF7  5291 AF29 C719 124B 61FA



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#512382: addressmanager.app: fails to import vcard exported from Evolution

2009-01-20 Thread Riccardo Lancellotti
Package: addressmanager.app
Version: 0.4.7-1+b1
Severity: normal


The program aborts when importing vcard files exported from evolution.
The application receives a signal (I think SIGABORT or SIGSEV). 

The parsing process seems to be OK because I can read on the command
line every token in the vcard file, but then something happens.
I suspect the bug to be not Debian-specific, but I cannot be sure.
If needed I can provide a sample file to reproduce the error.

-- System Information:
Debian Release: 5.0
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.26.3 (SMP w/2 CPU cores)
Locale: LANG=it_IT.UTF-8, LC_CTYPE=it_IT.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages addressmanager.app depends on:
ii  gnustep-back0.14  0.14.0-2   The GNUstep GUI Backend
ii  gnustep-base-runtime  1.16.1-2lenny1 GNUstep Base library
ii  gnustep-gpbs  0.14.0-2   The GNUstep PasteBoard
Server
ii  gnustep-gui-runtime   0.14.0-3lenny1 GNUstep GUI Library -
runtime file
ii  libaddresses0 0.4.7-1+b1 Database API backend
framework for
ii  libaddressview0   0.4.7-1+b1 Address display/edit
framework for
ii  libc6 2.7-18 GNU C Library: Shared
libraries
ii  libgcc1   1:4.3.2-1.1GCC support library
ii  libgnustep-base1.16   1.16.1-2lenny1 GNUstep Base library
ii  libgnustep-gui0.140.14.0-3lenny1 GNUstep GUI Library
ii  libobjc2  4.3.2-1.1  Runtime library for GNU
Objective-

Versions of packages addressmanager.app recommends:
ii  gnumail.app 1.2.0~pre3+snap20071004-4+b1 fully featured mail
application

addressmanager.app suggests no packages.

-- no debconf information






-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org