[mythtv-commits] [MythTV/mythtv] 991d0f: Fix crash of goom visualisation in MythMusic
kmdewaal
noreply at github.com
Wed Dec 22 19:49:02 UTC 2021
Branch: refs/heads/master
Home: https://github.com/MythTV/mythtv
Commit: 991d0f473226d68693b40bf2ae3a5b3c5b6cd461
https://github.com/MythTV/mythtv/commit/991d0f473226d68693b40bf2ae3a5b3c5b6cd461
Author: Klaas de Waal <kdewaal at mythtv.org>
Date: 2021-12-22 (Wed, 22 Dec 2021)
Changed paths:
M mythtv/libs/libmythtv/visualisations/goom/goom_tools.h
Log Message:
-----------
Fix crash of goom visualisation in MythMusic
The value returned by iRAND is used as an array index in
visualisations/goom/tentacle3d.cpp at line 188:
float val = (float)(ShiftRight(data[0][iRAND(511)],10)) * rapport;
The problem is that iRAND can return negative values and this
is sufficient to cause the problems mentioned in issue #429.
Fixed by changing the iRAND function to return an unsigned integer value.
Note there is a commented-out implementation of iRAND present in the code;
this implementation does also return an unsigned integer value.
Refs #429
More information about the mythtv-commits
mailing list