diff options
| author | Johan B. C. Engelen <jbc.engelen@swissonline.ch> | 2013-08-03 19:15:44 +0000 |
|---|---|---|
| committer | Johan B. C. Engelen <j.b.c.engelen@alumnus.utwente.nl> | 2013-08-03 19:15:44 +0000 |
| commit | 25de599c368f7859ec2a8e3e8c128fced3d52a9e (patch) | |
| tree | f3e1b46a54bef171499cb50b13bcacc92a2c668c /src/livarot/Shape.cpp | |
| parent | Fix gears test (diff) | |
| download | inkscape-25de599c368f7859ec2a8e3e8c128fced3d52a9e.tar.gz inkscape-25de599c368f7859ec2a8e3e8c128fced3d52a9e.zip | |
cppcheck Common realloc mistake: 'qrsData' nulled but not freed upon failure
(bzr r12448)
Diffstat (limited to 'src/livarot/Shape.cpp')
| -rw-r--r-- | src/livarot/Shape.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/livarot/Shape.cpp b/src/livarot/Shape.cpp index c29444a33..ab93486e5 100644 --- a/src/livarot/Shape.cpp +++ b/src/livarot/Shape.cpp @@ -130,7 +130,12 @@ Shape::MakeQuickRasterData (bool nVal) if (_has_quick_raster_data == false) { _has_quick_raster_data = true; - qrsData = (quick_raster_data*)realloc(qrsData, maxAr * sizeof(quick_raster_data)); + quick_raster_data* new_qrsData = static_cast<quick_raster_data*>(realloc(qrsData, maxAr * sizeof(quick_raster_data))); + if (!new_qrsData) { + g_error("Not enough memory available for reallocating Shape::qrsData"); + } else { + qrsData = new_qrsData; + } } } else |