FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix the encoding leak when input CGImage is not RGBA8888 or RGB888 by dreampiggy · Pull Request #54 · SDWebImage/SDWebImageWebPCoder · GitHub

Fix the encoding leak when input CGImage is not RGBA8888 or RGB888 - #54

Merged
dreampiggy merged 1 commit into
masterfrom
bugfix_encoding_leak
Feb 24, 2021
Merged

Fix the encoding leak when input CGImage is not RGBA8888 or RGB888#54
dreampiggy merged 1 commit into
masterfrom
bugfix_encoding_leak

Conversation

Copy link
Copy Markdown
Contributor

See Apple's documentation about vImageBuffer_InitWithCGImage

You're responsible for releasing the memory that buf->data points to back to the system using free().

https://developer.apple.com/documentation/accelerate/1399118-vimagebuffer_initwithcgimage?language=objc

codecov Bot commented Feb 24, 2021
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #54 (44b1aae) into master (7d68ede) will not change coverage.
The diff coverage is 37.50%.

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   77.49%   77.49%           
=======================================
  Files           2        2           
  Lines         813      813           
=======================================
  Hits          630      630           
  Misses        183      183           
Flag Coverage Δ
ios 77.49% <37.50%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
SDWebImageWebPCoder/Classes/SDImageWebPCoder.m 77.37% <37.50%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d68ede...44b1aae. Read the comment docs.

dreampiggy added the leak Memory Leak label Feb 24, 2021
dreampiggy merged commit 9d3c8fd into master Feb 24, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

leak Memory Leak

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL