Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix the 'st-flash erase' crash in Cygwin issue #413

Closed
wants to merge 1 commit into from

Conversation

gerryqd
Copy link

@gerryqd gerryqd commented May 14, 2016

In this case, function 'get_opts()' will crash, as it will
access memory outside of 'av' array.

This fix add the protection for this case.

In this case, function 'get_opts()' will crash, as it will
access memory outside of 'av' array.
@xor-gate xor-gate closed this May 14, 2016
@Nightwalker-87 Nightwalker-87 modified the milestones: v1.3.0, v1.7.0 Mar 17, 2020
@Nightwalker-87 Nightwalker-87 modified the milestones: v1.7.0, Feedback required Mar 20, 2020
@Nightwalker-87 Nightwalker-87 changed the base branch from master to develop March 21, 2020 13:53
@stlink-org stlink-org deleted a comment Apr 7, 2020
@stlink-org stlink-org deleted a comment from xor-gate Apr 7, 2020
@Nightwalker-87 Nightwalker-87 modified the milestones: Old issues, v1.6.1 Apr 7, 2020
@Nightwalker-87 Nightwalker-87 self-assigned this Apr 7, 2020
@Nightwalker-87 Nightwalker-87 self-requested a review April 7, 2020 19:32
@Nightwalker-87 Nightwalker-87 removed their request for review April 13, 2020 16:09
@Nightwalker-87 Nightwalker-87 modified the milestones: v1.6.1, Old issues Apr 13, 2020
@Nightwalker-87
Copy link
Member

Nightwalker-87 commented Apr 14, 2020

Obsolete with commit 7c99eb0 that merged with PR #459.

@Nightwalker-87 Nightwalker-87 removed this from the Old issues milestone Apr 14, 2020
@Nightwalker-87 Nightwalker-87 added this to the v1.3.0 milestone Apr 14, 2020
@stlink-org stlink-org locked as resolved and limited conversation to collaborators Apr 14, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants