bugTransport Sample Protocol - Bugs: bug #15021, buggy connect_url for Solaris8

 
 

bug #15021: buggy connect_url for Solaris8

Submitter:  None
Submitted:  Sun 20 Nov 2005 09:06:54 PM UTC
   
 
Category:  Core Severity:  3 - Normal
Status:  Ready For Test Privacy:  Public
Assigned to:  erk Open/Closed:  Closed
Release:  0.7.0 Fixed Release:  0.7.2
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 04 Dec 2005 09:55:14 PM UTC, comment #2: 

My previous correction was silly as pointed out
by Euskadi...

Now it should be OK...to be tested though :))

Eric NOULARD <erk>
Group administrator
Tue 29 Nov 2005 10:14:37 PM UTC, comment #1: 

I have replace atoi with strtol which should
update errno properly.

Eric NOULARD <erk>
Group administrator
Sun 20 Nov 2005 09:06:54 PM UTC, original submission:  

J'ai trouvé le bug sous Solaris 8.

Ma conf :
uname -a : SunOS seven 5.8 Generic_108528-27 sun4u sparc SUNW,Ultra-5_10
compilo : /home/Workshop/SW5.0/SUNWspro/bin/cc


Voici l'arbre d'appel des fonctions :


----+ TSP_consumer_connect_url
    +---+ TSP_remote_open_server
        +---+ TSP_remote_open_prog_id
            +---+ clnt_create


Le premier appel à TSP_consumer_connect_url se fait avec seven/:0.
Tout va bien jusqu'au clnt_create. L'appel marche bien (cad ne retourne pas
un pointer null) mais errno est positionné à 22, soit EINVAL.

Lors de l'appel suivant à TSP_consumer_connect_url avec seven/:1,
la ligne 477 de tsp_consummer.c m'invalide le servernumber. car errno
vaut toujours 22.

Du coup, je passe par la portion de code qui traite les "partial url", c'est-à-dire
des lignes 505 à 519, dans laquelle je découvre de la récursivité.
Comme personne ne remet errno à 0, on empile les appels et on explose la stack.
CQFD.
Pourquoi errno est positionnée à 22 ?

C'est idem avec gcc.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 7 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-06-03 erk Open/ClosedOpen Closed
2006-03-22 erk ReleaseNone 0.7.0
    Fixed ReleaseNone 0.7.2
2005-12-04 erk Carbon-CopyRemoved euskadi -
2005-12-04 erk Carbon-Copy- Added euskadi
2005-11-29 erk StatusNone Ready For Test
    Assigned toNone erk

Back to the top

Powered by Savane 3.13-d3ae.
Corresponding source code