#29 Out of bounds memory access

[most yuut]

https://sourceforge.net/p/aruco/aruco-git/ci/97c0c33d92d57ad9d8f1fb6ebb512c00af490881/

This fixes an error where the function doesn't return a value. The output parameter nRotations might remain unset while a random true is returned.

In markerdetector.cpp is can cause an out of bounds access here if nRotations < 0 or nRotations > 4:

std::rotate(markers_omp[omp_get_thread_num()].back().begin(), markers_omp[omp_get_thread_num()].back().begin() + 4 - nRotations, markers_omp[omp_get_thread_num()].back().end());

Since the issue is already fixed, this ticket has the purpose to make sure you know that this was a significant issue and should warrant a new release.