bugCERTI - Bugs: bug #19534, XmlParser Bug report

 
 

bug #19534: XmlParser Bug report

Submitter:  moosefly <moosefly>
Submitted:  Fri 06 Apr 2007 01:17:08 AM UTC
   
 
Category:  CERTI Severity:  * 3 - Normal
Item Group:  Software error Status:  Ready For Test
Privacy:  Public Assigned to:  erk
Open/Closed:  Closed Release:  * 3.2.1
Reproducibility:  Every Time Fixed Release:  * 3.2.4
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 04 May 2007 12:56:38 PM UTC, comment #1: 

Ok now fixed in CVS.

In fact
 RootObject::addRoutingSpace(const RoutingSpace &rs)

stores a copy of the argument and not a reference,
thus modifying RoutingSpace object after
it has been added has not effect on the RootObject.

You may test if you have a CVS access.

Eric NOULARD <erk>
Group administrator
Fri 06 Apr 2007 01:17:08 AM UTC, original submission:  

 libCERTI/XmlParser.cc :

void
XmlParser::parseRoutingSpace()
{
    D[pdTrace] << "New Routing Space" << endl ;

    DimensionHandle freeDimensionHandle = 1 ;
    xmlNodePtr prev = cur ;
    RoutingSpace current ;
    current.setHandle(freeSpaceHandle++);
    current.setName((char *) xmlGetProp(cur, ATTRIBUTE_NAME));
--- root->addRoutingSpace(current);

    // Dimensions
    cur = cur->xmlChildrenNode ;
    while (cur != NULL) {
        if ((!xmlStrcmp(cur->name, NODE_DIMENSION))) {
            Dimension dimension(freeDimensionHandle++);
            dimension.setName((char *) xmlGetProp(cur, ATTRIBUTE_NAME));
            current.addDimension(dimension);
        }
        cur = cur->next ;
    }
+++ root->addRoutingSpace(current);
    cur = prev ;
}

ps:
the routingspace should be added after all dimensions have been added .

moosefly <moosefly>

 

(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

 

Carbon-Copy List
  • -email is unavailable- added by erk (Posted a comment)
  • -email is unavailable- added by moosefly (Submitted the item)
  •  

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-11-19 erk Open/ClosedOpen Closed
    2007-05-04 erk Fixed ReleaseNextRelease 3.2.4
    2007-05-04 erk StatusNone Ready For Test
        Assigned toNone erk
        ReleaseNone 3.2.1
        Fixed ReleaseNone NextRelease

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code