Fread et '\0'

Version imprimable

Fread et '\0'

Bonjour,

Je crée ce sujet parce que je suis confronté à un bug que je ne vois pas du tout comment résoudre, mais je pense savoir ce que c'est et d'où il provient.

J'essaie d'échanger des données binaires (contenues dans des fichiers) avec des sockets UDP. J'utilise donc la fonction fread et j'effectue l'envoi avec sendto à la socket réceptrice ensuite.

Le problème se fait lors de la lecture avec fread. Fread fonctionne et me renvoie bien le nombre de données que je lui demande de lire, mais la variable dans laquelle je veux stocker ces données lues effectue un formatage pour le moins étrange et s'arrête dès qu'il a un '\0. Dit comme ça, ça paraît très logique, le problème est que c'est la première fois que cela m'arrive.

Je vous donne le code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 FILE* file = NULL; file = fopen(namefile, "rb"); char data[1454]; int length = 1; if (file != NULL) { printf("File exists, sending file...\n"); while(feof(file) == 0){ length = fread(data, 1, PACKETSIZE-6, file); printf("Retour de fread : %d\n", length); // Affiche 1454 soit la taille de PACKETSIZE-6 printf("data size : %lu\n", strlen(data)); // Affiche 250 et quelques, le 251ème caractère étant un '\0'
Fread récupère donc bien 1454 caractères, mais mon buffer data n'en récupère pas la totalité. J'ai déjà adopté cette méthode, et ça avait fonctionné. Là je ne comprends vraiment pas. J'ai surement du manquer quelque chose.

Merci d'avance pour votre aide

29/04/2016, 14h01
Médinoc

C'est strlen() qui s'arrête au \0, pas fread().
29/04/2016, 18h21
Endast

Effectivement, je n'ai pas choisi le meilleur exemple pour mon printf.

Pour autant, cela n'explique pas pourquoi data s'arrête au premier \0 lorsque j'affiche son contenu.
29/04/2016, 18h42
SkyZoThreaD

C'est le même problème avec printf... le \0 signifie qu'on est en fin de chaine.
Je suppose qu'on doit pouvoir forcer la longueur de la chaine avec un truc genre %32s à la place de %s si la longeur est de 32 par example.
Ca doit se trouver dans la doc de printf. Ou alors tu peux imprimer caractère par caractère avec une boucle for mais c'est très moche :aie:

edit: non c'est pas ça... tu dois utiliser write() avec le descripteur numéro 1 (pour standard output)

Code:

write(1, txt, 8); // où txt est la chaine et 8 sa longueur ;)
29/04/2016, 19h15
Endast

Merci pour vos réponses au fait :aie:

Le problème avec ta solution SkyZoThreaD, c'est que je ne cherche pas tellement à afficher les données lues, mais à les envoyer à une socket lié à un descripteur (j'ai pas mis la suite du code, elle ne sert à rien tant que le début n'est pas réglé).

Pour précision, à l'envoi (avec un sendto), data contient les 250 premiers caractères corrects puis est complété avec 1200 \0 (car je renseigne la taille que fread renvoie) :

Code:

sendto(desc2, packet, length+6, 0, (struct sockaddr *) client, sizeclient);

L'une des pistes dans mes recherches a été le type de data, mais j'ai tout essayé : tableau de charactère, pointeur sur une chaine... rien n'y fait
29/04/2016, 19h39
SkyZoThreaD

Ah ok! faut être précis :mrgreen: :

Citation:

lorsque j'affiche son contenu.

Effectivement c'est pas bon pour un sendto mais du coup ça devrait pas poser de problème de \0 puisque la taille est controllée.
Pour le type de données, le char* convient très bien ou tout simplement void*.
Maintenant, sans le reste, je ne vois pas d'où ça peut venir... :(
29/04/2016, 20h04
Médinoc

Citation:

Envoyé par SkyZoThreaD

tu dois utiliser write() avec le descripteur numéro 1 (pour standard output)

Code:

write(1, txt, 8); // où txt est la chaine et 8 sa longueur ;)

1? Pourquoi ce nombre magique? Le descripteur de la sortie standard, c'est STDOUT_FILENO, comme son nom l'indique!
29/04/2016, 21h05
SkyZoThreaD

Par ce que la doc que j'ai donnait la valeur en direct :roll:
Merci pour la précision :zoubi:

Voilà le code complet :

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
int transferfile(int desc2, fd_set descripteurs, struct sockaddr_in *client, char* namefile){
 
    int sizeclient = sizeof(*client);
 
    int update = 0;
 
    FILE* file = NULL;
    file = fopen(namefile, "rb");
 
    char data[1454] = "";
    int currentseq = 0;
 
    char packet[PACKETSIZE] = "";
    char numberseqchar[6] = "";
    char numberseqfinal[6] = "";
 
    int length = 1;
 
    if (file != NULL)
    {
        printf("File exists, sending file...\n");
 
        while(feof(file) == 0){
 
            length = fread(data, 1, PACKETSIZE-6, file);
 
            //printf("Retour de fread : %d\n", length);
            //printf("data size : %lu\n", strlen(data));
            //printf("data content : %s\n", data);
 
            currentseq++;
            printf("currentseq : %d\n",currentseq);
 
            charsequence(packet, currentseq, numberseqchar, numberseqfinal);
 
            strcat(packet, data);
 
            sendto(desc2, packet, length+6, 0, (struct sockaddr *) client, sizeclient);
 
            ///////////////////// RECEIVE ACK ///////////////
 
            update = checkack(desc2, client, numberseqfinal);
 
            while(!update)
            {
                sendto(desc2, packet, PACKETSIZE, 0, (struct sockaddr *) client, sizeclient);           //Renvoi en cas de perte
                update = checkack(desc2, client, numberseqfinal);                                           //Check again ACK
            }
 
            /////////////////////////////////////////////////
 
            //memset(numberseqchar, '\0', 6);
            memset(numberseqfinal, 0 , 6);
            memset(data, 0, PACKETSIZE-6);
            memset(packet, 0, PACKETSIZE);
    	}
 
        sendfin(desc2, client, sizeclient);
 
    }
    else
    {
        printf("File does not exist on the server\n");
        exit(-1);
    }
 
    return 0;
 
}

Comme je vous le disais je suis pas sur que ça soit utile, parce que même quand j'essaie d'afficher uniquement data simplement, il se comporte comme une chaîne de caractères.
Je suis dans le cadre de l'échange de données avec socket, y'a donc toute une partie qui concerne la constitution d'un numéro de séquence et la vérification d'ACK.

30/04/2016, 13h09
Médinoc

Citation:

strcat(packet, data);

Boum! Perdu.

Tant que tu persisteras à traiter les data comme une chaîne de caractères, ton programme ne marchera pas. Ce sont des données binaires que tu dois échanger, et si sendto() exige un const char* (c'est le cas sous Windows alors que ça n'a aucun sens), alors tu casteras.