Return-Path: Delivered-To: apmail-httpd-bugs-archive@www.apache.org Received: (qmail 93898 invoked from network); 24 Feb 2005 17:46:11 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur-2.apache.org with SMTP; 24 Feb 2005 17:46:11 -0000 Received: (qmail 93780 invoked by uid 500); 24 Feb 2005 17:46:10 -0000 Delivered-To: apmail-httpd-bugs-archive@httpd.apache.org Received: (qmail 93734 invoked by uid 500); 24 Feb 2005 17:46:10 -0000 Mailing-List: contact bugs-help@httpd.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Reply-To: "Apache HTTPD Bugs Notification List" Delivered-To: mailing list bugs@httpd.apache.org Received: (qmail 93706 invoked by uid 99); 24 Feb 2005 17:46:09 -0000 X-ASF-Spam-Status: No, hits=0.2 required=10.0 tests=NO_REAL_NAME X-Spam-Check-By: apache.org Received: from ajax-1.apache.org (HELO ajax.apache.org) (192.87.106.226) by apache.org (qpsmtpd/0.28) with ESMTP; Thu, 24 Feb 2005 09:46:09 -0800 Received: from ajax.apache.org (ajax.apache.org [127.0.0.1]) by ajax.apache.org (8.12.11/8.12.11) with ESMTP id j1OHk6b6029620 for ; Thu, 24 Feb 2005 18:46:06 +0100 Received: (from nobody@localhost) by ajax.apache.org (8.12.11/8.12.11/Submit) id j1OHk6Y2029617; Thu, 24 Feb 2005 18:46:06 +0100 Date: Thu, 24 Feb 2005 18:46:06 +0100 Message-Id: <200502241746.j1OHk6Y2029617@ajax.apache.org> From: bugzilla@apache.org To: bugs@httpd.apache.org Subject: DO NOT REPLY [Bug 28376] - log overflows with a long request X-Bugzilla-Reason: AssignedTo X-Virus-Checked: Checked X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG� RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND� INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bug.cgi?id=28376 jorton@redhat.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |FIXED ------- Additional Comments From jorton@redhat.com 2005-02-24 18:46 ------- As per comments above this is fixed in current releases. I don't agree with either of the changes in comment 2: - if (n < bytes_handled + len) { + if (n <= bytes_handled + len) { this means that exactly N bytes cannot be stored in the buffer, only N - 1; why is that necessary? - *read = n; + *read = n-1; /* ensure this string is terminated */ (*s)[n-1] = '\0'; since *read is suppoed to be set to the total length of the string *including* the NUL terminator, that isn't correct either. I'm going to mark this closed and suggest opening new a new issue for a new bug. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org For additional commands, e-mail: bugs-help@httpd.apache.org