Project

General

Profile

Actions

Bug #1512

closed

mod_deflate: filesize check is too weak, move to new logstyle

Added by stbuehler about 16 years ago. Updated over 15 years ago.

Status:
Fixed
Priority:
Normal
Category:
mod_deflate
Target version:
ASK QUESTIONS IN Forums:

Description

I changed the filesize check to check for the end of the file, not the current offset (abs_offset):


-       if (abs_offset > sce->st.st_size) {
+       if (c->file.length + c->file.start > sce->st.st_size) {

And i replaced the log_error_write calls with ERROR.


Files

1512.patch.gz (4.03 KB) 1512.patch.gz attach to trac glen, 2008-01-18 10:52
Actions #1

Updated by stbuehler about 16 years ago

I'd really liked to add an attachment, but i got


Submission rejected as potential spam (IP *.*.*.* blacklisted by sc.surbl.org, SpamBayes determined spam probability of 0.00%, Content contained blacklisted patterns)
Actions #3

Updated by glen about 16 years ago

403 Forbidden - Unknown commit object

Actions #4

Updated by Anonymous about 16 years ago

Yeah, sry. As i keep rebasing to the trunk, history and therefore commit ids are changing.

Current (i will tag them somehow so they won't vanish again):

http://repo.or.cz/w/lighttpd.git?a=commitdiff;h=68a2a5758854492d56d547fb691883bb01d61865

http://repo.or.cz/w/lighttpd.git?a=commitdiff_plain;h=68a2a5758854492d56d547fb691883bb01d61865

Actions #5

Updated by glen about 16 years ago

  • Status changed from New to Assigned

damn. i lost it again :)
i will next time download it before new commits

Actions #6

Updated by Anonymous about 16 years ago

This time the Trac WikiFormatting got me (i didn't pull yet ;-)).

http://repo.or.cz/w/lighttpd.git?a=commitdiff;h=68a2a5758854492d56d547fb691883bb01d61865

http://repo.or.cz/w/lighttpd.git?a=commitdiff_plain;h=68a2a5758854492d56d547fb691883bb01d61865

So next try, with preview and link testing ;-)

(It would be cool to be able to edit ticket comments, i just forget to preview every time^^)

Actions #7

Updated by glen about 16 years ago

  • Status changed from Assigned to Fixed
  • Resolution set to fixed

Committed revision r2055

Actions

Also available in: Atom