haiwen/libsearpc
1
0
Fork 0
mirror of https://github.com/haiwen/libsearpc.git synced 2025-09-19 17:25:06 +00:00
Code Issues Releases Wiki Activity

Fixed a bug in named pipe transport.

Browse Source
This commit is contained in:
Shuai Lin
2016-06-28 11:21:34 +08:00
parent 48a9c675ed
commit 25ce2aeceb
3 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
.travis.yml
View File

@@ -13,6 +13,6 @@ before_install:
script: script:
- ./configure - ./configure
- make -j8 - make -j8
- make check - make check -j8
notifications: notifications:
email: false email: false

6
lib/searpc-named-pipe-transport.c
View File

@@ -220,11 +220,17 @@ static void* named_pipe_client_handler(void *arg)
g_debug ("start to serve on pipe client\n"); g_debug ("start to serve on pipe client\n");
while (1) { while (1) {
len = 0;
if (pipe_read_n(connfd, &len, sizeof(uint32_t)) < 0) { if (pipe_read_n(connfd, &len, sizeof(uint32_t)) < 0) {
g_warning("failed to read rpc request size: %s", strerror(errno)); g_warning("failed to read rpc request size: %s", strerror(errno));
break; break;
} }
if (len == 0) {
g_debug("EOF reached, pipe connection lost");
break;
}
while (bufsize < len) { while (bufsize < len) {
bufsize *= 2; bufsize *= 2;
buf = realloc(buf, bufsize); buf = realloc(buf, bufsize);

1
tests/searpc.c
View File

@@ -370,7 +370,6 @@ test_searpc__pipe_simple_call (void)
g_free (result); g_free (result);
/* error should return */ /* error should return */
result = NULL;
result = searpc_client_call__string (client_with_pipe_transport, "get_substring", &error, result = searpc_client_call__string (client_with_pipe_transport, "get_substring", &error,
2, "string", "hello", "int", 10); 2, "string", "hello", "int", 10);
cl_assert (error->message); cl_assert (error->message);