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.

Reply via email to