Re: RE : [sympa-fr] Plantage wwsympa.fcgi..process a

[David Verdin <adresse@cachée>]

Bonjour,

Nous avons corrigé ce comportement dans la branche de stabilisation de 
Sympa.
Vous trouverez le correctif en suivant le lien ci-dessous :
http://sourcesup.cru.fr/cgi/viewvc.cgi?view=rev&root=sympa&revision=6379

Le comportement est le suivant : si, dans les 50 premières lignes non 
vides, aucun email valide n'est trouvé, on laisse tomber et on logge une 
erreur, dont le listmaster sera prévenu.
Ce comportement est appliqué aux fichiers locaux et aux fichiers distants.

Cordialement,

David Verdin

Le 25/03/2010 11:41, adresse@cachée a écrit :
> Bonjour
> Nous avons trouve la nature du probleme , nos tests sur la v6 sont fait 
> volontairement sur un serveur sous dimensionne .
> Lorsque nous essayons d'iclure un fichier corrompu ( trops d'adresses mails 
> invalides ) =>  dans notre cas un fichier excel de plusieurs megas au lieu 
> d'un fichier texte , wwsympa.fcgi  monte a 100% et reste bloque ( plusieurs 
> heures ) ..
> Nous avons resolu le probleme en detectant un seuil d'erreurs dans la 
> procedure _include_users_file  ( invalid email) et en sortant des ce seuil 
> atteint ... il semble ne pas y avoir de controle sur le format du fcihier a 
> inclure dans cette procedure ...un fichier corrompu peut donc planter sympa 
> .. ( le fichier include est place dans le repertoire shared des listes donc 
> depose par l'administrateur de la liste qui est donc amene a faire des 
> erreurs )
> Cordialement
>
> mathias Brethous .
> ________________________________________
> De : adresse@cachée [adresse@cachée] de la part de 
> adresse@cachée [adresse@cachée]
> Date d'envoi : mercredi 24 mars 2010 09:59
> À : adresse@cachée
> Objet : [sympa-fr] Plantage wwsympa.fcgi..process a 
> 100%..on_the_fly_sync_include()
>
> Bonjour
> Nous utilisons ( en pre production , nous ne sommes pas encore passé en 
> production ) et testons votre version 6.0.1 avec tous les patchs et 
> correctifs recences a ce jour .(qui a ce jour ont corriges tous les bugs que 
> nous avons rencontres ).
>
> Nous sommes confrontes a un bug majeur , dans la procedure 
> List::on_the_fly_sync_include() Synchronizing list members...
>
> sur Inclusion d'un fichier (include_file)
> sympa reste bloque sur la procedure suivante ( process wwsympa.fcgi monte a 
> 100% , plus rien ne reponds )...
> Mar 24 09:53:43 nopfc022 wwsympa[13377]: List::on_the_fly_sync_include() 
> Synchronizing list members...
>
> avant que nous cherchions de notre cote ce qui provoque cela , avez vous eu 
> ce probleme sur cette version ?(nous n'avons pas ce probleme sur notre vielle 
> version 5.2.4).
> ps =>  le fichier include est volontairement corrompu , inclu des adresses 
> invalides et caracteres exotiques afin de tester un mauvais comportement 
> utilisateur , l'erreur vient sans doute de cela mais cela ne devrait pas planter 
> l'interface .
> Sauf erreur de ma part , je n'ai pas trouvé trace de ce bug dans vos forums 
> ni correctifs .
> Merci pour votre eventuelle reponse / correction
>
> Cordialement
> Mathias Brethous .
>
> *********************************
> This message and any attachments (the "message") are confidential and 
> intended solely for the addressees.
> Any unauthorised use or dissemination is prohibited.
> Messages are susceptible to alteration.
> France Telecom Group shall not be liable for the message if altered, changed 
> or falsified.
> If you are not the intended addressee of this message, please cancel it 
> immediately and inform the sender.
> ********************************
>
>
>
> *********************************
> This message and any attachments (the "message") are confidential and 
> intended solely for the addressees.
> Any unauthorised use or dissemination is prohibited.
> Messages are susceptible to alteration.
> France Telecom Group shall not be liable for the message if altered, changed 
> or falsified.
> If you are not the intended addressee of this message, please cancel it 
> immediately and inform the sender.
> ********************************
>
>
>    

--
David Verdin
Comité réseau des universités