aniani/elinks
1
0
Fork 0
mirror of https://github.com/rkd77/elinks.git synced 2024-12-04 14:46:47 -05:00
Code

[data] Premature optimisation. One memacpy less for base64 encoded data.

Browse Source
This commit is contained in:
Witold Filipczyk 2020-09-19 19:30:47 +02:00
parent 01c232c4e1
commit 267948b2e8
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/protocol/data.c
View File

@ -116,7 +116,7 @@ data_protocol_handler(struct connection *conn)
{
struct uri *uri = conn->uri;
struct cache_entry *cached = get_cache_entry(uri);
unsigned char *data_start, *data;
unsigned char *data_start, *data = NULL;
int base64 = 0;
int decodedlen = 0;
int datalen;
@ -137,14 +137,8 @@ data_protocol_handler(struct connection *conn)
/* Allocate the data string because URI decoding will possibly modify
* it. */
datalen = uri->datalen - (data_start - uri->data);
data = memacpy(data_start, datalen);
if (!data) {
abort_connection(conn, connection_state(S_OUT_OF_MEM));
return;
}
if (base64) {
unsigned char *decoded = base64_decode_bin(data, datalen, &decodedlen);
unsigned char *decoded = base64_decode_bin(data_start, datalen, &decodedlen);
if (!decoded) {
abort_connection(conn, connection_state(S_OUT_OF_MEM));
@ -153,6 +147,12 @@ data_protocol_handler(struct connection *conn)
mem_free_set(&data, decoded);
} else {
data = memacpy(data_start, datalen);
if (!data) {
abort_connection(conn, connection_state(S_OUT_OF_MEM));
return;
}
decode_uri(data);
/* Use strlen() to get the correct decoded length */
decodedlen = strlen(data);