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

Bus error when handling (root seek of disk64_trailer) in zzip_disk_findfirst (src/zzip/mmapped.c) [CVE-2018-6542] #17

Closed
ProbeFuzzer opened this issue Feb 1, 2018 · 5 comments

Comments

@ProbeFuzzer
Copy link

ProbeFuzzer commented Feb 1, 2018

On latest version (0.13.67) and master branch of zziplib:
there is a bus error (when handling seek of disk64_trailer) caused by loading of misaligned address in zzip_disk_findfirst function of src/zzip/mmapped.c, which could be triggered by the POC below. Note that this issue is different from #15.

The issue happens since the pointer "trailer" (line 444) could be manipulated by a crafted zip file, resulting in a misaligned memory access and bus error. Note that the issue is in libzip and may affect downstream programs. The POC is as small as 100 bytes.

444             zzip_size_t rootseek = zzip_disk64_trailer_get_rootseek(trailer);

To reproduce the issue, run: ./unzip-mem -p $POC
The POC could be downloaded at: https://github.com/ProbeFuzzer/poc/blob/master/zziplib/zziplib_0-13-67_unzip-mem_memory-alignment-errors_zzip_disk_findfirst_64.zip

master/src/zzip/mmapped.c:444:25: runtime error: load of misaligned address 0x7f798f31e163 for type 'uint64_t', which requires 8 byte alignment
0x7f798f31e163: note: pointer points here
71 00 00 00 3d 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
^

@abergmann
Copy link

CVE-2018-6542 was assigned to this issue.
https://nvd.nist.gov/vuln/detail/CVE-2018-6542

gdraheim added a commit that referenced this issue Feb 4, 2018
gdraheim added a commit that referenced this issue Feb 4, 2018
@gdraheim
Copy link
Owner

gdraheim commented Feb 4, 2018

can not reproduce in test_65427 on master

@gdraheim
Copy link
Owner

gdraheim commented Feb 5, 2018

after double-checking the download-size of the zip, the error is now reproduced in

make test_65422

@gdraheim
Copy link
Owner

gdraheim commented Feb 5, 2018

turns out that the fopen may have failed, after which the testcase is OK

@gdraheim gdraheim added this to the v0.13.68 next release milestone Feb 5, 2018
@gdraheim
Copy link
Owner

gdraheim commented Feb 5, 2018

checking back with v0.13.67 where the testcase had shown the problem => fixed

@gdraheim gdraheim closed this as completed Feb 5, 2018
@gdraheim gdraheim changed the title Bus error when handling (root seek of disk64_trailer) in zzip_disk_findfirst (src/zzip/mmapped.c) Bus error when handling (root seek of disk64_trailer) in zzip_disk_findfirst (src/zzip/mmapped.c) [CVE-2018-6542] Mar 28, 2018
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

3 participants