Autor Tema: Cliente Udp (socket) (Leído 4691 veces)

mojayu · « **en:** Martes 29 de Enero de 2008, 18:36 »

Les comento. TEngo el siguiente codigo que trata de conectarse con un servidor mediante sockets y este le responde indicando la hora.

Código: Text

#include &#60;stdio.h&#62;
#include &#60;stdlib.h&#62;
#include &#60;string.h&#62;
#include &#60;errno.h&#62;
#include &#60;time.h&#62;
#include &#60;WinSock2.h&#62;
 
#define _USE_32BIT_TIME_T
 
#define PUERTO 37
#define TAM_ENVIA 20 //Aunque luego se envia un datagrama vacio
 
int main (int argc, char *argv[])
{
  //Inicializacion de la biblioteca
  int EntDevuelto;
  WORD Version;
  struct WSAData DatWinSock;
  
  Version = MAKEWORD(2,0);
  EntDevuelto = WSAStartup(Version, &DatWinSock);
  if (EntDevuelto != 0)
  { fprintf(stderr, &#34;No hay biblioteca Winsock.&#092;n&#34;);
    exit (-1);
  }
  
  if ((LOBYTE(DatWinSock.wVersion)!=2) || (HIBYTE(DatWinSock.wVersion)!=0))
  { fprintf(stderr, &#34;Version incorrecta.&#092;n&#34;);
    WSACleanup();
    exit (-1);
  }
 
  int result;
  SOCKET SocConexion;
  struct sockaddr_in DirServidor;
  unsigned long int hora;
  char *strDir;
  char *horaLeg;
  char Peticion[TAM_ENVIA];
  int longitud = sizeof(struct sockaddr);
            
  //Para conectar mediante nombre y no por IP
  struct hostent *info;
                
  if (argc&#60;2) {
    printf(&#34;Direccion remota no especificadad. Usando 'localhost'&#092;n&#34;);
    strDir=&#34;127.0.0.1&#34;;
    }
  else strDir=argv[1];
                               
  //Creacion del socket de conexion
  SocConexion = socket(PF_INET, SOCK_DGRAM, 0);
  if (SocConexion == INVALID_SOCKET) {
    fprintf(stderr, &#34;Error %d en socket()n&#34;, WSAGetLastError());
  WSACleanup();
    closesocket(SocConexion);
    exit (-1);
    }
  printf(&#34;Creacion del socket&#34;);
                                                    
  info = gethostbyname(strDir);
                                                        
  //Especificacion de la direccion del servidor
  DirServidor.sin_family = AF_INET;
  DirServidor.sin_port = htons(PUERTO);
  //DirServidor.sin_addr.s_addr = inet_addr(strDir);
  memcpy(&DirServidor.sin_addr.s_addr, info-&#62;h_addr_list[0], sizeof(DirServidor.sin_addr.s_addr));
  /*                                                                
  //Conectar el socket
  result = connect(SocConexion, (struct sockaddr *)&DirServidor, sizeof(DirServidor));
  if (result == -1) {
    fprintf(stderr, &#34;Error %d en connect(): %s&#092;n&#34;, errno, strerror(errno));
    closesocket (SocConexion);
    exit (-1);
    }
 */
  //Enviar datagrama (al ser UDP envia un datagrama vacio previo)
  result = sendto(SocConexion, Peticion, sizeof(Peticion), 0, (struct sockaddr *)&DirServidor, sizeof(DirServidor));
    if (result == SOCKET_ERROR) {
      fprintf(stderr, &#34;Error %d en sendto().&#34;, WSAGetLastError());
    WSACleanup();
      closesocket (SocConexion);
      exit (-1);
    }
  printf(&#34;Datagrama vacio enviado&#34;);
    
  //Recibir respuesta
  result = recvfrom(SocConexion, (char *)&hora, sizeof(hora), 0, (struct sockaddr *)&DirServidor, &longitud);
    if (result == SOCKET_ERROR) {
      fprintf(stderr, &#34;Error %d en rcvfrom().&#34;, WSAGetLastError());
    WSACleanup();
      closesocket(SocConexion);
      exit (-1);
    }
  printf(&#34;Respuesta recivida&#34;);
                    
  //Procesar la respueta
  hora = ntohl(hora);
  hora = hora - 2208988800lu; printf(&#34;Hora procesada&#34;);
  horaLeg = ctime((time_t *)&hora); 
  printf(&#34;Respuesta recibida: %s&#34;, horaLeg);
                              
  //Cerrar la conexion
  WSACleanup();
  closesocket(SocConexion);
  return 1;
}
 
 

Al ejecutarlo sale la tipica ventana de dbe cerrarse el programa justo antes de la ejecucion de la funcion ctime. Al compilar me sale un warning justo en esa linea diciendo que la funcion no es segura... no se si tendra algo que ver... Si me puedieran ayudar se lo agradeceria.

Por cierto, se ejecuta por linea de comandos y para probar se puede conectar a sirio.edv.uniovi.es

mojayu

No he probado porque estoy en la universidad y no puedo pero es posible que el fallo sea por no poner

Código: Text

#define _USE_32BIT_TIME_T
 

en la primera linea del codigo?

Eternal Idol

No podes usar hora como si fuera time_t, al menos en VC++ 8 (.NET 2005) un time_t ocupa 8 bytes y un unsigned long int 4 bytes.

Asi si funciona:

Código: Text

 
time_t dt = hora;
horaLeg = ctime(&dt);

mojayu

Pero el servidor manda la hora en 32 bits. Para eso pongo el primer define

Código: Text

#define _USE_32BIT_TIME_T
 

Ya me funciona. Lo que hice fue ponerlo en la primera linea del codigo.

En algunos ordenadores no me sale ningun warning en ctime, supongo que sea porque tienen una version mas ntigua del sdk, pero una pregunta.
Si hago la modificacion que tu me dijiste, ¿compilara igual en los pc con las versiones antiguas?

Es q yo no se que version tiene el profesor y es en el de él donde tiene que compilar y ejecutarse

Eternal Idol

Si eso (_USE_32BIT_TIME_T) esta definido antes que time.h se procese entonces tu codigo va a funcionar. Es exactamente lo que te dije, normalmente time_t 8 bytes y usando 4 tenes problemas, con ese define time_t tendra 4 bytes (32 bits) y entonces el tamaño sera el mismo en ambas variables.

El warning no sale por el tipo de la variable ya que haces un casting, sale por otra razon que realmente no viene al caso (Microsoft creo nuevas funciones mas seguras para todas las funciones que usen cadenas como parametros).

mojayu

Cierto, pero al cambiar ctime por ctime_s esta requiere otros parametros. Sabes como debe de llmarse para evitar el warning?

Eternal Idol

Cita de: "mojayu"

Cierto, pero al cambiar ctime por ctime_s esta requiere otros parametros. Sabes como debe de llmarse para evitar el warning?

No tiene la mas minima importancia ese warning ... si pones esta linea al principio del codigo no lo mostrara:

Código: Text

 
#pragma warning( disable : 4996 )

PD. No uses ctime_s ya que solo esta disponible desde VC++ 8.0 (.NET 2005) y no funcionara en VC++ 6 o 7.1 (.NET 2003).

mojayu

Muchas gracias por tu ayuda... Tengo otro problema con otro programa, es de socket tb pero esta vez es TCP por si si debo ponerlo en otro hilo avisame y lo hago

Se trata de un cliente y un servidor que se comunican por sockets pero esta vez codificando con xdr. Todo me funciona perfecto pero en el servidor, cuando se desconecta el cliente me sale: violacion de segmento y se cierra cuando deberia de permanecer en espera de mas clientes porque esta en un bucle infinito.

Código: Text

 
#include &#60;sys/socket.h&#62;     // socket, send, recv. . .
#include &#60;netinet/in.h&#62;     // sockaddr_in, htons, htonl, etc.
#include &#60;arpa/inet.h&#62;      // inet_addr
#include &#60;unistd.h&#62;       
#include &#60;stdlib.h&#62;        
#include &#60;stdio.h&#62;          
#include &#60;errno.h&#62;         
#include &#60;string.h&#62;         
#include &#34;tipos.h&#34;
 
int main(int argc,char* argv[])
{
  int socketescm, sockDatos, sockEsc, ret, i, puerto;
  struct sockaddr_in dir;
  struct sockaddr_in dircliente;
  int tamanio ,suma_par ,suma_impar;
  Vector elementos; // es un vector de datos (enteros)  que se ha definido en tipos.x
  FILE *f;
  XDR operacion;
  float resultado;
  int n, numero;
  casos Mens;
 
  printf(&#34;&#092;nServidor en funcionamiento!!&#34;);
  
  // Comprobacion de que se introduzca el puerto por linea de comandos
  if(argc&#60;2)
    {
      printf(&#34;Error. Puerto no especificado.&#34;);
      exit (-1);
    }      
  else
    {
      puerto=atoi(argv[1]);  
      printf(&#34;&#092;nUtilizando el puerto:%d&#092;n&#34;,puerto);
    }
  
  // Creacion socket de escucha  
  sockEsc=socket(PF_INET, SOCK_STREAM, 0);
  dir.sin_family=AF_INET;
  dir.sin_port=htons(puerto);
  dir.sin_addr.s_addr=htonl(INADDR_ANY);
   ret=bind(sockEsc,(struct sockaddr *)&dir, sizeof(dir));
   if (ret==-1)
     {
       perror(&#34;Al asignar direccion&#092;n&#34;);
       close(sockEsc);
       exit(-1);
     }
     
   // Poner socket en modo escucha
   ret=listen(sockEsc, SOMAXCONN);
   if (ret==-1) 
     {
       perror(&#34;Al poner en modo escucha&#34;);
       close(sockEsc);
       exit(-1);
     }
   
   while(1)
     {
       tamanio=sizeof(dircliente);
       printf(&#34;Esperando por clientes........&#092;n&#34;);
       sockDatos = accept(sockEsc,(struct sockaddr *)&dircliente,&tamanio);
       
       if (sockDatos == -1)
   {
     perror(&#34;Error en accept&#092;n&#34;);
     exit(-1);
   }
       printf(&#34;Se ha conectado un cliente&#092;n&#34;);
       
       elementos.Vector_val=NULL;
       f=fdopen(sockDatos,&#34;r+&#34;);
       if(f==NULL)
   {
     printf(&#34;Error al leer los datos&#092;n&#34;);
     exit(-1);
   }
       
       xdrstdio_create(&operacion,f,XDR_DECODE);
       if(xdr_Vector(&operacion,&elementos)!=TRUE)
   {
     printf(&#34;Error al decodificar la estructura&#092;n&#34;);
     exit(-1);
   }
       if(elementos.Vector_len!=0)
   {
     printf(&#34;El numero de elementos es: %d&#092;n&#34;,elementos.Vector_len);
     i=0;
     suma_par=0;
     suma_impar=0;
     while(i&#60;elementos.Vector_len)
       {
          numero = elementos.Vector_val[i];
    if(numero%2==1){
      suma_impar=suma_impar+numero;
    }
    else{
      suma_par=suma_par+numero;
    }
         i++;
       }
     
     printf(&#34;suma_par: %d --- suma_impar: %d&#34;,suma_par,suma_impar);
    
     //ahora codificamos en XDR
     if(suma_impar==0)
       {
         Mens.tipo=1;
         Mens.casos_u.Mensaje=&#34;Error:division por 0&#34;;
       }
     else
       {
         resultado=(float)suma_par/(float)suma_impar;
         Mens.tipo=2;
         Mens.casos_u.Cociente=resultado;
       }
   }// fin 'if'
       else   
   {
      Mens.tipo=1;
      Mens.casos_u.Mensaje=&#34;Error: no has introducido datos&#092;n&#34;;
   }  
       
       f=fdopen(sockDatos,&#34;w+&#34;);
       xdrstdio_create(&operacion,f,XDR_ENCODE);
       if(xdr_casos(&operacion,&Mens)!=TRUE)
   {   
     printf(&#34;Error al codificar los datos&#092;n&#34;);        
   }
       
       fflush(f);
       fclose(f);
 
       if(suma_impar!=0)
   xdr_free((xdrproc_t) xdr_Vector,(char*) elementos.Vector_val);
       
       close(sockDatos);
 
     }// fin 'while'
}//fin 'main'
 
 

Esta vez en programacion sobre UNIX. Creo q el fallo esta en el servidor (pero no descarto nada) y que es por algo de la reserva de la memoria o que no libero algo... Muchas gracias de nuevo

Eternal Idol

Depura el servidor (o adaptalo a Winsock si queres que lo haga yo

) y asi sabras en que punto da el error y la razon de darlo. Despues muevo esta segunda parte a otro hilo nuevo.

mojayu

no manejo muy bien lo de depurar y para pasarlo a winsock deberia tb pasar el cliente y el tipos.x?

Eternal Idol

Cita de: "mojayu"

no manejo muy bien lo de depurar y para pasarlo a winsock deberia tb pasar el cliente y el tipos.x?

Trata de depurarlo, es una habilidad muy pero muy util. Para pasarlo a Winsock y que yo pueda depurarlo (no tengo ganas ni tiempo de hacerlo) tenes que hacer lo mismo que con el programa de UDP, inicializar Winsock y usar las librerias de Windows.

SoloCodigo

Autor Tema: Cliente Udp (socket) (Leído 4691 veces)

mojayu

Cliente Udp (socket)

mojayu

Re: Cliente Udp (socket)

Eternal Idol

Re: Cliente Udp (socket)

mojayu

Re: Cliente Udp (socket)

Eternal Idol

Re: Cliente Udp (socket)

mojayu

Re: Cliente Udp (socket)

Eternal Idol

Re: Cliente Udp (socket)

mojayu

Re: Cliente Udp (socket)

Eternal Idol

Re: Cliente Udp (socket)

mojayu

Re: Cliente Udp (socket)

Eternal Idol

Re: Cliente Udp (socket)