| Message ID | 20260112015047.2184003-1-liujian56@huawei.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | [v2] rpc: create valid fd to pass libtirpc validation | expand |
Hi! Sigh, for some reasons the CI didn't work this time..patchwork seems to be too slow again. On Mon Jan 12, 2026 at 2:50 AM CET, Liu Jian wrote: > From: Liu Jian via ltp <ltp@lists.linux.it> > > The testcase(rpc_svc_destroy, rpc_svcfd_create, rpc_xprt_register, > rpc_xprt_unregister) was failing due to an invalid fd, which > caused libtirpc's internal validation to reject the operation. > This change ensures a valid socket fd is created and can pass the > validation checks in libtirpc. > Do you have a practical example why it was failing and where it was failing? And eventually arch, kernel version, etc. It would be nice to have it inside the description.
Hi all, [ Cc Steve and linux-nfs ] > The testcase(rpc_svc_destroy, rpc_svcfd_create, rpc_xprt_register, > rpc_xprt_unregister) was failing due to an invalid fd, which > caused libtirpc's internal validation to reject the operation. > This change ensures a valid socket fd is created and can pass the > validation checks in libtirpc. + I would also like to know more details about the failure (libtirpc version, or do you still use Sun RPC from old glibc, distro, arch, ...). Because while we have some problems with some of these tests, I'm not sure if connecting to 127.0.0.1 is a valid approach. And if it is, should it be used in more tests? Also rpc_svc_destroy_stress.c and rpc_svcfd_create_limits.c use svcfd_create(). Below are few minor implementation details. First, maybe add a common header, with code in the function which would be used? But it can be done later (RPC test code is really awful, more duplicity will not make it worse). > Signed-off-by: Liu Jian <liujian56@huawei.com> > --- > v2: Fix a compilation error on Alpine. > .../rpc_svc_destroy.c | 27 +++++++++++++++++++ > .../rpc_svcfd_create.c | 26 ++++++++++++++++++ > .../rpc_xprt_register.c | 25 +++++++++++++++++ > .../rpc_xprt_unregister.c | 25 +++++++++++++++++ > 4 files changed, 103 insertions(+) > diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c > index 22e560843..b9240ccba 100644 > --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c > @@ -30,6 +30,12 @@ > #include <time.h> > #include <rpc/rpc.h> > +#include <unistd.h> > +#include <sys/socket.h> > +#include <netinet/in.h> > +#include <arpa/inet.h> > +#include <string.h> > + > //Standard define > #define PROCNUM 1 > #define VERSNUM 1 > @@ -43,6 +49,27 @@ int main(void) > int test_status = 1; //Default test result set to FAILED > int fd = 0; > SVCXPRT *svcr = NULL; > + struct sockaddr_in server_addr; > + > + fd = socket(AF_INET, SOCK_DGRAM, 0); > + if (fd < 0) { > + printf("socket creation failed"); Maybe fprintf(stderr, ...) or perror()? > + return test_status; > + } > + > + memset(&server_addr, 0, sizeof(server_addr)); Maybe just: struct sockaddr_in server_addr = {0}; instead of memset() ? > + server_addr.sin_family = AF_INET; > + server_addr.sin_port = htons(9001); > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > + printf("inet_pton failed"); > + close(fd); > + return test_status; > + } > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { > + printf("connect failed"); > + close(fd); Funny that all testsuites don't close fd, but not closing single fd is not that problematic. Kind regards, Petr > + return test_status; > + } > //First of all, create a server > svcr = svcfd_create(fd, 0, 0); > diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c > index f0d89ba48..ea4418961 100644 > --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c > @@ -29,6 +29,11 @@ > #include <stdlib.h> > #include <time.h> > #include <rpc/rpc.h> > +#include <unistd.h> > +#include <sys/socket.h> > +#include <netinet/in.h> > +#include <arpa/inet.h> > +#include <string.h> > //Standard define > #define PROCNUM 1 > @@ -43,6 +48,27 @@ int main(void) > int test_status = 1; //Default test result set to FAILED > int fd = 0; > SVCXPRT *svcr = NULL; > + struct sockaddr_in server_addr; > + > + fd = socket(AF_INET, SOCK_DGRAM, 0); > + if (fd < 0) { > + printf("socket creation failed"); > + return test_status; > + } > + > + memset(&server_addr, 0, sizeof(server_addr)); > + server_addr.sin_family = AF_INET; > + server_addr.sin_port = htons(9001); > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > + printf("inet_pton failed"); > + close(fd); > + return test_status; > + } > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { > + printf("connect failed"); > + close(fd); > + return test_status; > + } > //create a server > svcr = svcfd_create(fd, 0, 0); > diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c > index b10a1ce5e..a40dad7fe 100644 > --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c > @@ -31,6 +31,10 @@ > #include <rpc/rpc.h> > #include <sys/types.h> > #include <sys/socket.h> > +#include <unistd.h> > +#include <netinet/in.h> > +#include <arpa/inet.h> > +#include <string.h> > //Standard define > #define PROCNUM 1 > @@ -45,6 +49,27 @@ int main(void) > int test_status = 1; //Default test result set to FAILED > SVCXPRT *svcr = NULL; > int fd = 0; > + struct sockaddr_in server_addr; > + > + fd = socket(AF_INET, SOCK_DGRAM, 0); > + if (fd < 0) { > + printf("socket creation failed"); > + return test_status; > + } > + > + memset(&server_addr, 0, sizeof(server_addr)); > + server_addr.sin_family = AF_INET; > + server_addr.sin_port = htons(9001); > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > + printf("inet_pton failed"); > + close(fd); > + return test_status; > + } > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { > + printf("connect failed"); > + close(fd); > + return test_status; > + } > //create a server > svcr = svcfd_create(fd, 1024, 1024); > diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c > index 3b6130eaa..5ac51de41 100644 > --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c > @@ -31,6 +31,10 @@ > #include <rpc/rpc.h> > #include <sys/types.h> > #include <sys/socket.h> > +#include <unistd.h> > +#include <netinet/in.h> > +#include <arpa/inet.h> > +#include <string.h> > //Standard define > #define PROCNUM 1 > @@ -49,6 +53,27 @@ int main(int argn, char *argc[]) > int test_status = 1; //Default test result set to FAILED > SVCXPRT *svcr = NULL; > int fd = 0; > + struct sockaddr_in server_addr; > + > + fd = socket(AF_INET, SOCK_DGRAM, 0); > + if (fd < 0) { > + printf("socket creation failed"); > + return test_status; > + } > + > + memset(&server_addr, 0, sizeof(server_addr)); > + server_addr.sin_family = AF_INET; > + server_addr.sin_port = htons(9001); > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > + printf("inet_pton failed"); > + close(fd); > + return test_status; > + } > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { > + printf("connect failed"); > + close(fd); > + return test_status; > + } > //create a server > svcr = svcfd_create(fd, 1024, 1024);
> -----Original Message----- > From: Petr Vorel [mailto:pvorel@suse.cz] > Sent: Tuesday, January 13, 2026 8:26 PM > To: liujian (CE) <liujian56@huawei.com> > Cc: ltp@lists.linux.it; andrea.cervesato@suse.com; linux-nfs@vger.kernel.org; > Steve Dickson <SteveD@redhat.com> > Subject: Re: [LTP] [PATCH v2] rpc: create valid fd to pass libtirpc validation > > Hi all, > > [ Cc Steve and linux-nfs ] > > > The testcase(rpc_svc_destroy, rpc_svcfd_create, rpc_xprt_register, > > rpc_xprt_unregister) was failing due to an invalid fd, which caused > > libtirpc's internal validation to reject the operation. > > This change ensures a valid socket fd is created and can pass the > > validation checks in libtirpc. > > + I would also like to know more details about the failure (libtirpc > + version, or > do you still use Sun RPC from old glibc, distro, arch, ...). > tested this on Debian 12 and Debian 10. I observed that the latest libtirpc (1.3.7) code has the same logic. svcfd_create ----svc_fd_create(fd, sendsize, recvsize); --------... --------getsockname(fd, (struct sockaddr *)(void *)&ss, &slen) < 0) --------... --------getpeername(fd, (struct sockaddr *)(void *)&ss, &slen) --------... The above is some code flow of `svcfd_create()` in libtirpc, where both `getsockname()` and `getpeername()` require the file descriptor (fd) to be a valid and connected socket. Additionally, in glibc, there are no similar operations in `svcfd_create()`, so there is no issue for this test case. > Because while we have some problems with some of these tests, I'm not > sure if connecting to 127.0.0.1 is a valid approach. And if it is, should it be > used in more tests? Also rpc_svc_destroy_stress.c and > rpc_svcfd_create_limits.c use svcfd_create(). > For a UDP socket, connect() does not trigger any message transmission and should have no other side effects. What do you all think? However, there are no `rpc_svcfd_create_limits` or `rpc_svc_destroy_stress` tests in `runtest/net.rpc_tests`? > Below are few minor implementation details. First, maybe add a common > header, with code in the function which would be used? But it can be done > later (RPC test code is really awful, more duplicity will not make it worse). > > > Signed-off-by: Liu Jian <liujian56@huawei.com> > > --- > > v2: Fix a compilation error on Alpine. > > .../rpc_svc_destroy.c | 27 +++++++++++++++++++ > > .../rpc_svcfd_create.c | 26 ++++++++++++++++++ > > .../rpc_xprt_register.c | 25 +++++++++++++++++ > > .../rpc_xprt_unregister.c | 25 +++++++++++++++++ > > 4 files changed, 103 insertions(+) > > > diff --git > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svc_destroy/rpc_svc_destroy.c > > b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svc_destroy/rpc_svc_destroy.c > > index 22e560843..b9240ccba 100644 > > --- > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svc_destroy/rpc_svc_destroy.c > > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_cre > > +++ atedestroy_svc_destroy/rpc_svc_destroy.c > > @@ -30,6 +30,12 @@ > > #include <time.h> > > #include <rpc/rpc.h> > > > +#include <unistd.h> > > +#include <sys/socket.h> > > +#include <netinet/in.h> > > +#include <arpa/inet.h> > > +#include <string.h> > > + > > //Standard define > > #define PROCNUM 1 > > #define VERSNUM 1 > > @@ -43,6 +49,27 @@ int main(void) > > int test_status = 1; //Default test result set to FAILED > > int fd = 0; > > SVCXPRT *svcr = NULL; > > + struct sockaddr_in server_addr; > > + > > + fd = socket(AF_INET, SOCK_DGRAM, 0); > > + if (fd < 0) { > > + printf("socket creation failed"); > Maybe fprintf(stderr, ...) or perror()? > > > + return test_status; > > + } > > + > > + memset(&server_addr, 0, sizeof(server_addr)); > > Maybe just: > struct sockaddr_in server_addr = {0}; > instead of memset() ? > > > + server_addr.sin_family = AF_INET; > > + server_addr.sin_port = htons(9001); > > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > > + printf("inet_pton failed"); > > + close(fd); > > + return test_status; > > + } > > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) > < 0) { > > + printf("connect failed"); > > + close(fd); > Funny that all testsuites don't close fd, but not closing single fd is not that > problematic. > > Kind regards, > Petr > > > + return test_status; > > + } > > > //First of all, create a server > > svcr = svcfd_create(fd, 0, 0); > > diff --git > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svcfd_create/rpc_svcfd_create.c > > b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svcfd_create/rpc_svcfd_create.c > > index f0d89ba48..ea4418961 100644 > > --- > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_created > > estroy_svcfd_create/rpc_svcfd_create.c > > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_cre > > +++ atedestroy_svcfd_create/rpc_svcfd_create.c > > @@ -29,6 +29,11 @@ > > #include <stdlib.h> > > #include <time.h> > > #include <rpc/rpc.h> > > +#include <unistd.h> > > +#include <sys/socket.h> > > +#include <netinet/in.h> > > +#include <arpa/inet.h> > > +#include <string.h> > > > //Standard define > > #define PROCNUM 1 > > @@ -43,6 +48,27 @@ int main(void) > > int test_status = 1; //Default test result set to FAILED > > int fd = 0; > > SVCXPRT *svcr = NULL; > > + struct sockaddr_in server_addr; > > + > > + fd = socket(AF_INET, SOCK_DGRAM, 0); > > + if (fd < 0) { > > + printf("socket creation failed"); > > + return test_status; > > + } > > + > > + memset(&server_addr, 0, sizeof(server_addr)); > > + server_addr.sin_family = AF_INET; > > + server_addr.sin_port = htons(9001); > > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > > + printf("inet_pton failed"); > > + close(fd); > > + return test_status; > > + } > > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) > < 0) { > > + printf("connect failed"); > > + close(fd); > > + return test_status; > > + } > > > //create a server > > svcr = svcfd_create(fd, 0, 0); > > diff --git > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_register/rpc_xprt_register.c > > b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_register/rpc_xprt_register.c > > index b10a1ce5e..a40dad7fe 100644 > > --- > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_register/rpc_xprt_register.c > > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_reg > > +++ unreg_xprt_register/rpc_xprt_register.c > > @@ -31,6 +31,10 @@ > > #include <rpc/rpc.h> > > #include <sys/types.h> > > #include <sys/socket.h> > > +#include <unistd.h> > > +#include <netinet/in.h> > > +#include <arpa/inet.h> > > +#include <string.h> > > > //Standard define > > #define PROCNUM 1 > > @@ -45,6 +49,27 @@ int main(void) > > int test_status = 1; //Default test result set to FAILED > > SVCXPRT *svcr = NULL; > > int fd = 0; > > + struct sockaddr_in server_addr; > > + > > + fd = socket(AF_INET, SOCK_DGRAM, 0); > > + if (fd < 0) { > > + printf("socket creation failed"); > > + return test_status; > > + } > > + > > + memset(&server_addr, 0, sizeof(server_addr)); > > + server_addr.sin_family = AF_INET; > > + server_addr.sin_port = htons(9001); > > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > > + printf("inet_pton failed"); > > + close(fd); > > + return test_status; > > + } > > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) > < 0) { > > + printf("connect failed"); > > + close(fd); > > + return test_status; > > + } > > > //create a server > > svcr = svcfd_create(fd, 1024, 1024); diff --git > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_unregister/rpc_xprt_unregister.c > > b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_unregister/rpc_xprt_unregister.c > > index 3b6130eaa..5ac51de41 100644 > > --- > > a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunre > > g_xprt_unregister/rpc_xprt_unregister.c > > +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_reg > > +++ unreg_xprt_unregister/rpc_xprt_unregister.c > > @@ -31,6 +31,10 @@ > > #include <rpc/rpc.h> > > #include <sys/types.h> > > #include <sys/socket.h> > > +#include <unistd.h> > > +#include <netinet/in.h> > > +#include <arpa/inet.h> > > +#include <string.h> > > > //Standard define > > #define PROCNUM 1 > > @@ -49,6 +53,27 @@ int main(int argn, char *argc[]) > > int test_status = 1; //Default test result set to FAILED > > SVCXPRT *svcr = NULL; > > int fd = 0; > > + struct sockaddr_in server_addr; > > + > > + fd = socket(AF_INET, SOCK_DGRAM, 0); > > + if (fd < 0) { > > + printf("socket creation failed"); > > + return test_status; > > + } > > + > > + memset(&server_addr, 0, sizeof(server_addr)); > > + server_addr.sin_family = AF_INET; > > + server_addr.sin_port = htons(9001); > > + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { > > + printf("inet_pton failed"); > > + close(fd); > > + return test_status; > > + } > > + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) > < 0) { > > + printf("connect failed"); > > + close(fd); > > + return test_status; > > + } > > > //create a server > > svcr = svcfd_create(fd, 1024, 1024);
diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c index 22e560843..b9240ccba 100644 --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svc_destroy/rpc_svc_destroy.c @@ -30,6 +30,12 @@ #include <time.h> #include <rpc/rpc.h> +#include <unistd.h> +#include <sys/socket.h> +#include <netinet/in.h> +#include <arpa/inet.h> +#include <string.h> + //Standard define #define PROCNUM 1 #define VERSNUM 1 @@ -43,6 +49,27 @@ int main(void) int test_status = 1; //Default test result set to FAILED int fd = 0; SVCXPRT *svcr = NULL; + struct sockaddr_in server_addr; + + fd = socket(AF_INET, SOCK_DGRAM, 0); + if (fd < 0) { + printf("socket creation failed"); + return test_status; + } + + memset(&server_addr, 0, sizeof(server_addr)); + server_addr.sin_family = AF_INET; + server_addr.sin_port = htons(9001); + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { + printf("inet_pton failed"); + close(fd); + return test_status; + } + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { + printf("connect failed"); + close(fd); + return test_status; + } //First of all, create a server svcr = svcfd_create(fd, 0, 0); diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c index f0d89ba48..ea4418961 100644 --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_createdestroy_svcfd_create/rpc_svcfd_create.c @@ -29,6 +29,11 @@ #include <stdlib.h> #include <time.h> #include <rpc/rpc.h> +#include <unistd.h> +#include <sys/socket.h> +#include <netinet/in.h> +#include <arpa/inet.h> +#include <string.h> //Standard define #define PROCNUM 1 @@ -43,6 +48,27 @@ int main(void) int test_status = 1; //Default test result set to FAILED int fd = 0; SVCXPRT *svcr = NULL; + struct sockaddr_in server_addr; + + fd = socket(AF_INET, SOCK_DGRAM, 0); + if (fd < 0) { + printf("socket creation failed"); + return test_status; + } + + memset(&server_addr, 0, sizeof(server_addr)); + server_addr.sin_family = AF_INET; + server_addr.sin_port = htons(9001); + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { + printf("inet_pton failed"); + close(fd); + return test_status; + } + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { + printf("connect failed"); + close(fd); + return test_status; + } //create a server svcr = svcfd_create(fd, 0, 0); diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c index b10a1ce5e..a40dad7fe 100644 --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_register/rpc_xprt_register.c @@ -31,6 +31,10 @@ #include <rpc/rpc.h> #include <sys/types.h> #include <sys/socket.h> +#include <unistd.h> +#include <netinet/in.h> +#include <arpa/inet.h> +#include <string.h> //Standard define #define PROCNUM 1 @@ -45,6 +49,27 @@ int main(void) int test_status = 1; //Default test result set to FAILED SVCXPRT *svcr = NULL; int fd = 0; + struct sockaddr_in server_addr; + + fd = socket(AF_INET, SOCK_DGRAM, 0); + if (fd < 0) { + printf("socket creation failed"); + return test_status; + } + + memset(&server_addr, 0, sizeof(server_addr)); + server_addr.sin_family = AF_INET; + server_addr.sin_port = htons(9001); + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { + printf("inet_pton failed"); + close(fd); + return test_status; + } + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { + printf("connect failed"); + close(fd); + return test_status; + } //create a server svcr = svcfd_create(fd, 1024, 1024); diff --git a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c index 3b6130eaa..5ac51de41 100644 --- a/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c +++ b/testcases/network/rpc/rpc-tirpc/tests_pack/rpc_suite/rpc/rpc_regunreg_xprt_unregister/rpc_xprt_unregister.c @@ -31,6 +31,10 @@ #include <rpc/rpc.h> #include <sys/types.h> #include <sys/socket.h> +#include <unistd.h> +#include <netinet/in.h> +#include <arpa/inet.h> +#include <string.h> //Standard define #define PROCNUM 1 @@ -49,6 +53,27 @@ int main(int argn, char *argc[]) int test_status = 1; //Default test result set to FAILED SVCXPRT *svcr = NULL; int fd = 0; + struct sockaddr_in server_addr; + + fd = socket(AF_INET, SOCK_DGRAM, 0); + if (fd < 0) { + printf("socket creation failed"); + return test_status; + } + + memset(&server_addr, 0, sizeof(server_addr)); + server_addr.sin_family = AF_INET; + server_addr.sin_port = htons(9001); + if (inet_pton(AF_INET, "127.0.0.1", &server_addr.sin_addr) <= 0) { + printf("inet_pton failed"); + close(fd); + return test_status; + } + if (connect(fd, (struct sockaddr *)&server_addr, sizeof(server_addr)) < 0) { + printf("connect failed"); + close(fd); + return test_status; + } //create a server svcr = svcfd_create(fd, 1024, 1024);