Bug 443366

Summary: celestia-1.5.1-1.77: 3 * call to wrong C++ delete
Product: [openSUSE] openSUSE 11.1 Reporter: David Binderman <dcb314>
Component: KDE3Assignee: E-mail List <kde-maintainers>
Status: RESOLVED DUPLICATE QA Contact: E-mail List <qa-bugs>
Severity: Minor    
Priority: P4 - Low    
Version: Factory   
Target Milestone: ---   
Hardware: All   
OS: openSUSE 11.0   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description David Binderman 2008-11-10 13:16:36 UTC
I just checked some of the source code in the Suse Linux factory package
celestia-1.5.1-1.77 and I noticed the following

$ egrep "new|delete" ../BUILD/celestia-1.5.1/src/celengine/modelfile.cpp | fgrep indices
        uint32* indices = new uint32[indexCount];
                delete indices;
                delete indices;
        uint32* indices = new uint32[indexCount];
                delete indices;

The deletes look wrong. Maybe

        delete [] indices; indices = NULL;

would be better.
Comment 1 Lubos Lunak 2008-11-11 12:51:37 UTC

*** This bug has been marked as a duplicate of bug 442933 ***