luke
/
magic
mirror of https://github.com/RTimothyEdwards/magic.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

calma/CalmaWrite.c: calmaProcessBoundaryZ() Dereference of null pointer 

Theoretical NULL pointer deref.  Seems like false positive as a
BOUNDARY record does not make sense with no points.

This guards against a potential crash from a bad data model.

SonarCloud
Dereference of null pointer
https://sonarcloud.io/project/issues?open=AZJB17hwNGfDNup0Rkub&id=dlmiles_magic
This commit is contained in:
Darryl L. Miles 2024-10-04 20:08:52 +01:00 committed by Tim Edwards
parent dd6bb9baf4
commit 97913ab08c
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
calma/CalmaWriteZ.c
View File

@ -1830,8 +1830,11 @@ calmaProcessBoundaryZ(
calmaOutI4Z(lbref->lb_start.p_y * calmaPaintScale, f);
chkcount++;
}
calmaOutI4Z(listtop->lb_start.p_x * calmaPaintScale, f);
calmaOutI4Z(listtop->lb_start.p_y * calmaPaintScale, f);
if (listtop != NULL)
{
calmaOutI4Z(listtop->lb_start.p_x * calmaPaintScale, f);
calmaOutI4Z(listtop->lb_start.p_y * calmaPaintScale, f);
}
if (chkcount != bounds->bt_points)
TxError("Points recorded=%d; Points written=%d\n",