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

out-of-bounds read in coders/viff.c:445 ReadVIFFImage #77

Closed
moshekaplan opened this issue Jan 13, 2016 · 7 comments

Comments

Projects
None yet
3 participants
@dlemstra

This comment has been minimized.

Copy link
Member

commented Jan 13, 2016

This issue seems to link to an issue with the viff coder. We can reproduce that issue and we will work on a fix for it.

Did you add the wrong link or did you set the wrong title?

@moshekaplan

This comment has been minimized.

Copy link
Author

commented Jan 13, 2016

The wrong title and link. My bad.

@moshekaplan moshekaplan changed the title out-of-bounds read in coders/psd.c:797 ReadPSDChannelPixels out-of-bounds read in coders/viff.c:445 ReadVIFFImage Jan 13, 2016

dlemstra pushed a commit that referenced this issue Jan 13, 2016

Cristy

dlemstra pushed a commit that referenced this issue Jan 13, 2016

Cristy
@urban-warrior

This comment has been minimized.

Copy link
Contributor

commented Jan 13, 2016

Grab the latest from GIT and let us know if the problem is resolved.

@dlemstra

This comment has been minimized.

Copy link
Member

commented Jan 15, 2016

Can you give it another try?

@urban-warrior

This comment has been minimized.

Copy link
Contributor

commented Jan 15, 2016

We cannot reproduce the problem. We have the address sanitizer enabled and it does not return an exception:

convert id-000071,sig-06,src-002008,op-flip1,pos-580 null:
convert: improper image header `id-000071,sig-06,src-002008,op-flip1,pos-580i==' @ error/viff.c/ReadVIFFImage/414.
convert: no images defined `null:' @ error/convert.c/ConvertImageCommand/3241.
@moshekaplan

This comment has been minimized.

Copy link
Author

commented Jan 15, 2016

You're correct. It appears that this bug was fixed.

@dlemstra dlemstra closed this Jan 15, 2016

urban-warrior pushed a commit that referenced this issue Sep 22, 2017

Cristy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.