[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[GNUnet-SVN] [libmicrohttpd] branch master updated: eliminate two more s
From: |
gnunet |
Subject: |
[GNUnet-SVN] [libmicrohttpd] branch master updated: eliminate two more strcpy()s |
Date: |
Tue, 27 Feb 2018 23:46:25 +0100 |
This is an automated email from the git hooks/post-receive script.
grothoff pushed a commit to branch master
in repository libmicrohttpd.
The following commit(s) were added to refs/heads/master by this push:
new 4507f419 eliminate two more strcpy()s
4507f419 is described below
commit 4507f419bd5417fcd400a3a268f816df480f6e48
Author: Christian Grothoff <address@hidden>
AuthorDate: Tue Feb 27 23:46:23 2018 +0100
eliminate two more strcpy()s
---
src/lib/connection_call_handlers.c | 33 +++++++++++++++++----------------
src/microhttpd/connection.c | 5 +++--
2 files changed, 20 insertions(+), 18 deletions(-)
diff --git a/src/lib/connection_call_handlers.c
b/src/lib/connection_call_handlers.c
index 7bdf8611..17f32dae 100644
--- a/src/lib/connection_call_handlers.c
+++ b/src/lib/connection_call_handlers.c
@@ -275,7 +275,7 @@ MHD_request_handle_read_ (struct MHD_Request *request)
(MHD_REQUEST_INIT == request->state)
? MHD_SC_CONNECTION_CLOSED
: MHD_SC_CONNECTION_RESET_CLOSED,
- (MHD_REQUEST_INIT == request->state)
+ (MHD_REQUEST_INIT == request->state)
? NULL
: _("Socket disconnected while reading
request.\n"));
return;
@@ -284,8 +284,8 @@ MHD_request_handle_read_ (struct MHD_Request *request)
(MHD_REQUEST_INIT == request->state)
? MHD_SC_CONNECTION_CLOSED
: MHD_SC_CONNECTION_READ_FAIL_CLOSED,
- (MHD_REQUEST_INIT == request->state)
- ? NULL
+ (MHD_REQUEST_INIT == request->state)
+ ? NULL
: _("Connection socket is closed due to error
when reading request.\n"));
return;
}
@@ -411,7 +411,7 @@ sendfile_adapter (struct MHD_Connection *connection)
if (0 > ret)
{
const int err = MHD_socket_get_error_();
-
+
if (MHD_SCKT_ERR_IS_EAGAIN_(err))
{
#ifdef EPOLL_SUPPORT
@@ -712,8 +712,9 @@ try_ready_chunked_body (struct MHD_Request *request)
(0 == response->total_size) )
{
/* end of message, signal other side! */
- strcpy (request->write_buffer,
- "0\r\n");
+ memcpy (request->write_buffer,
+ "0\r\n",
+ 3);
request->write_buffer_append_offset = 3;
request->write_buffer_send_offset = 0;
response->total_size = request->response_write_position;
@@ -928,7 +929,7 @@ MHD_request_handle_write_ (struct MHD_Request *request)
MHD_SC_CONNECTION_WRITE_FAIL_CLOSED,
_("Connection was closed while sending
response body.\n"));
return;
- }
+ }
request->write_buffer_send_offset += ret;
MHD_connection_update_last_activity_ (connection);
if (MHD_REQUEST_CHUNKED_BODY_READY != request->state)
@@ -2244,7 +2245,7 @@ socket_start_normal_buffering (struct MHD_Connection
*connection)
MHD_SCKT_OPT_BOOL_ cork_val = 0;
socklen_t param_size = sizeof (cork_val);
#endif /* TCP_CORK */
-
+
mhd_assert(NULL != connection);
#if defined(TCP_CORK)
/* Allow partial packets */
@@ -2521,7 +2522,7 @@ call_request_handler (struct MHD_Request *request)
if (NULL != request->response)
return; /* already queued a response */
- if (NULL == (action =
+ if (NULL == (action =
daemon->rc (daemon->rc_cls,
request,
request->url,
@@ -2573,7 +2574,7 @@ process_request_body (struct MHD_Request *request)
(available >= 2) )
{
size_t i;
-
+
/* skip new line at the *end* of a chunk */
i = 0;
if ( ('\r' == buffer_head[i]) ||
@@ -2904,14 +2905,14 @@ connection_update_event_loop_info (struct
MHD_Connection *connection)
{
struct MHD_Daemon *daemon = connection->daemon;
struct MHD_Request *request = &connection->request;
-
+
/* Do not update states of suspended connection */
if (connection->suspended)
return; /* States will be updated after resume. */
#ifdef HTTPS_SUPPORT
{
struct MHD_TLS_Plugin *tls;
-
+
if ( (NULL != (tls = daemon->tls_api)) &&
(tls->update_event_loop_info (tls->cls,
connection->tls_cs,
@@ -3079,7 +3080,7 @@ MHD_request_handle_idle_ (struct MHD_Request *request)
{
#ifdef HTTPS_SUPPORT
struct MHD_TLS_Plugin *tls;
-
+
if ( (NULL != (tls = daemon->tls_api)) &&
(! tls->idle_ready (tls->cls,
connection->tls_cs)) )
@@ -3460,7 +3461,7 @@ MHD_request_handle_idle_ (struct MHD_Request *request)
case MHD_REQUEST_FOOTERS_SENT:
{
struct MHD_Response *response = request->response;
-
+
if (MHD_HTTP_PROCESSING == response->status_code)
{
/* After this type of response, we allow sending another! */
@@ -3474,8 +3475,8 @@ MHD_request_handle_idle_ (struct MHD_Request *request)
socket_start_no_buffering_flush (connection);
else
socket_start_normal_buffering (connection);
-
- if (NULL != response->termination_cb)
+
+ if (NULL != response->termination_cb)
{
response->termination_cb (response->termination_cb_cls,
MHD_REQUEST_TERMINATED_COMPLETED_OK,
diff --git a/src/microhttpd/connection.c b/src/microhttpd/connection.c
index 0afbe2ac..39253a9f 100644
--- a/src/microhttpd/connection.c
+++ b/src/microhttpd/connection.c
@@ -1210,8 +1210,9 @@ try_ready_chunked_body (struct MHD_Connection *connection)
(0 == response->total_size) )
{
/* end of message, signal other side! */
- strcpy (connection->write_buffer,
- "0\r\n");
+ memcpy (connection->write_buffer,
+ "0\r\n",
+ 3);
connection->write_buffer_append_offset = 3;
connection->write_buffer_send_offset = 0;
response->total_size = connection->response_write_position;
--
To stop receiving notification emails like this one, please contact
address@hidden
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [GNUnet-SVN] [libmicrohttpd] branch master updated: eliminate two more strcpy()s,
gnunet <=