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

Buffer overflow in lib/filename.c:44:13 and lib/iso9660.c:594:16 #6

Closed
plusky opened this issue Apr 6, 2016 · 8 comments
Closed

Buffer overflow in lib/filename.c:44:13 and lib/iso9660.c:594:16 #6

plusky opened this issue Apr 6, 2016 · 8 comments
Assignees
Milestone

Comments

@plusky
Copy link
Contributor

plusky commented Apr 6, 2016

When building ebiso for openSUSE, following issues are observed by automatic checks of build log:
I: Statement is overflowing a buffer
E: ebiso bufferoverflow lib/filename.c:44:13
E: ebiso bufferoverflow lib/iso9660.c:594:16

See also openSUSE bug 974360

@gozora gozora added the bug label Apr 7, 2016
@gozora gozora added this to the v0.2.3 milestone Apr 7, 2016
@gozora gozora self-assigned this Apr 7, 2016
gozora added a commit that referenced this issue Apr 7, 2016
@gozora
Copy link
Owner

gozora commented Apr 7, 2016

@plusky Can you try to rebuild?

@plusky
Copy link
Contributor Author

plusky commented Apr 7, 2016

@gozora
Copy link
Owner

gozora commented Apr 7, 2016

@plusky Good! Thanks for info.
I've created one more commit d3f534a, not big changes, just version string update ...

@plusky
Copy link
Contributor Author

plusky commented Apr 7, 2016

Thanks a lot

@jsmeix
Copy link

jsmeix commented Apr 7, 2016

@gozora
many thanks for your very fast fix!

I am currently updating ebiso to 0.2.3
in the openSUSE build service.

@gozora
Copy link
Owner

gozora commented Apr 7, 2016

@jsmeix No problem, that were really stupid buffer overflows, I should reed more advanced guides to valgrind ...
@plusky Thanks for that pullrequest!

@plusky
Copy link
Contributor Author

plusky commented Apr 8, 2016

I guess this issue can be closed now

@plusky plusky closed this as completed Apr 8, 2016
@gozora
Copy link
Owner

gozora commented Apr 8, 2016

Yes.
Thanks to both of you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants