Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix RGB_DISABLE_TIMEOUT overflow warning (#9866) #9874

Merged
merged 2 commits into from
Aug 5, 2020

Conversation

BeefaloKing
Copy link
Contributor

@BeefaloKing BeefaloKing commented Jul 30, 2020

Description

Forces long integer math in the expansion of RGB_DISABLE_TIMEOUT when a config file uses the deprecated define RGB_DISABLE_AFTER_TIMEOUT to prevent overflow with values greater than 27.

#    define RGB_DISABLE_TIMEOUT (RGB_DISABLE_AFTER_TIMEOUT * 1200ul)

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@zvecr zvecr requested a review from a team July 30, 2020 12:34
quantum/rgb_matrix.c Outdated Show resolved Hide resolved
@drashna drashna merged commit 349646b into qmk:master Aug 5, 2020
@BeefaloKing BeefaloKing deleted the rgb-timeout-overflow branch August 5, 2020 06:13
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request Aug 9, 2020
* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)
fdawans pushed a commit to fdawans/qmk_firmware that referenced this pull request Aug 11, 2020
* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)
fdawans added a commit to fdawans/qmk_firmware that referenced this pull request Aug 11, 2020
nicocesar pushed a commit to nicocesar/qmk_firmware that referenced this pull request Aug 12, 2020
* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)
fcoury pushed a commit to fcoury/qmk_firmware_archive that referenced this pull request Sep 20, 2020
* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Expansion of RGB_DISABLE_TIMEOUT overflows
4 participants