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 address bounds check #195

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jpmeijers
Copy link

@jpmeijers jpmeijers commented Dec 3, 2019

Fixes #194

Any pull request raised here MUST be submitted according to here MUST be submitted according to this template or it will be flagged with 'Not enough information'. No action will be taken till all the prerequisite information (according to the right template) is provided. If no information is provided for over a month after the 'Not enough information' label is applied, the issue will be closed.

Pull request details

  • Please check if the PR fulfills these requirements
  • The commit message/s explain/s all the changes clearly
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • Bug fix
  • Added feature
  • Documentation update
  • Other - Please explain here:
  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior? (if this is a feature change)

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:


DO NOT DELETE OR EDIT anything below this

Note 1: Make sure to add all the information needed to understand the bug so that someone can help. If any essential information is missing we'll add the 'Needs more information' label and close the issue until there is enough information.

Note 2: For support questions (for example, tutorials on how to use the library), please use the Arduino Forums. This repository's issues are reserved for feature requests and bug reports.


GitHub issue state GitHub issue title GitHub issue author GitHub issue label GitHub issue comments GitHub issue age GitHub issue last updateGitHub pull request check contexts

Copy link
Owner

@Marzogh Marzogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this :)

@Marzogh Marzogh self-assigned this Sep 16, 2021
@Marzogh Marzogh added the bugfix Fixes an existing bug in the current version of the code label Sep 16, 2021
@Marzogh Marzogh added this to the v3.4.1 milestone Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes an existing bug in the current version of the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug report - can't erase last sector of flash
2 participants