https://bz.apache.org/ooo/show_bug.cgi?id=127449
Issue ID: 127449
Issue Type: DEFECT
Summary: Possible access to unintended variable in
"main/svx/source/svdraw/svdxcgv.cxx" line 500
Product: General
Version: 4.2.0-dev
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: Normal
Priority: P5 (lowest)
Component: code
Assignee: [email protected]
Reporter: [email protected]
Target Milestone: ---
While experimenting with a CodeSonar plugin we develop, we noticed a
potential issue in file "main/svx/source/svdraw/svdxcgv.cxx" line 500 function
ImpPasteObject
nSizX*=aMapFact.X().GetNumerator();
nSizX*=rMap.GetScaleX().GetNumerator();
nSizX*=aDstFr.GetDenominator();
nSizX/=aMapFact.X().GetDenominator();
nSizX/=rMap.GetScaleX().GetDenominator();
nSizX/=aDstFr.GetNumerator();
nSizY*=aMapFact.Y().GetNumerator();
nSizY*=rMap.GetScaleY().GetNumerator();
nSizX*=aDstFr.GetDenominator(); //Line 500
nSizY/=aMapFact.Y().GetDenominator();
nSizY/=rMap.GetScaleY().GetDenominator();
nSizY/=aDstFr.GetNumerator();
Shouldn't you update nSizY instead of nSizX? (in other words, I think the last
character of the identifier should be Y and not X).
Regards,
Petru-Florin Mihancea
--
You are receiving this mail because:
You are the assignee for the issue.