bugTransport Sample Protocol - Bugs: bug #14318, valeur de retour de tsp_close

 
 

bug #14318: valeur de retour de tsp_close

Submitted by:  choquet mathias <mathiaschoquet>
Submitted on:  Mon 29 Aug 2005 09:09:15 AM UTC  
 
Category: CoreSeverity: 2 - Minor
Status: FixedPrivacy: Public
Assigned to: Eric NOULARD <erk>Open/Closed: Closed
Release: NoneFixed Release: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 23 Oct 2005 07:37:26 AM UTC, comment #3:

En fait certaine fonction RPC étaient déclarées avec
un retour void (et pas int) ce qui donnait
un void* dans le code RPC et pas mal
de confusion sur "quoi" renvoyée.

Toutes les fonctions renvoies désormais une valeur
et celles qui renvoyaient void renvoient 'int' avec
comme sémantique

  1. --> OK

!=0 --> NOK

C'est commité et testé.

Eric NOULARD <erk>
Project AdministratorIn charge of this item.
Tue 18 Oct 2005 07:53:28 PM UTC, comment #2:

I do not understand my previous remark
since tsp_rpc.x seems to always have defined
TSP_request_close as void return type??

Eric NOULARD <erk>
Project AdministratorIn charge of this item.
Tue 04 Oct 2005 07:34:03 AM UTC, comment #1:

The fix in coming up in the forthcoming 0.7.0
the branche br_TSP_0_6_5_REFACTOR_DEVEL already contains
the fix.
Note that this bug has been discovered by Mathias. C.
when preparin async_write

Eric NOULARD <erk>
Project AdministratorIn charge of this item.
Mon 29 Aug 2005 09:09:15 AM UTC, original submission:

Dans le fichier de génération de code tsp_rpc.x : tsp_close est déclaré avec une valeur de retour ‘int’.
Pour chaque couche du protocole

(consumer-> client -> rpc_clnt ->rpc_svc -> server ->provider)

il est écrit avec le passage de ce ‘int’.

Le problème est que cette valeur est mauvaise puisque dans ‘src/core/rpc/tsp_server.c’ la valeur de retour n’est pas renseigné. :

void tsp_request_close_1_svc(TSP_request_close_t req_close, struct svc_req rqstp)
{
static char* dummy=0;
STRACE_IO(("-->IN"));
TSP_provider_request_close(&req_close);
STRACE_IO(("-->OUT"));
return ((void*) &dummy);
}

choquet mathias <mathiaschoquet>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 6 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Sun 23 Oct 2005 07:37:26 AM UTCerkOpen/ClosedOpen=>Closed
Sun 23 Oct 2005 07:37:25 AM UTCerkStatusNeed Info=>Fixed
Tue 18 Oct 2005 07:53:28 PM UTCerkSeverity3 - Normal=>2 - Minor
  StatusIn Progress=>Need Info
Tue 04 Oct 2005 07:34:03 AM UTCerkAssigned toNone=>erk
Tue 04 Oct 2005 07:34:02 AM UTCerkStatusNone=>In Progress

Back to the top


Powered by Savane 3.1-cleanup1