diff options
author | Stefan Bühler <stbuehler@web.de> | 2015-02-08 12:37:10 +0000 |
---|---|---|
committer | Stefan Bühler <stbuehler@web.de> | 2015-02-08 12:37:10 +0000 |
commit | 6afad87d2ed66a48cda2a7c01dbcc59023774b73 (patch) | |
tree | 9b371e94cf911630c5794f5b4ff6e3edd6439a3b /src/mod_setenv.c | |
parent | 3521be8b8599ae2cc12361c8f600fc58a473de91 (diff) | |
download | lighttpd-git-6afad87d2ed66a48cda2a7c01dbcc59023774b73.tar.gz |
fix buffer, chunk and http_chunk API
* remove unused structs and functions
(buffer_array, read_buffer)
* change return type from int to void for many functions,
as the return value (indicating error/success) was never checked,
and the function would only fail on programming errors and not on
invalid input; changed functions to use force_assert instead of
returning an error.
* all "len" parameters now are the real size of the memory to be read.
the length of strings is given always without the terminating 0.
* the "buffer" struct still counts the terminating 0 in ->used,
provide buffer_string_length() to get the length of a string in a
buffer.
unset config "strings" have used == 0, which is used in some places
to distinguish unset values from "" (empty string) values.
* most buffer usages should now use it as string container.
* optimise some buffer copying by "moving" data to other buffers
* use (u)intmax_t for generic int-to-string functions
* remove unused enum values: UNUSED_CHUNK, ENCODING_UNSET
* converted BUFFER_APPEND_SLASH to inline function (no macro feature
needed)
* refactor: create chunkqueue_steal: moving (partial) chunks into another
queue
* http_chunk: added separate function to terminate chunked body instead of
magic handling in http_chunk_append_mem().
http_chunk_append_* now handle empty chunks, and never terminate the
chunked body.
From: Stefan Bühler <stbuehler@web.de>
git-svn-id: svn://svn.lighttpd.net/lighttpd/branches/lighttpd-1.4.x@2975 152afb58-edef-0310-8abb-c4023f1b3aa9
Diffstat (limited to 'src/mod_setenv.c')
-rw-r--r-- | src/mod_setenv.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/mod_setenv.c b/src/mod_setenv.c index ad916091..60e9b555 100644 --- a/src/mod_setenv.c +++ b/src/mod_setenv.c @@ -185,8 +185,8 @@ URIHANDLER_FUNC(mod_setenv_uri_handler) { ds_dst = data_string_init(); } - buffer_copy_string_buffer(ds_dst->key, ds->key); - buffer_copy_string_buffer(ds_dst->value, ds->value); + buffer_copy_buffer(ds_dst->key, ds->key); + buffer_copy_buffer(ds_dst->value, ds->value); array_insert_unique(con->request.headers, (data_unset *)ds_dst); } @@ -199,8 +199,8 @@ URIHANDLER_FUNC(mod_setenv_uri_handler) { ds_dst = data_string_init(); } - buffer_copy_string_buffer(ds_dst->key, ds->key); - buffer_copy_string_buffer(ds_dst->value, ds->value); + buffer_copy_buffer(ds_dst->key, ds->key); + buffer_copy_buffer(ds_dst->value, ds->value); array_insert_unique(con->environment, (data_unset *)ds_dst); } |