ports/128637: [patch] fix SEGV in mutt-devel

[Eygene Ryabinkin]

>Number:         128637
>Category:       ports
>Synopsis:       [patch] fix SEGV in mutt-devel
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Nov 06 10:20:00 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Eygene Ryabinkin
>Release:        FreeBSD 7.1-PRERELEASE amd64
>Organization:
Code Labs
>Environment:

System: FreeBSD XXX 7.1-PRERELEASE FreeBSD 7.1-PRERELEASE #17: Thu Nov 6 12:14:18 MSK 2008 root at XXX:/usr/obj/usr/src/sys/XXX amd64

>Description:

There is a known bug in the Mutt 1.5.18: http://dev.mutt.org/trac/ticket/3057
It is rather nasty, because it crashes Mutt.

>How-To-Repeat:

>From the bug description:
-----
I start mutt. With c+<TAB>+<TAB> I change to imap.gmail.com where I have
to enter the passwort. With c+! I change back to home server. When I
press c+<TAB>+>TAB> again to access another preconfigured mailbox, mutt
crashes reproducible.
-----

>Fix:

Since Mutt release cycle is "floating" and it is not currently known
where the next release will happen, I propose to patch FreeBSD port
just now.  The patch itself was obtained from the upstream.

--- 1.5.18_4-fix-SEGV.patch begins here ---
diff -urN ./Makefile ../mutt-devel/Makefile
--- ./Makefile	2008-11-05 11:29:47.000000000 +0300
+++ ../mutt-devel/Makefile	2008-11-05 11:30:40.000000000 +0300
@@ -117,7 +117,7 @@
 
 PORTNAME=	mutt-devel
 PORTVERSION=	1.5.18
-PORTREVISION?=	3
+PORTREVISION?=	4
 CATEGORIES+=	mail ipv6
 .if defined(WITH_MUTT_NNTP)
 CATEGORIES+=	news
diff -urN ./files/patch-imap::browse.c-fix-SEGV ../mutt-devel/files/patch-imap::browse.c-fix-SEGV
--- ./files/patch-imap::browse.c-fix-SEGV	1970-01-01 03:00:00.000000000 +0300
+++ ../mutt-devel/files/patch-imap::browse.c-fix-SEGV	2008-11-05 11:32:54.000000000 +0300
@@ -0,0 +1,13 @@
+Index: imap/browse.c
+Obtained from: http://dev.mutt.org/trac/changeset/5401%3Abba92e401440/imap/browse.c?old=5378%3Aac00273b3a88&old_path=imap%2Fbrowse.c
+Bug description: http://dev.mutt.org/trac/ticket/3057
+===================================================================
+--- imap/browse.c
++++ imap/browse.c
+@@ -229,5 +229,5 @@
+   }
+ 
+-  if (!imap_mxcmp(mx.mbox, idata->mailbox))
++  if (idata->ctx && !imap_mxcmp(mx.mbox, idata->mailbox))
+   {
+     state->new = idata->ctx->new;
--- 1.5.18_4-fix-SEGV.patch ends here ---
>Release-Note:
>Audit-Trail:
>Unformatted: