Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id A08274196F0 for ; Tue, 27 Apr 2010 16:36:45 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -2.89 X-Spam-Level: X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5 tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01] autolearn=ham Received: from olra.theworths.org ([127.0.0.1]) by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jTaWx4i09jbz; Tue, 27 Apr 2010 16:36:43 -0700 (PDT) Received: from yoom.home.cworth.org (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id 9B9CF431FC1; Tue, 27 Apr 2010 16:36:43 -0700 (PDT) Received: by yoom.home.cworth.org (Postfix, from userid 1000) id 4CA3E568DEC; Tue, 27 Apr 2010 16:36:43 -0700 (PDT) From: Carl Worth To: Dirk Hohndel , notmuch@notmuchmail.org Subject: Re: [PATCH 1/2] Fix SEGV in _thread_cleanup_author if author ends with ', ' In-Reply-To: <1272410963-2106-2-git-send-email-hohndel@infradead.org> References: <1272410963-2106-1-git-send-email-hohndel@infradead.org> <1272410963-2106-2-git-send-email-hohndel@infradead.org> User-Agent: Notmuch/0.3-7-g2baa576 (http://notmuchmail.org) Emacs/23.1.1 (i486-pc-linux-gnu) Date: Tue, 27 Apr 2010 16:36:37 -0700 Message-ID: <8739ygsc1m.fsf@yoom.home.cworth.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Apr 2010 23:36:45 -0000 --=-=-= On Tue, 27 Apr 2010 16:29:22 -0700, Dirk Hohndel wrote: > Admittedly, an author name ending in ',' guarantees this is spam, and > indeed this was triggered by a spam email, but that doesn't mean we > shouldn't handle this case correctly. > We now check that there is actually a component of the name (presumably > the first name) after the comma in the author name. Thanks. This is pushed out to the 0.3.x branch, and I'll merge it into master shortly. -Carl --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iD8DBQFL13UF6JDdNq8qSWgRAhejAJ9SNXli6H034c4uB+7pLRGJmyWg9ACfUlxp 3V5gE5NElCtagJKKRMD+las= =mFqM -----END PGP SIGNATURE----- --=-=-=--