Este codigo, parece estar correcto , pero no me funciona

[Biohazard]

Hola , estoy aprendiendo programacion en redes , y se me ocurrio , que era una buena idea , para empezar hacer un codigo fuente, de un escaner de puertos, que por su forma de funcionamiento , no son muy complejos ...

Pues bien , este es el codigo , que he hecho , y es correcto, sin embargo, si abro el programa , nunca me aparece ningun puerto abierto, y eso es imposible, asi que si alguien tiene una idea , de que es lo que falla ... Gracias , de antemano , quiero decir que estoy programando sobre windows , asi que empleo las librerias winsock ...

#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
#include <winsock.h>

int main(int argc, char *argv[])
{
WSADATA d;
WORD winsock;
int s;
struct sockaddr_in *h;
int mp, MP; //mp min port ; MP max port;

if(argc<2)
{
printf("Uso : %s <ip> [min port] [max port]\n",argv[0]);
exit(1);
}
else if (argc==4)
{
mp=atoi(argv[2]);
MP=atoi(argv[3]);
}
else
{
mp = 1;
MP = 1024;
}
winsock = MAKEWORD(1,1);
if(WSAStartup(winsock,&d)!=0){fprintf(stderr,"WSAStartup ha fallado.\n");exit(1);}

s=socket(PF_INET,SOCK_STREAM,0);

if(s==-1){fprintf(stderr,"Socket() ha fallado\n");exit(1);}
system("CLS");
printf("Escaneando %s de %d a %d de TCP\n",argv[1],mp,MP);
printf("----------------------------------------------\n");

int i;
for(i=mp;i<=MP;i++)
{
h->sin_family = AF_INET;
h->sin_port = htons(i);
h->sin_addr.s_addr = inet_addr(argv[1]);
memset(&(h->sin_zero),'\0',;

if((connect(s,(struct sockaddr *)&h,sizeof(struct sockaddr)))==0){printf("Puerto %d -TCP- ABIERTO\n",i);}
}

printf("Escaner Completado\n");
printf("------------------\n");

closesocket(s);
WSACleanup();

system("PAUSE");
system("CLS");
return 0;
}

Este es el codigo, a ver si alguien sabe que es lo que falla...

[cheroky]

El error principal es que usas un puntero del tipo struct sockaddr_in y este puntero no apunta a ningun bloque valido, no referencia a un objeto de ese tipo, por tanto o bien declaras un variable del tipo mencionado o bien reservas memoria para dicho puntero. Otro error esta en la comprobacion del numero de argumentos.

El codigo con las modificaciones.. puede que haya algun error mas..

[Biohazard]

Gracias por las respuesta , pero no me sirvio de nada , sin embargo , eso si es verdad , es muy lento , asi que voy a mirar eso de los raw ... Si sabeis de algun tutorial de programacion redes , que este explicado para windows ... agradeceria que lo posteaseis... Muchisimas gracias, por la ayuda ...

[cheroky]

No entiendo que la respuesta no te sirviera de nada ¿?. Compilado con mingw y pasandole un router como host ip la salida es:

[Biohazard]

pues no me sirvio de nada, corregi , lo de la struct sockaddr_in *h sustituyendolo por struct sockaddr_in h, y lo compile con Dev-Cpp, y escaneé el ip local 127.0.0.1 y aun con el navegador abierto , no tenia ni el puerto 80 abierto ...

[rir3760]