Changeset 11122

Show
Ignore:
Timestamp:
02/26/10 10:23:28 (3 years ago)
Author:
robert
Message:

From Ryan Kawicki, "I guess I missed these during my testing, but if the database pager has outstanding requests while the application is shutting down, the archive can become invalidated through unsafe calls to ReaderWriterTXP::getArchive. I've made this function return a ref_ptr and change other locations to as needed to conform to the change. I've tested this and no more crashes.

Following files from revision 11057 have been attached."

Location:
OpenSceneGraph/trunk/src/osgPlugins/txp
Files:
5 modified

Legend:

Unmodified
Added
Removed

  • OpenSceneGraph/trunk/src/osgPlugins/txp/ReaderWriterTXP.cpp

    r11037 r11122  
    5959        //the txpNode. 
    6060        int id = ++_archiveId; 
    61         TXPArchive* archive = getArchive(id,osgDB::getFilePath(fileName)); 
     61        osg::ref_ptr< TXPArchive > archive = getArchive(id,osgDB::getFilePath(fileName)); 
    6262 
    6363        if (archive != NULL) 
    … …  
    7070            } 
    7171 
    72             txpNode->loadArchive(archive); 
    73              
     72            txpNode->loadArchive(archive.get()); 
     73 
    7474            return txpNode.get(); 
    7575        } 
    … …  
    8686        unsigned int id; 
    8787        sscanf(name.c_str(),"tile%d_%dx%d_%u",&lod,&x,&y,&id); 
    88         TXPArchive* archive = getArchive(id,osgDB::getFilePath(file)); 
     88        osg::ref_ptr< TXPArchive > archive = getArchive(id,osgDB::getFilePath(file)); 
    8989 
    9090        // The way this is done a 'tile' should only be created for lod 0 only, 
    … …  
    107107 
    108108        std::vector<TXPArchive::TileLocationInfo> childrenLoc; 
    109         osg::ref_ptr<osg::Node> tileContent = getTileContent(info,x,y,lod,archive, childrenLoc); 
     109        osg::ref_ptr<osg::Node> tileContent = getTileContent(info,x,y,lod,archive.get(), childrenLoc); 
    110110         
    111111        tileContent->setName("TileContent"); 
    … …  
    199199        unsigned int id; 
    200200        sscanf(name.c_str(),"subtiles%d_%dx%d_%u",&lod,&x,&y,&id); 
    201         TXPArchive* archive = getArchive(id,osgDB::getFilePath(file)); 
     201        osg::ref_ptr< TXPArchive > archive = getArchive(id,osgDB::getFilePath(file)); 
    202202 
    203203        int majorVersion, minorVersion; 
    … …  
    259259                    continue; 
    260260 
    261                 osg::ref_ptr<osg::Node> tileContent = getTileContent(info, loc, archive, childrenChildLoc); 
     261                osg::ref_ptr<osg::Node> tileContent = getTileContent(info, loc, archive.get(), childrenChildLoc); 
    262262 
    263263                tileContent->setName("TileContent"); 
    … …  
    361361                    continue; 
    362362 
    363                     osg::ref_ptr<osg::Node> tileContent = getTileContent(info,tileX,tileY,tileLOD,archive, childrenLoc); 
     363                    osg::ref_ptr<osg::Node> tileContent = getTileContent(info,tileX,tileY,tileLOD,archive.get(), childrenLoc); 
    364364 
    365365                    tileContent->setName("TileContent"); 
    … …  
    564564} 
    565565 
    566 TXPArchive *ReaderWriterTXP::getArchive(int id, const std::string& dir) 
    567 { 
    568     TXPArchive* archive = NULL; 
     566osg::ref_ptr< TXPArchive > ReaderWriterTXP::getArchive(int id, const std::string& dir) 
     567{ 
     568    osg::ref_ptr< TXPArchive > archive = NULL; 
    569569 
    570570    std::map< int,osg::ref_ptr<TXPArchive> >::iterator iter = _archives.find(id); 

  • OpenSceneGraph/trunk/src/osgPlugins/txp/ReaderWriterTXP.h

    r11037 r11122  
    8282    ReadResult local_readNode(const std::string& file, const osgDB::ReaderWriter::Options* options); 
    8383 
    84     TXPArchive *getArchive(int id, const std::string&); 
     84    osg::ref_ptr< TXPArchive > getArchive(int id, const std::string&); 
    8585  
    8686    osg::Node* getTileContent(const TXPArchive::TileInfo &info, int x, int y, int lod, TXPArchive* archive,  std::vector<TXPArchive::TileLocationInfo>& childrenLoc); 

  • OpenSceneGraph/trunk/src/osgPlugins/txp/TXPNode.cpp

    r11037 r11122  
    9797      } 
    9898   } 
    99 } 
    100  
    101 TXPArchive* TXPNode::getArchive() 
    102 { 
    103     return _archive.get(); 
    10499} 
    105100 

  • OpenSceneGraph/trunk/src/osgPlugins/txp/TXPNode.h

    r8437 r11122  
    7272    bool loadArchive(TXPArchive*); 
    7373     
    74     TXPArchive* getArchive(); 
     74    TXPArchive* getArchive() { return _archive.get(); } 
    7575 
    7676    void setArchive( TXPArchive* archive ) 

  • OpenSceneGraph/trunk/src/osgPlugins/txp/TXPParser.h

    r5309 r11122  
    8080 
    8181    GeodeGroup() : osg::Group(), _geode(NULL) 
    82         {} 
     82    {} 
    8383     
    8484    GeodeGroup(const GeodeGroup& gg,const osg::CopyOp& copyop=osg::CopyOp::SHALLOW_COPY): 
    8585    osg::Group(gg, copyop), _geode(gg._geode) 
    86         {} 
     86    {} 
    8787     
    8888    META_Node(txp, GeodeGroup); 
    … …  
    118118    } 
    119119 
    120         // Gets the archive 
    121         inline TXPArchive* getArchive() 
    122         { 
    123                 return _archive; 
    124         } 
    125      
     120    // Gets the archive 
     121    inline TXPArchive* getArchive() 
     122    { 
     123        return _archive.get(); 
     124    } 
     125 
    126126    // Scene parser 
    127127    osg::Group *parseScene( 
    … …  
    130130        std::map<int,osg::ref_ptr<osg::Node> > &models, 
    131131        double realMinRange, double realMaxRange, double usedMaxRange); 
    132          
     132 
    133133    // Returns the current Top Group 
    134134    inline osg::Group* getCurrTop() 
    … …  
    149149    } 
    150150 
    151         // Ensure material is loaded 
    152         inline void loadMaterial( int ix ) 
    153         { 
    154                 _archive->loadMaterial( ix ); 
    155         } 
     151    // Ensure material is loaded 
     152    inline void loadMaterial( int ix ) 
     153    { 
     154        _archive->loadMaterial( ix ); 
     155    } 
    156156     
    157157    // New to TerraPage 2.0 - local materials 
    … …  
    270270 
    271271    // gets tile center, from the top lod node 
    272         inline const osg::Vec3 getTileCenter() const 
    273         { 
    274                 return _tileCenter; 
    275         } 
    276  
    277         inline void setCurrentNode( osg::Node* node ) 
    278         { 
    279                 _currentNode = node; 
    280         } 
     272    inline const osg::Vec3 getTileCenter() const 
     273    { 
     274        return _tileCenter; 
     275    } 
     276 
     277    inline void setCurrentNode( osg::Node* node ) 
     278    { 
     279        _currentNode = node; 
     280    } 
    281281 
    282282   // After parsing this will return the number of trpgChildRef node found. 
    … …  
    303303     
    304304    // THE archive 
    305     TXPArchive    *_archive; 
     305    osg::ref_ptr< TXPArchive > _archive; 
    306306     
    307307    // Current parent 
    … …  
    324324     
    325325    // Materials 
    326         typedef std::map<int,osg::ref_ptr<osg::StateSet> >* MaterialMapType; 
    327         MaterialMapType _materialMap; 
     326    typedef std::map<int,osg::ref_ptr<osg::StateSet> >* MaterialMapType; 
     327    MaterialMapType _materialMap; 
    328328     
    329329    // Local materials 
    … …  
    331331     
    332332    // Model list 
    333         typedef std::map<int,osg::ref_ptr<osg::Node> > OSGModelsMapType; 
     333    typedef std::map<int,osg::ref_ptr<osg::Node> > OSGModelsMapType; 
    334334    OSGModelsMapType*      _models; 
    335335 
    … …  
    384384 
    385385    geomRead(TXPParser *in_parse) : _parse(in_parse) 
    386         {} 
     386    {} 
    387387    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    388388protected: 
    … …  
    396396public: 
    397397    groupRead(TXPParser *in_parse) : _parse(in_parse) 
    398         {} 
     398    {} 
    399399    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    400400protected: 
    … …  
    407407public: 
    408408    attachRead(TXPParser *in_parse) : _parse(in_parse) 
    409         {} 
     409    {} 
    410410    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    411411protected: 
    … …  
    456456public: 
    457457    lodRead(TXPParser *in_parse) : _parse(in_parse) 
    458         {} 
     458    {} 
    459459    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    460460protected: 
    … …  
    467467public: 
    468468    tileHeaderRead(TXPParser *in_parse) : _parse(in_parse) 
    469         {} 
     469    {} 
    470470    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    471471protected: 
    … …  
    479479public: 
    480480    modelRefRead(TXPParser *in_parse) : _parse(in_parse) 
    481         {} 
     481    {} 
    482482    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    483483protected: 
    … …  
    490490public: 
    491491    billboardRead(TXPParser *in_parse) : _parse(in_parse) 
    492         {} 
     492    {} 
    493493    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    494494protected: 
    … …  
    501501public: 
    502502    lightRead(TXPParser *in_parse) : _parse(in_parse) 
    503         {} 
     503    {} 
    504504    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    505505protected: 
    … …  
    512512public: 
    513513    layerRead(TXPParser *in_parse)  : _parse(in_parse) 
    514         {} 
     514    {} 
    515515    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    516516protected: 
    … …  
    523523public: 
    524524    labelRead(TXPParser *in_parse)  : _parse(in_parse) 
    525         {} 
     525    {} 
    526526    void *Parse(trpgToken tok,trpgReadBuffer &buf); 
    527527protected: