cv::transpose does not work in-place (Bug #1090)


Added by Johannes Jordan almost 14 years ago. Updated almost 14 years ago.


Status:Done Start date:
Priority:Blocker Due date:
Assignee:Vadim Pisarevsky % Done:

0%

Category:core
Target version:-
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

Using cv::transpose on the same matrix which is NxM, N != M results in memory corruption!

Also, the dimensionality of the matrix is not changed to MxN.

Illustration:
cv::transpose(matrix, matrix); // does NOT work correctly

Workaround:
cv::Mat tmp;
cv::transpose(matrix, tmp);
matrix = tmp;

It should at least throw an assert if in-place is not to be supported.


Associated revisions

Revision ec1df9f5
Added by Roman Donchenko over 11 years ago

Merge pull request #1090 from asmorkalov:android_tbb_downgrade

History

Updated by Vadim Pisarevsky almost 14 years ago

the problem can not be reproduced in the latest version of OpenCV (since migration from cv::Mat to cv::InputArray and cv::OutputArray)

  • Status changed from Open to Done
  • (deleted custom field) set to fixed

Also available in: Atom PDF