#8046 closed defect (invalid)
Free a non-heap memory
| Reported by: | wurongxin | Owned by: | |
|---|---|---|---|
| Priority: | minor | Component: | avcodec |
| Version: | git-master | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Reproduced by developer: | no | |
| Analyzed by developer: | no |
Description
Summary of the bug:
How to reproduce:
% ffmpeg -i input ... output ffmpeg version built on ...
Patches should be submitted to the ffmpeg-devel mailing list and not this bug tracker.
In the source exr.c, at Line 1615, the function av_dict_set may free the variable "key" which is a non-heap memory object. For more details, please read the code snippet in the attachment.
Attachments (1)
Change History (5)
by , 5 years ago
| Attachment: | Screenshot 2019-07-28 at 10.09.55 PM.png added |
|---|
comment:1 by , 5 years ago
| Priority: | normal → critical |
|---|
comment:2 by , 5 years ago
| Version: | unspecified → git-master |
|---|
follow-up: 4 comment:3 by , 5 years ago
| Priority: | critical → minor |
|---|---|
| Resolution: | → invalid |
| Status: | new → closed |
Why you hide in your screenshot actual flag used?
If flags are not set stuff are simply copied.
comment:4 by , 5 years ago
Replying to richardpl:
Why you hide in your screenshot actual flag used?
If flags are not set stuff are simply copied.
Oh, you are right. Sorry that I did not notice the flag is 0. Actually, I reported another one with the screenshot (https://trac.ffmpeg.org/attachment/ticket/8045/Screenshot%202019-07-28%20at%2010.00.01%20PM.png) without shadowing the flag.
Actually, I have other cases where flag is not 0. I will investigate further and report in this bug report later. Thanks for your prompt reply.
Code snippet of the bug