Project

General

Profile

Actions

Bug #1850

closed

Too waste memory in mod_proxy.c

Added by ycheng almost 16 years ago. Updated almost 16 years ago.

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

Description

See:
http://redmine.lighttpd.net/repositories/entry/lighttpd/branches/lighttpd-1.4.x/src/mod_proxy.c
Function:
static int proxy_demux_response(server *srv, handler_ctx *hctx)
Line:
lien 655: buffer_prepare_append(hctx->response, hctx->response->used + b);

Too waste memory.
int buffer_prepare_append(buffer *b, size_t size) expands the content of the buffer, not set. The argument size is a "incremental" quantity, not the "absolute" quantity. See the srouce.
So line 655 should be changed to:
buffer_prepare_append(hctx->response, b);
I have had a test, and it works OK.

Actions #1

Updated by icy almost 16 years ago

  • Target version set to 1.4.21
  • Pending changed from No to Yes

Looked over it and is valid, good find.
Thanks for looking over the source ycheng!

Actions #2

Updated by stbuehler almost 16 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset r2365.

Actions

Also available in: Atom