Ticket #68 (new defect)

Opened 6 years ago

Last modified 3 months ago

Codes retour non testés

Reported by: jerome Assigned to: anonymous
Priority: normal Milestone: alternc-1.0
Component: Shell-scripts et binaires Version:
Severity: major Keywords:
Cc:

Description

Dans /usr/lib/alternc/src/mem_add, par exemple, le code retour de la fonction "mkdir" n'est pas testé, et le binaire retourne zéro même si le répertoire n'a pas été créé.

Cette absence de test de code retour semble être également présent dans le code du bureau, notemment lors d'éxécutions de requête de mise à jour sql. A investiguer...

Change History

12/15/04 07:48:51 changed by anarcat

On a le meme probleme un peu partour dans m_mailman.php. Il est possible que mailman.create et mailman.delete ne fonctionnent pas. Du moins, je le crois.

Il faudrait vérifier si les commandes retournent un bon résultat dans le PHP.

Pour reproduire:

--- ./bureau/class/m_mailman.php.orig Fri Apr 23 01:02:41 2004

+++ ./bureau/class/m_mailman.php Fri Apr 23 01:08:39 2004 @@ -125,7 +125,7 @@

$quota->inc("mailman"); // incrémentation du quota // Creation de la liste : 1. recherche du nom de la liste // CA NE MARCHE PAS !

- $name=$login; + $name=$login.'-'.$domain;

$db->query("INSERT INTO mailman (uid,list,domain,name) VALUES (".$this->uid.",'$login','$domain','$name');"); if (!$mail->add_wrapper($login,$domain,"/var/lib/mailman/mail/mailman post $name","mailman")

!$mail->add_wrapper($login."-request",$domain,"/var/lib/mailman/mail/mailman request $name","mailman")

Créer et retirer une liste.

J'ai remarqué que la liste était retirée des la db mysql, même si la commande mailman.delete n'Avait pas réussi à retirer la liste.

03/31/06 22:15:40 changed by anarcat

  • type set to defect.
  • milestone set to 2.0.

06/23/06 06:15:08 changed by anarcat

  • version deleted.
  • milestone changed from 2.0 to 1.0.

11/27/06 23:19:00 changed by anarcat

(In [1741]) check for errors in file browser handlers, see #68

04/24/08 19:58:44 changed by anarcat

(In [2197]) try to detect errors from mailman.create handler

See #68 See #1096

04/24/08 20:43:52 changed by anarcat

(In [2203]) better error checking in list deletion too

See #68 See #1096