From be958e3fdbe8c646c55061ea2d2fbe30fd988864 Mon Sep 17 00:00:00 2001 From: Marek Roszko Date: Sat, 5 Feb 2022 11:23:05 -0500 Subject: [PATCH] Avoid potential divide by zero in footprint spreading PVS V609 (cherry picked from commit 8588bea4b230ebec97804462167f3d36c72ecea5) --- pcbnew/autorouter/spread_footprints.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pcbnew/autorouter/spread_footprints.cpp b/pcbnew/autorouter/spread_footprints.cpp index 8b140f3ce5..28c5ec70f2 100644 --- a/pcbnew/autorouter/spread_footprints.cpp +++ b/pcbnew/autorouter/spread_footprints.cpp @@ -297,12 +297,12 @@ void SpreadFootprints( std::vector* aFootprints, wxPoint aSpreadArea { int Xsize_allowed = (int) ( sqrt( placementsurface ) * 4.0 / 3.0 ); - if( Xsize_allowed < 0 || Xsize_allowed > INT_MAX/2 ) + if( Xsize_allowed <= 0 || Xsize_allowed > INT_MAX/2 ) Xsize_allowed = INT_MAX/2; int Ysize_allowed = (int) ( placementsurface / Xsize_allowed ); - if( Ysize_allowed < 0 || Ysize_allowed > INT_MAX/2 ) + if( Ysize_allowed <= 0 || Ysize_allowed > INT_MAX/2 ) Ysize_allowed = INT_MAX/2; CRectPlacement placementArea;