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

Heap corruption in j2k encoder #5

Closed
gcode-importer opened this issue Jul 31, 2009 · 7 comments
Closed

Heap corruption in j2k encoder #5

gcode-importer opened this issue Jul 31, 2009 · 7 comments

Comments

@gcode-importer
Copy link

Originally reported on Google Code with ID 5

(OpenJPEG 1.3, Vista Business)

On some gray16 images (conceivably which have too many different colors)
j2k encoder crashes while freeing memory in tcd_free_encode(opj_tcd_t *tcd)
on this string:
   opj_free(prc->cblks.enc[cblkno].data - 2);

dbgheap.c
> image_to_j2k.exe!_CrtIsValidHeapPointer(const void *
pUserData=0x01069f20)

Version 1.2 didn't crash on such images, but i tested it less than 1.3,
since i need lossless j2k.

Steps to reproduce: try to encode attached file (random.tif) with
image_to_j2k.exe

Reported by go_go_johnny_go@mail.ru on 2009-07-31 15:14:09


- _Attachment: [random.tif](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-5/comment-0/random.tif)_
@gcode-importer
Copy link
Author

Indeed, the memory allocated for prc->cblks.enc[cblkno].data is too small for 16 bit

depth components. I'm working on this one.

Reported by fodevaux on 2010-04-08 20:23:36

  • Status changed: Started

@gcode-importer
Copy link
Author

Reported by malaterre on 2012-02-10 10:34:53

@gcode-importer
Copy link
Author

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=672455

Reported by malaterre on 2012-05-11 08:14:53

@gcode-importer
Copy link
Author

This issue was updated by revision r1703.

Update issue 62

Reported by malaterre on 2012-05-29 17:21:09

@gcode-importer
Copy link
Author

Reported by malaterre on 2014-02-24 17:11:39

@gcode-importer
Copy link
Author

Closing this one. Please follow issue 259 instead

Reported by malaterre on 2014-02-25 12:40:22

  • Status changed: WontFix

@gcode-importer
Copy link
Author

Reported by malaterre on 2014-03-07 15:11:26

This was referenced May 23, 2017
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