severity 392493 important
tags 392493 confirmed patch
reassign 392493 libetpan
thanks

  Hi,

On Thu, 12 Oct 2006 01:29:20 +0200
Fabian Gorsler <[EMAIL PROTECTED]> wrote:

> Package: sylpheed-claws-gtk2
> Version: 2.5.3-1
> Severity: grave

  Sorry, but what you describe here doesn't qualify as grave [1].
 
> Hi,
> 
> today I've tried sylpheed-claws-gtk2, but testing it didn't take a long
> time, because getting the folder list on an IMAPS connection leads to a
> seg fault. 
> 
> If sylpheed-claws-gtk2 is started with --debug the last printed lines
> are:
> 
> -----------
> summaryview.c:964:empty folder ((nil) NULL (nil) 0)
> 
> summaryview.c:984:TIMING summary_show : 0s001ms
> [01:21:25] IMAP4< * OK dovecot ready.
> imap-thread.c:312:generic_cb
> Segmentation fault
> -----------

  I've set up the enviroment you describe and I can confirm the segfault,
but the bug is not in sylpheed-claws-gtk2, it's in libetpan, the library
claws uses for IMAP access.

  The attached patch solved the bug for me, but I'm CCing libetpan's
upstream so he can verify it.

> Using IMAP without SSL works fine. The connection can be established
> and the folder listing can be retrived.
> 
> On server-side runs dovecot 0.99.14-1sarge0 with a self-signed
> certificate. sylpheed-claws-gtk2 is the current version from 
> unstable. Perhaps this information could be useful. 

  Certanly, I'm running against a IMAPS server also, with a self-signed
certificate too (but courier on OpenBSD) and had no problems so far.

  thanks,

[1] http://www.debian.org/Bugs/Developer#severities
-- 
 Ricardo Mones
 http://people.debian.org/~mones
 «Q: What is the difference between Texas and yogurt? A: Yogurt has 
 culture.»
diff -urN libetpan-0.46.orig/src/data-types/mmapstring.c libetpan-0.46/src/data-types/mmapstring.c
--- libetpan-0.46.orig/src/data-types/mmapstring.c	2006-10-12 21:31:36.000000000 +0200
+++ libetpan-0.46/src/data-types/mmapstring.c	2006-10-12 21:34:37.000000000 +0200
@@ -497,7 +497,11 @@
 mmap_string_append (MMAPString     *string,
 		    const char *val)
 {  
-  return mmap_string_insert_len (string, string->len, val, strlen(val));
+  if (val != NULL) {
+    return mmap_string_insert_len (string, string->len, val, strlen(val));
+  }
+
+  return string;
 }
 
 MMAPString*

Attachment: signature.asc
Description: PGP signature

Reply via email to