client-server prog not working and further questions

client.c
reset; gcc -g -Wall -ansi -pedantic client.c -o tt

Code:

#include <stdio.h> #include <string.h> #include <sys/types.h> #include <sys/socket.h> #include <netdb.h> #include <netinet/in.h> #include <arpa/inet.h> #include <stdlib.h> #include <signal.h> #include <unistd.h> int main(void) { int fd, ret, nread; struct sockaddr_in addr; char buffer[50]="aa\nbb\ncc\ndd\nee\n"; socklen_t addrlen; char * pch; strcat(buffer,"LST\n"); strcat(buffer,"555\n"); if((fd=socket(AF_INET,SOCK_DGRAM,0))==-1) exit(1); memset((void*)&addr,(int)'\0',sizeof(addr)); addr.sin_family=AF_INET; inet_pton(AF_INET, "127.0.0.1", &(addr.sin_addr)); /*addr.sin_addr.s_addr=htonl(INADDR_ANY); * ???*/ addr.sin_port=htons(9000); printf("strlen(buffer) =%d",strlen(buffer)); ret=sendto(fd,buffer, strlen(buffer),0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); pch = strtok (buffer,"\n"); while (pch != NULL) { printf ("strtok = %s\n",pch); pch = strtok (NULL,"\n"); } return 0; while(1) { addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); ret=sendto(fd,buffer,nread,0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); } }

server.c
reset; gcc -g -Wall -ansi -pedantic server.c -o tt

Code:

#include <stdio.h> #include <string.h> #include <sys/types.h> #include <sys/socket.h> #include <netdb.h> #include <netinet/in.h> #include <arpa/inet.h> #include <stdlib.h> #include <signal.h> #include <unistd.h> int main(void) { int fd, ret, nread; struct sockaddr_in addr; char buffer[128]; socklen_t addrlen; char * pch; if((fd=socket(AF_INET,SOCK_DGRAM,0))==-1) exit(1); memset((void*)&addr,(int)'\0',sizeof(addr)); addr.sin_family=AF_INET; inet_pton(AF_INET, "127.0.0.1", &(addr.sin_addr)); /*addr.sin_addr.s_addr=htonl(INADDR_ANY); * ???*/ addr.sin_port=htons(9000); ret=bind(fd,(struct sockaddr*)&addr,sizeof(addr)); if(ret==-1)exit(1); addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); pch = strtok (buffer,"\n"); while (pch != NULL) { printf ("strtok = %s\n",pch); pch = strtok (NULL,"\n"); } return 0; while(1) { addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); ret=sendto(fd,buffer,nread,0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); } }

hi,
i have those pieces of code of server client code. they are not working for some reason , smb knows why?

No idea why it does not work; but, why are you using a size of 128 for buffer in the recvfrom call?
Doing this results in unpredictable results; this could include your compliant.

Code:

char buffer[50]="aa\nbb\ncc\ndd\nee\n";

You said it was not working, but you didn't tell us how it is not working. Compiler errors? Crashes? Hangs? No output? Strange output? Details about your problem help us

If you properly formatted and indented your code, it might be a little easier to follow the flow. Take your client code for example:

Code:

#include <stdio.h> #include <string.h> #include <sys/types.h> #include <sys/socket.h> #include <netdb.h> #include <netinet/in.h> #include <arpa/inet.h> #include <stdlib.h> #include <signal.h> #include <unistd.h> int main(void) { int fd, ret, nread; struct sockaddr_in addr; char buffer[50]="aa\nbb\ncc\ndd\nee\n"; socklen_t addrlen; char * pch; strcat(buffer,"LST\n"); strcat(buffer,"555\n"); if((fd=socket(AF_INET,SOCK_DGRAM,0))==-1) exit(1); memset((void*)&addr,(int)'\0',sizeof(addr)); addr.sin_family=AF_INET; inet_pton(AF_INET, "127.0.0.1", &(addr.sin_addr)); /*addr.sin_addr.s_addr=htonl(INADDR_ANY); * ???*/ addr.sin_port=htons(9000); printf("strlen(buffer) =%d",strlen(buffer)); ret=sendto(fd,buffer, strlen(buffer),0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); pch = strtok (buffer,"\n"); while (pch != NULL) { printf ("strtok = %s\n",pch); pch = strtok (NULL,"\n"); } return 0; while(1) { addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); ret=sendto(fd,buffer,nread,0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); } }

Take a look at line 42. What do you think happens there? When do you think lines 44-52 execute?

If you're competent enough in C to write socket code, you should be able to print useful error messages when something goes wrong (look into the perror function) instead of just exiting silently, and to print other debug messages to see the status of your program as it runs. You should also be able to use a debugger to trace your program's execution and examine the state of everything as it runs and when it fails. A debugger is an invaluable tool for a programmer.

first of all i want to be able to send the message from client to the server. i have more questions about my code, but 1st of all i want to solve that problem.
my code is very similar to this one: Writing a simple UDP client/server in a Unix environment
i get and error on the client side: sendto: Invalid argument

client.c

Code:

#include <stdio.h> #include <string.h> #include <sys/types.h> #include <sys/socket.h> #include <netdb.h> #include <netinet/in.h> #include <arpa/inet.h> #include <stdlib.h> #include <signal.h> #include <unistd.h> int main(void) { int fd, ret, nread; struct sockaddr_in addr; char buffer[128]="aa\nbb\ncc\ndd\nee\n"; socklen_t addrlen; char * pch; strcat(buffer,"LST\n"); strcat(buffer,"555\n"); if((fd=socket(AF_INET,SOCK_DGRAM,0))==-1) exit(1); memset((void*)&addr,(int)'\0',sizeof(addr)); addr.sin_family=AF_INET; inet_pton(AF_INET, "127.0.0.1", &(addr.sin_addr)); /*addr.sin_addr.s_addr=htonl(INADDR_ANY); * ???*/ addr.sin_port=htons(9000); printf("strlen(buffer) =%d\n",strlen(buffer)); ret=sendto(fd,buffer, 128,0,(struct sockaddr*)&addr,addrlen); if(ret==-1) { perror("sendto"); exit(1); } pch = strtok (buffer,"\n"); while (pch != NULL) { printf ("strtok = %s\n",pch); pch = strtok (NULL,"\n"); } return 0; while(1) { addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); ret=sendto(fd,buffer,nread,0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); } }

server.c

Code:

#include <stdio.h> #include <string.h> #include <sys/types.h> #include <sys/socket.h> #include <netdb.h> #include <netinet/in.h> #include <arpa/inet.h> #include <stdlib.h> #include <signal.h> #include <unistd.h> int main(void) { int fd, ret, nread; struct sockaddr_in addr; char buffer[128]; socklen_t addrlen; char * pch; if((fd=socket(AF_INET,SOCK_DGRAM,0))==-1) exit(1); memset((void*)&addr,(int)'\0',sizeof(addr)); addr.sin_family=AF_INET; inet_pton(AF_INET, "127.0.0.1", &(addr.sin_addr)); /*addr.sin_addr.s_addr=htonl(INADDR_ANY); * ???*/ addr.sin_port=htons(9000); ret=bind(fd,(struct sockaddr*)&addr,sizeof(addr)); if(ret==-1)exit(1); addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1) { perror("recvfrom"); exit(1); } pch = strtok (buffer,"\n"); while (pch != NULL) { printf ("strtok = %s\n",pch); pch = strtok (NULL,"\n"); } return 0; while(1) { addrlen=sizeof(addr); nread=recvfrom(fd,buffer,128,0,(struct sockaddr*)&addr,&addrlen); if(nread==-1)exit(1); ret=sendto(fd,buffer,nread,0,(struct sockaddr*)&addr,addrlen); if(ret==-1)exit(1); } }

Point to the line that set the value of addrlen; note, this needs to be done before you use the value in most programs.
I have no resent experience with sockets; its been so long ago, I have forgotten all I know about them.

Tim S.

Tim has the right of it. A few printf statements printing important variables, or using a debugger, and you would have figured that out on your own much sooner. You would do well to implement those techniques when developing and debugging your programs.

Also, you need to check all of your functions for error codes, and use perror for each one. That includes inet_pton, socket and bind, plus all instances of recvfrom and sendto.