client C server JAVA

Version imprimable

Salut à tous,
je (crise) un peu, j'ai une application client/serveur en JAVA qui tourne très bien,
la partie de cette appli fonctionne en TCP :

voici la fonction qui recupere les messages en byte par byte en java

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
	/**
         * reception de bytes
         * @return
         * @throws IOException
         */
	public byte[] readBytes() throws IOException {
		InputStream in = s.getInputStream();
		DataInputStream dis = new DataInputStream(in);
 
		int len = dis.readInt();
		byte[] data = new byte[len];
 
		if(len>0) {
			dis.readFully(data);
		}
		return data;
	}

celle ci fonctionne très bien en JAVA,

mais en C , c'est tout autre chose, mon client se connecte sur mon serveur, jusqu'ici très bien, mais rien ne parvient a être envoyé, j'utilise la fonction :

Code:

1
2
  char *buffer2 = "testnom\r\n";
  send(sock,buffer2,sizeof(buffer),0);

j'ai des tests sur chacunes des fonctions importante en C qui me retourne un message d'erreur en cas d'echec, et toutes les fonctions "fonctionnent parfaitment".

Vous voyez autre chose ?

mon code pour l'instant assez sale en C

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
#include <stdio.h>
#include <stdlib.h>
#include <signal.h>
 
#include <netdb.h>
#include <netinet/in.h>
#include <sys/socket.h>
 
#define SERVEURNAME "127.0.0.1"
 
int sock = -1;
 
void
main (void)
{
 
  char *server_name = SERVEURNAME;
  struct sockaddr_in serverSockAddr;
  struct hostent *serverHostEnt;
  long hostAddr;
  long status;
  char buffer[512];
 
 
 
  bzero (&serverSockAddr, sizeof (serverSockAddr));
  hostAddr = inet_addr (SERVEURNAME);
  if ((long) hostAddr != (long) -1)
    bcopy (&hostAddr, &serverSockAddr.sin_addr, sizeof (hostAddr));
  else
    {
      serverHostEnt = gethostbyname (SERVEURNAME);
      if (serverHostEnt == NULL)
	{
	  printf ("ServerHost ERROR\n");
	  return (-1);
	}
 
      bcopy (serverHostEnt->h_addr, &serverSockAddr.sin_addr,
	     serverHostEnt->h_length);
    }
  serverSockAddr.sin_port = htons (50987);
  serverSockAddr.sin_family = AF_INET;
 
/* creation de la socket */
 
  sock = createTCP();
/* requete de connexion */
  if (connect (sock,
	       (struct sockaddr *) &serverSockAddr,
	       sizeof (serverSockAddr)) < 0)
    {
      printf ("Connection ERROR\n");
      return (-1);
    }
  printf("connexion success\n");
/* envoie de donne et reception */
  char *buffer2 = "testnom\0";
  send(sock,buffer2,sizeof(buffer+1),0);
  printf("emmssion du nom terminée");
  read (sock, buffer, 512);
  printf (buffer);
/* fermeture de la connection */
 
  close (sock);
}
 
 
 
//file mainfun.c
int createTCP() {
  if ((sock = socket (AF_INET, SOCK_STREAM, 0)) < 0)
    {
      printf ("creation socket ERROR\n");
      return (-1);
    }
  return sock;
}

01/06/2012, 13h24
matafan

"sizeof(buffer)" est la taille d'un pointeur, pas la taille de ta chaine. Ce que tu veux envoyer c'est strlen(buffer) (ou strlen(buffer) + 1 si tu veux envoyer le 0 final).

ouep je viens de m'en rendre compte aussi, maintenant j'ai recuperer une belle fonction

Code:

1
2
3
4
5
6
7
  char *text = "Hello world\n";
   int n = send (sock, text, strlen (text), 0);
   if (n >= 0)
   {
      /* debug */
      printf ("%d octet%s sent\n", n, n != 1 ? "s" : "");
   }

mais j'obtiens mainentenant une erreur dans ma focntion de reception dans mon serveur JAVA

execption java.loang.outOfMemoryError: Java heap space

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
	/**
         * reception de bytes
         * @return
         * @throws IOException
         */
	public byte[] readBytes() throws IOException {
		InputStream in = s.getInputStream();
		DataInputStream dis = new DataInputStream(in);
		//ci dessous ligne pointée par l'erreur
		>>>>> int len = dis.readInt();
		byte[] data = new byte[len];
 
		if(len>0) {
			dis.readFully(data);
		}
		return data;
	}

01/06/2012, 14h49
Bktero

Buffer est un tableau statique : char buffer[512]; donc sizeof(buffer) devrait rendre 512. La fonction envoie donc 513 octets. Il est quand même mieux de mettre strlen()pour ne pas envoyer ce qui ne fait pas partie de la chaine de texte.

Je ne suis pas assez à l'aise avec ces classes Java pour trouver l'origine du problème. Mais question : comment fais-tu pointeur le InputStream vers le socket réseau ?

yess, pour le strlen c'est régler, merci ;)

pour le inputStream :

Code:

1
2
3
4
5
 
...
serveursocket = new ServerSocket(50987);
...
start();

Code:

1
2
3
4
5
6
7
public void run() {
 
        try {
        	while(true){
s = serveursocket.accept();
sendBytes("Exemple pour devellopez.net".getBytes());
...

01/06/2012, 17h02
matafan

Apparemment le code de ton serveur s'attend à recevoir en premier un entier de 4 octets, qu'il interprète comme le nombre de caractères à lire. Or ton code C envoie directement les caractères. Le serveur lit donc ces caractères, qui lorsqu'il sont interprétés comme un int donnent un très grand nombre (dans len). Quand tu alloues un tableau de len bytes, forcément, tu exploses le heap.

Bref il faut que ton code C commence par envoyer le nombre de caractères qu'il va écrire, sous la forme d'un entier au format réseau (donc, utiliser htonl() pour le convertir avant de l'envoyer).
01/06/2012, 19h41
marco911

Merci, c'est exactement l'erreur que je fais je pense, l me reste plus qu'a faire ce que tu viens de dire en C, et ça c'est une autre paire de manche :mrgreen:
01/06/2012, 20h30
marco911
humm j'arrive pas a faire marcher ce saligot !
Code:

1 2 3 4 5 6 7 8 char *buffer[1000]; char **toSend="test"; ... sprintf(buffer,"%d",htonl(strlen(toSend)+4)); strcat(buffer,toSend); int n = send(sock,buffer,strlen(buffer),0);
htonl(strlen(toSend)+4) car j'ajoute à ce que je veux envoyer 4 octets qui correspondent à l'unsigned int.. j'ai toujours pas compris j'imagine ?! :calim2:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
char *buffer2[1000];
char **toSend = "test\0";
sprintf(buffer2,"%d",htonl(strlen(toSend)));
//strcat(buffer2,toSend);
 
 
 
//printf("I wrote %d\n",strlen(toSend)+9);
 
int n = send(sock,htonl(strlen(toSend)),strlen(htonl(strlen(toSend))),0);
 
if(n>=0){
printf("%d octets sent\n",n,n != 1 ? "s" : "");
}
 
n = send(sock,toSend,strlen(toSend),0);
 
if(n>=0){
printf("%d octets sent\n",n,n != 1 ? "s" : "");
}

apres reflexion,....mais ca ne fonctionne toujours pas ....

04/06/2012, 09h02
Bktero

Code:

char **toSend = "test\0";

==>

Citation:

warning: initialization from incompatible pointer type|

"Arrête de ton char, Ben Hur !"

Code:

printf("%d octets sent\n",n,n != 1 ? "s" : "");

La chaine formatée n'attends qu'un seul "paramètre", tu lui en donnes deux.

La fonction htonl me semble mal utilisée : http://man.developpez.com/man3/htonl.3.php

Sans oublier les warning de paramètres de types erronés pour strlen et sprintf.
04/06/2012, 13h13
matafan

La taille des données doit être envoyée tel quel comme une donnée binaire, pas sous forme d'une chaine ascii. Il faut envoyer directement le retour de htonl(), sans passer par un sprintf.

Et la taille des données à envoyer, c'est 4 (sizeof(uint32_t), qui est le type de la valeur retournée par htonl()), pas strlen() de je ne sais quoi.

super !merci, tout fonctionne nikel
Code:

1 2 3 4 5 6 7 8 9 10 int receiveTCP (int sock, char **recvd) { int a; recv (sock, &a, 4, 0); int na = ntohl (a); int nb = recv (sock, recvd, sizeof (recvd), 0); recvd[na + 1] = '\0'; }
maintenant je vais a la chasse au strlen pour enlever toutces warning !