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

Null pointer dereference in ProcessJpegData (processor.cc:790) #38

Closed
josephbisch opened this issue Mar 16, 2017 · 2 comments
Closed

Null pointer dereference in ProcessJpegData (processor.cc:790) #38

josephbisch opened this issue Mar 16, 2017 · 2 comments

Comments

@josephbisch
Copy link

Compiled with afl-g++ and ASan.

./guetzli id:000000,sig:06,src:000000,op:flip2,pos:164.min /dev/null

(linked file is a zip file...unzip first before running above command)

ASAN:DEADLYSIGNAL
=================================================================
==25977==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x55a45ae98e29 bp 0x7ffd18ad4a90 sp 0x7ffd18ad4540 T0)
    #0 0x55a45ae98e28 in ProcessJpegData guetzli/processor.cc:790
    #1 0x55a45aeaecda in guetzli::ProcessJpegData(guetzli::Params const&, guetzli::JPEGData const&, guetzli::Comparator*, guetzli::GuetzliOutput*, guetzli::ProcessStats*) guetzli/processor.cc:836
    #2 0x55a45aeaecda in guetzli::Process(guetzli::Params const&, guetzli::ProcessStats*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >*) guetzli/processor.cc:863
    #3 0x55a45adb2dcf in main guetzli/guetzli.cc:237
    #4 0x7fe6dc95d2b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #5 0x55a45adb3b69 in _start (/home/joseph/fuzz/guetzli/guetzli-afl-debug/bin/Debug/guetzli+0xab69)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV guetzli/processor.cc:790 in ProcessJpegData
==25977==ABORTING
@josephbisch
Copy link
Author

This is on commit 9faf961 btw.

@robryk
Copy link
Contributor

robryk commented Mar 16, 2017

Thank you for the report. It appears that I've ended up introducing this when I fixed another problem with small images.

robryk added a commit to robryk/guetzli that referenced this issue Mar 17, 2017
robryk added a commit to robryk/guetzli that referenced this issue Mar 17, 2017
robryk added a commit to robryk/guetzli that referenced this issue Mar 17, 2017
@robryk robryk closed this as completed in b0cf821 Mar 17, 2017
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

No branches or pull requests

2 participants