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

Invalid memory address dereference in ReadImage (in input-bmp.ci:338) #24

Closed
fantasy7082 opened this issue Feb 13, 2018 · 1 comment
Closed

Comments

@fantasy7082
Copy link

fantasy7082 commented Feb 13, 2018

Hi, i found a bug in the sam2p 0.49.4. It crashed in function ReadImage .the details are below(ASAN):

./sam2p 010-unknown-defer-readimage EPS: /dev/null
This is sam2p 0.49.4.
Available Loaders: PS PDF JAI PNG JPEG TIFF PNM BMP GIF LBM XPM PCX TGA.
Available Appliers: XWD Meta Empty BMP PNG TIFF6 TIFF6-JAI JPEG-JAI JPEG PNM GIF89a+LZW XPM PSL1C PSL23+PDF PSL2+PDF-JAI P-TrOpBb.
ASAN:SIGSEGV

==13230==ERROR: AddressSanitizer: SEGV on unknown address 0x61710000f8f1 (pc 0x000000425adf bp 0x61600000fc80 sp 0x7fffffffd310 T0)
    #0 0x425ade in ReadImage /root/sam2p_ASAN2/sam2p/input-bmp.ci:338
    #1 0x425ade in bmp_load_image(_IO_FILE*) /root/sam2p_ASAN2/sam2p/input-bmp.ci:215
    #2 0x426859 in in_bmp_reader /root/sam2p_ASAN2/sam2p/in_bmp.cpp:23
    #3 0x475999 in Image::load(Image::Loader::UFD*, SimBuffer::Flat const&, char const*) /root/sam2p_ASAN2/sam2p/image.cpp:1427
    #4 0x40384a in run_sam2p_engine(Files::FILEW&, Files::FILEW&, char const* const*, bool) /root/sam2p_ASAN2/sam2p/sam2p_main.cpp:1055
    #5 0x402463 in main /root/sam2p_ASAN2/sam2p/sam2p_main.cpp:1148
    #6 0x7ffff6ac082f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)
    #7 0x402d38 in _start (/usr/local/sam2p-asan2/bin/sam2p+0x402d38)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /root/sam2p_ASAN2/sam2p/input-bmp.ci:338 ReadImage
==13230==ABORTING

POC FILE: https://github.com/fantasy7082/image_test/blob/master/010-unknown-defer-readimage

@pts pts closed this as completed in 118cb81 Feb 27, 2018
@pts
Copy link
Owner

pts commented Feb 27, 2018

Thank you for reporting this! Fixed in 118cb81.

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