[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