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

Integer Overflow in function lsr_translate_coords at laser/lsr_dec.c:856 (CVE-2022-4202) #2333

Closed
2 tasks done
ajakk opened this issue Nov 29, 2022 · 2 comments
Closed
2 tasks done

Comments

@ajakk
Copy link

ajakk commented Nov 29, 2022

Hi, a CVE was issued affecting gpac and I wasn't able to find any report here or any fix. VulDB has a "writeup", which links to an advisory in Google Drive, which links to a reproducer

I can indeed reproduce when built from 4112fc3 (current HEAD at the time of writing):

# MP4Box -bt /poc-integer-Overflow                                                                                       
[iso file] extra box maxr found in hinf, deleting
[iso file] extra box maxr found in hinf, deleting
[iso file] Unknown box type 80rak in parent moov
[iso file] Unknown box type drzf in parent dinf
[iso file] Missing dref box in dinf
[iso file] Incomplete box mdat - start 11495 size 853090
[iso file] Incomplete file while reading for dump - aborting parsing
[iso file] extra box maxr found in hinf, deleting
[iso file] extra box maxr found in hinf, deleting
[iso file] Unknown box type 80rak in parent moov
[iso file] Unknown box type drzf in parent dinf
[iso file] Missing dref box in dinf
[iso file] Incomplete box mdat - start 11495 size 853090
[iso file] Incomplete file while reading for dump - aborting parsing
MPEG-4 LASeR Scene Parsing
[LASeR] sameg coded in bitstream but no g defined !
Reading 515 bits but max should be 64, skipping 451 most significants bits
laser/lsr_dec.c:856:27: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
@jeanlf jeanlf closed this as completed in b3d821c Dec 12, 2022
@ajakk
Copy link
Author

ajakk commented Dec 13, 2022

Thanks!

@ajakk
Copy link
Author

ajakk commented Dec 13, 2022

I can confirm the PoC now exits with this error:

Error loading scene: BitStream Not Compliant

        Error: BitStream Not Compliant

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

1 participant