[Scummvm-tracker] [ScummVM :: Bugs] #15294: AGS: Incorrect Use of Bitwise OR Instead of Logical OR in SetSliderValue
ScummVM :: Bugs
trac at scummvm.org
Thu Jul 25 06:52:13 UTC 2024
#15294: AGS: Incorrect Use of Bitwise OR Instead of Logical OR in SetSliderValue
--------------------+-------------------------
Reporter: RainRat | Owner: (none)
Type: defect | Status: new
Priority: low | Component: Engine: AGS
Version: | Keywords:
Game: |
--------------------+-------------------------
In the `SetSliderValue` and `GetSliderValue` functions, the validation
checks for the GUI number (`guin`) are using the bitwise OR operator (`|`)
instead of the logical OR operator (`||`).
Affected Code:
{{{
void SetSliderValue(int guin, int objn, int valn) {
if ((guin < 0) | (guin >= _GP(game).numgui)) quit("!SetSliderValue:
invalid GUI number");
[...]
int GetSliderValue(int guin, int objn) {
if ((guin < 0) | (guin >= _GP(game).numgui)) quit("!GetSliderValue:
invalid GUI number");
}}}
Corrected Code:
{{{
void SetSliderValue(int guin, int objn, int valn) {
if ((guin < 0) || (guin >= _GP(game).numgui)) quit("!SetSliderValue:
invalid GUI number");
[...]
int GetSliderValue(int guin, int objn) {
if ((guin < 0) || (guin >= _GP(game).numgui)) quit("!GetSliderValue:
invalid GUI number");
}}}
--
Ticket URL: <https://bugs.scummvm.org/ticket/15294>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM
More information about the Scummvm-tracker
mailing list