opencv_contrib icon indicating copy to clipboard operation
opencv_contrib copied to clipboard

Remove unused size of get_subbox_chart_physical()

Open y-guyon opened this issue 2 years ago • 0 comments

The argument size of get_subbox_chart_physical() is unused. get_subbox_chart_physical() is called only in one place. The local variable h could overflow the cast to int and lead to a SIGILL in some environments, see https://github.com/opencv/opencv_contrib/issues/3320.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • [x] I agree to contribute to the project under Apache 2 License.
  • [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • [x] The PR is proposed to the proper branch
  • [x] There is a reference to the original bug report and related work

y-guyon avatar Aug 08 '22 20:08 y-guyon