Skip to content

Commit

Permalink
avoid memcpy with NULL as source
Browse files Browse the repository at this point in the history
- 2nd arg of memcpy() is assumed to be not NULL
- fixes sanitizer warning
CglPreProcess.cpp:8077:9: runtime error: null pointer passed as argument 2, which is declared to never be null
  • Loading branch information
svigerske committed Jan 18, 2022
1 parent d558398 commit e5adc40
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/CglPreProcess/CglPreProcess.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8074,7 +8074,8 @@ void CglPreProcess::addCutGenerator(CglCutGenerator *generator)
{
CglCutGenerator **temp = generator_;
generator_ = new CglCutGenerator *[numberCutGenerators_ + 1];
memcpy(generator_, temp, numberCutGenerators_ * sizeof(CglCutGenerator *));
if( numberCutGenerators_ > 0 )
memcpy(generator_, temp, numberCutGenerators_ * sizeof(CglCutGenerator *));
delete[] temp;
generator_[numberCutGenerators_++] = generator->clone();
}
Expand Down

0 comments on commit e5adc40

Please sign in to comment.