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 returning pointer to freed memory in FCreateFImage. #427

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

Quipyowert2
Copy link
Contributor

@Quipyowert2 Quipyowert2 commented Feb 6, 2021

When XCreateImage fails, fim is freed and then returned. The returned
pointer value is invalid.

  • What does this PR do?
    Fixes returning an invalid memory address in FCreateFImage by returning NULL.

  • Screenshots (if applicable)

  • Issue number(s)
    Code Cleanup: Codacy issues list #107
    If this PR addresses any issues, please ensure the appropriate commit
    message(s) contains:

Fixes #XXX

at the end of your commit message, where XXX should be replaced with the
relevant issue number.

If there is more than one issue fixed then use:

Fixes #XXX, fixes #YYY, fixes #ZZZ

@Quipyowert2 Quipyowert2 force-pushed the nm/fix-returning-freed branch from 53a3d9e to a2d5d29 Compare February 6, 2021 00:42
@Quipyowert2 Quipyowert2 changed the title Fix returning freed pointer. Fix returning pointer to freed memory in FCreateFImage. Feb 6, 2021
When XCreateImage fails, fim is freed and then returned. The returned
pointer value is invalid.
@Quipyowert2 Quipyowert2 force-pushed the nm/fix-returning-freed branch from a2d5d29 to ec7ecef Compare February 6, 2021 01:58
@ThomasAdam ThomasAdam merged commit 446a57c into fvwmorg:master Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants