RE : [Tsp-devel] Patch pour le bug 1979

[ALAUX, Virginie]

Bonjour,

J'ai la version TSP 8.0.2
Je te joins le code tsp_session.c auquel j'ai appliqué le patch 
bug19179-erk1.patch avec la commande "patch -pN < bug19179-erk1.patch"

Le code :


int
TSP_session_get_garbage_session(channel_id_t* channel_id) {
  int i;
  int found;

  TSP_LOCK_MUTEX(&X_session_list_mutex,FALSE);

  found = FALSE;
  /* There is multiple criterion for session garbage collection */ 
  for( i = 0 ;  i < X_session_nb ; i++) {

    /* Do not garbage collect session in the following state */
    if ((X_session_t[i].state == TSP_SESSION_STATE_UNKNOWN) ||
        (X_session_t[i].state == TSP_SESSION_STATE_CLOSED)
        ) {
      continue;
    }

    /* Garbage collect Data link broken session */
    if (X_session_t[i].state == TSP_SESSION_STATE_BROKEN_LINK ) {
      found = TRUE;
      STRACE_INFO(("Garbage Collector thread found broken link session 
<%d>",X_session_t[i].channel_id));
      *channel_id = X_session_t[i].channel_id;
      break;
    }

    /*
     * Garbage collect datapool terminated session
     * either in SAMPLING or SAMPLE_DESTROY_OK state
     */
    if (NULL != X_session_t[i].session_data) {
      /* Datapool has been deleted */
      if (NULL == X_session_t[i].session_data->datapool) {
        /* FIXME nothing to do here */
        /* X_session_t[i].session_data->datapool
         * is the LOCAL datapool which is used when
         * a provider (possibly PASSIVE) is giving
         * ONE datapool per session
         * In the case of GLOBAL datapool this datapool
         * is ALWAYS NULL and never used in fact.
         * see bug #19179
         */
        continue;
      }
      /* Datapool has been terminated */
      else {
        if ( (TRUE==X_session_t[i].session_data->datapool->terminated) &&
            ((TSP_SESSION_STATE_SAMPLING                  == 
X_session_t[i].state)  ||
             (TSP_SESSION_STATE_REQUEST_SAMPLE_DESTROY_OK == 
X_session_t[i].state)
          STRACE_INFO(("Garbage Collector thread found Datapool TERMINATED for 
session <%d>",X_session_t[i].channel_id));
             )
            ){
          found = TRUE;
          *channel_id = X_session_t[i].channel_id;
        }
      }
    }
    /* FIXME to be added like 'timeout on idle consumer' */
  }

  TSP_UNLOCK_MUTEX(&X_session_list_mutex,FALSE);

  return found;
} /* end of TSP_session_get_garbage_session */

A la ligne 754:
    STRACE_INFO(("Garbage Collector thread found Datapool TERMINATED for 
session <%d>",X_session_t[i].channel_id));
Je pense que cette trace n'est pas incluse dans les accolades du if...
Donc j'ai l'erreur : syntax error.

Merci
Virginie Alaux (55-24)


-----Original Message-----
From: Eric Noulard [mailto:address@hidden
Sent: Monday, March 12, 2007 3:31 PM
To: Transport Sample Protocol development list
Subject: Re: [Tsp-devel] Patch pour le bug 1979


Le 12/03/07, ALAUX, Virginie <address@hidden> a écrit :
> Bonjour,
>
> Apres avoir pris le patch corrigeant le bug 19179 concernant le problème du 
> sample_destroy, j'ai recompilé TSP et la compil est NOK (bugs dans 
> tsp_session.c, line 754 : syntax error...)

Tout à fait étrange.

>
> Y a t il eu un nouveau correctif à ce patch?

ben non :))
;-((


Peux-tu m'envoyer ton fichier tsp_session.c
et me préciser

1)  De quelle version TSP es-tu parti?
2)  Comment tu as appliqué le patch?

Tu as aussi une autre option pour tester ce TSP patché qui est de prendre
la version actuellement dans CVS:
https://savannah.nongnu.org/cvs/?group=tsp

je pense que nous ferons une 0.8.3 d'ici peu qui ne devrait
pas être tellement différente de la version actuellement présente
dans le CVS.


--
Erk


_______________________________________________
Tsp-devel mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/tsp-devel

Ce courriel (incluant ses eventuelles pieces jointes) peut contenir des 
informations confidentielles et/ou protegees ou dont la diffusion est 
restreinte. Si vous avez recu ce courriel par erreur, vous ne devez ni le 
copier, ni l'utiliser, ni en divulguer le contenu a quiconque. Merci d'en 
avertir immediatement l'expediteur et d'effacer ce courriel de votre systeme. 
Astrium decline toute responsabilite en cas de corruption par virus, 
d'alteration ou de falsification de ce courriel lors de sa transmission par 
voie electronique.

This email (including any attachments) may contain confidential and/or 
privileged information or information otherwise protected from disclosure. If 
you are not the intended recipient, please notify the sender immediately, do 
not copy this message or any attachments and do not use it for any purpose or 
disclose its content to any person, but delete this message and any attachments 
from your system. Astrium disclaims any and all liability if this email 
transmission was virus corrupted, altered or falsified.
---------------------------------------------------------------------
Astrium SAS (393 341 516 RCS Paris) - Siege social: 6 rue Laurent Pichat, 75016 
Paris, France