KhronosGroup / KhronosGroup/OpenCOLLADA

PVS-Studio check

Open
#240 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
679
Forks
260
PR merge metrics
No merged PRs in 30d

Description

## PVS-Studio (http://www.viva64.com/) some messages:

V514 Dividing sizeof a pointer 'sizeof (objectGroups)' by another value. There is a probability of logical error presence. mayadmdagnode.h 40

struct ObjectGroups{
componentList objectGrpCompList;
int objectGroupId;
short objectGrpColor;
void write(FILE\* file) const;
}\* objectGroups;

void write(FILE\* file) const
{
size_t size = sizeof(objectGroups)/sizeof(ObjectGroups);
for(size_t i=0; iParserString::ParserString(....)' should be used. generatedsaxparsertypes.h 56

struct ParserString : public UnionString
{
ParserString()
{
UnionString::str = 0;
UnionString::length = 0;
}

ParserString(const int& val)
{
ParserString();
}
};
# add

V603 The object was created but it is not being used. If you wish to call constructor, 'this->URI::URI(....)' should be used. colladabuuri.cpp 169
## V603 The object was created but it is not being used. If you wish to call constructor, 'this->URI::URI(....)' should be used. colladabuuri.cpp 180

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the PVS-Studio findings in the named files, including mayadmdagnode.h, mayadmtypes.h, generatedsaxparsertypes.h, colladamayadocumentexporter.cpp, and colladabuuri.cpp. Run the PVS-Studio check and inspect each reported warning; done means the listed warnings are addressed without introducing new analyzer findings.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.