From 95349b79a5a646736c706fe19645181146ee9486 Mon Sep 17 00:00:00 2001 From: wangzhengquan <wangzhengquan85@126.com> Date: 星期二, 20 十月 2020 16:29:59 +0800 Subject: [PATCH] update --- src/socket/net_mod_server_socket.c | 44 ++++++++++++++++++++++++++++++-------------- 1 files changed, 30 insertions(+), 14 deletions(-) diff --git a/src/socket/net_mod_server_socket.c b/src/socket/net_mod_server_socket.c index 51e10e9..9448a39 100644 --- a/src/socket/net_mod_server_socket.c +++ b/src/socket/net_mod_server_socket.c @@ -43,6 +43,7 @@ socklen_t clientlen; struct sockaddr_storage clientaddr; char portstr[32]; + if (signal(SIGPIPE, SIG_IGN) == SIG_ERR) err_msg(errno, "signal"); //shmModSocket = new ShmModSocket; sprintf(portstr, "%d", port); @@ -58,7 +59,7 @@ /* Wait for listening/connected descriptor(s) to become ready */ pool.ready_set = pool.read_set; pool.nready = select(pool.maxfd + 1, &pool.ready_set, NULL, NULL, NULL); - + // LoggerFactory::getLogger()->debug("select return \n"); /* If listening descriptor ready, add new client to pool */ if (FD_ISSET(listenfd, &pool.ready_set)) { @@ -135,11 +136,12 @@ request_head = NetModSocket::decode_request_head(request_head_bs); if(request_head.content_length > max_buf) { - buf = realloc(buf, request_head.content_length); - max_buf = request_head.content_length; - if(buf == NULL) { - LoggerFactory::getLogger()->error(errno, "process_client realloc"); + + if( (buf = realloc(buf, request_head.content_length)) == NULL) { + LoggerFactory::getLogger()->error(errno, "NetModServerSocket::process_client realloc buf"); exit(1); + } else { + max_buf = request_head.content_length; } } @@ -148,7 +150,12 @@ } if(request_head.mod == REQ_REP) { +// printf("server response===========\n"); + memcpy(response_head.host, request_head.host, NI_MAXHOST); + response_head.port = request_head.port; + response_head.key = request_head.key; if(shmModSocket.sendandrecv_unsafe(buf, request_head.content_length, request_head.key, &recv_buf, &recv_size) != 0) { + response_head.code = 1; response_head.content_length = 0; if( rio_writen(connfd, NetModSocket::encode_response_head(response_head), NET_MODE_RESPONSE_HEAD_LENGTH) != NET_MODE_RESPONSE_HEAD_LENGTH ) @@ -160,8 +167,14 @@ response_buf_size = NET_MODE_RESPONSE_HEAD_LENGTH + recv_size; if(max_response_buf < response_buf_size) { - buf = (char *)realloc(response_buf, response_buf_size); - max_response_buf = response_buf_size; + if( (response_buf = (char *)realloc(response_buf, response_buf_size)) == NULL ) { + LoggerFactory::getLogger()->error(errno, "NetModServerSocket::process_client realloc response_buf"); + exit(1); + } else { + max_response_buf = response_buf_size; + } + + } memcpy(response_buf, NetModSocket::encode_response_head(response_head), NET_MODE_RESPONSE_HEAD_LENGTH); memcpy(response_buf + NET_MODE_RESPONSE_HEAD_LENGTH, recv_buf, recv_size); @@ -176,11 +189,11 @@ } else if(request_head.mod == BUS) { if(request_head.topic_length > max_topic_buf) { - topic_buf = realloc(topic_buf, request_head.topic_length); - max_topic_buf = request_head.topic_length; - if(topic_buf == NULL) { - LoggerFactory::getLogger()->error(errno, "process_client realloc"); + if( (topic_buf = realloc(topic_buf, request_head.topic_length)) == NULL ) { + LoggerFactory::getLogger()->error(errno, "NetModServerSocket::process_client realloc topic_buf"); exit(1); + } else { + max_topic_buf = request_head.topic_length; } } @@ -200,7 +213,7 @@ { int i, connfd; //rio_t *rio; - + Logger * logger = LoggerFactory::getLogger(); for (i = 0; (i <= pool.maxi) && (pool.nready > 0); i++) { @@ -211,10 +224,13 @@ { pool.nready--; if(process_client(connfd) != 0) { - LoggerFactory::getLogger()->debug("===server close client %d\n", connfd); - Close(connfd); //line:conc:echoservers:closeconnfd + + + Close(connfd); FD_CLR(connfd, &pool.read_set); pool.clientfd[i] = -1; + logger->debug("===server close client %d\n", connfd); + // printf("===server close client %d\n", connfd); } } -- Gitblit v1.8.0