Reprise du message précédent :

buf = malloc(len);
if (buf) {
    memset(buf, 0, len);
}

buf = calloc(1, len);

authchunks = malloc(sockopt_len);
memset(authchunks, 0, sizeof(sockopt_len));

self-reply: ah non le calloc et malloc+memset(0) sont bien équivalents, le problème c'est qu'il y a un malloc+memset(!0) qui est remplacé dans le commit:

-        data->saved_message.data = malloc(inl);
-        memcpy(data->saved_message.data, in, inl);
+        data->saved_message.data = calloc(1, inl);

oops: http://freshbsd.org/commit/openbsd [...] 3eeccb4d63

(VC++ 5.0 date de 1997, yep, 17 ans)

buf = malloc(len);
if (buf) {
    memset(buf, 0, len);
}

buf = calloc(1, len);

authchunks = malloc(sockopt_len);
memset(authchunks, 0, sizeof(sockopt_len));

 
en gros je pense que le risque c'est de générer une exception quand on écrit dans le buffer alloué avec calloc et de rendre vulnérable donc le code à une timing-attack ( http://en.wikipedia.org/wiki/Timing_attack ).
je ne vois que ça...
 

-        data->saved_message.data = malloc(inl);
-        memcpy(data->saved_message.data, in, inl);
+        data->saved_message.data = calloc(1, inl);

OK, je suis allé trop loin dans mon analyse  

Oui, en fait, il y avait un bon remplacement de data->saved_message.data = malloc(inl); par data->saved_message.data = calloc(1, inl); et simplement la ligne suivante, memcpy(data->saved_message.data, in, inl); a sauté en même temps que la première, sans que celui qui faisait la modification s'en rende compte.
 
A+,

Le remplacement est pas considéré "bon" puisqu'il a été reverté, je présume qu'ils ont fait tourner un script pour trouver/remplacer des paires de malloc+memset par calloc. Si tu regardes les commits ils n'ont pas (encore) l'air de remplacer tous les malloc par calloc, uniquement les malloc + bzero.

Sur les malloc seuls ils ne font qu'enlever les casts.

Je suis preneur de conseils, leurs autres modèles (de cette marque) ont moins bonne réputation

Oui, je ne voyais pas le remplacement introduire d'erreur, c'est ce que j'ai voulu dire par bon, mais en y réfléchissant à deux fois, c'était un remplacement inutile, puisque le memcpy qui suit réécrit toute la zone, la RAZ ne sert donc à rien.
 
A+,
 

Les smiley dans la réponse rapide, c'est nouveau ?

non, non, ça a toujours été là, tu remarques à peine ?

http://forum.hardware.fr/hfr/Discu [...] 8785_1.htm

si j'ai bien compris les petites lignes, c'est les vieux windows dont il parle.

quelqu'un pourrait m'envoyer un mail sur trainoo.com et un autre sur nraynaud.com svp ? (séparément, en me le disant ici, n'importe quoi comme préfixe)

dion > super, merci
 
 
(j'en reviens pas, j'ai réussi à faire un acte d'administration qui marche , bon il m'a fallu quelques tempt pour me rendre compte que j'avais niqué le mail de 2 domaines quand même)

Les nouveaux Windows sont pas plus POSIX que les anciens, donc pour le moment il parle de tous les windows. Genre http://anoncvs.estpak.ee/cgi-bin/c [...] 5e50418066 AFAIK il n'y a toujours pas de SIGPIPE sous Windows. Et ils ont dégagé le backend CryptoAPI.

Sur plus long terme, ça va être comme OpenSSH je présume (si ca fizzle pas), ils commencent par nettoyer et moderniser le code en BSD-only, puis ils portent petit à petit.

Hum ok.
 
Quelqu'un pour mater mon CV en mp svp ?

fépété

Oh putain le machin feeligo tout moche.

nraynal, si je peux t'aider pour le CV, no soucÿ, balance

si ça peut aider ma candidature, je tiens à signaler que j'ai des ancêtres qui dealaient du diamant à Anvers si tu vois ce que je veux dire ...

 
Vazy

Quand il t'a mp qu'il voulait voir ton pénis c'était pas ça qu'il recherchait

Si t'as leur nom, probable que mon oncle Moché les connaisse

J'ai mal à ma ram.

en tout cas il y a une question qui fait l'unanimité: mon CV est moche.

gimme gimme gimme  

 
Même dans les pires pays du Moyen Orient on demande pas aux femmes lapidées d'apporter les pierres.

Ca y est, ils ont trouvé qqun qui veut nous acheter.
 
Mais pas vraiment tout ce qui était à vendre,
 
Et on sait toujours pas qui c'est

Des chinois ou des quatari selon le figaro

Javascript, Date() :  

 
Parce que fuck you.

 
 
C'est merveilleux

un classique, idem dans le Calendar Java ...

 
Fais gaffe aux DST dans le passé, hein

Ou même dans le présent en Arizona