Return-Path: Delivered-To: apmail-httpd-cvs-archive@www.apache.org Received: (qmail 1656 invoked from network); 24 Apr 2008 19:50:31 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 24 Apr 2008 19:50:31 -0000 Received: (qmail 1508 invoked by uid 500); 24 Apr 2008 19:50:32 -0000 Delivered-To: apmail-httpd-cvs-archive@httpd.apache.org Received: (qmail 1317 invoked by uid 500); 24 Apr 2008 19:50:32 -0000 Mailing-List: contact cvs-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: List-Post: List-Id: Delivered-To: mailing list cvs@httpd.apache.org Received: (qmail 1306 invoked by uid 99); 24 Apr 2008 19:50:32 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Apr 2008 12:50:32 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 24 Apr 2008 19:49:56 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 8A5D91A9832; Thu, 24 Apr 2008 12:50:09 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r651370 - /httpd/httpd/branches/2.2.x/STATUS Date: Thu, 24 Apr 2008 19:50:07 -0000 To: cvs@httpd.apache.org From: trawick@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20080424195009.8A5D91A9832@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: trawick Date: Thu Apr 24 12:49:59 2008 New Revision: 651370 URL: http://svn.apache.org/viewvc?rev=651370&view=rev Log: return -ENOGROK Modified: httpd/httpd/branches/2.2.x/STATUS Modified: httpd/httpd/branches/2.2.x/STATUS URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=651370&r1=651369&r2=651370&view=diff ============================================================================== --- httpd/httpd/branches/2.2.x/STATUS (original) +++ httpd/httpd/branches/2.2.x/STATUS Thu Apr 24 12:49:59 2008 @@ -204,6 +204,16 @@ Backport version for 2.2.x of patch: http://people.apache.org/~covener/2.2.x-pr44560.diff +1 covener, niq + trawick: I don't understand the API for the affected array. + Consider uldap_cache_getuserdn(): + Before this change, we returned a new list of ptrs to strings, + with a NULL ptr signifying the end of the list. (Yes, we messed + up and could miss some of the strings.) + After this change, we return a new list of ptrs to strings, + of indeterminate length since NULL pointers are scattered + through the array. So how does the caller know how to walk + through the list? (It seems that we need need to allocate numvals+1, + then store in (*retvals)[j++] when vals[i] != NULL.) * mod_headers: Add merge option to avoid duplicate values within the same header.