search

HexHive / ViDeZZo

ViDeZZo source code.
30 stars 4 forks source link

QEMU removed "--disable-sanitizers" option? #14

Closed Chan9Yan9 closed 1 month ago

Chan9Yan9 commented 1 month ago

Hi, I have found that in the current master branch, QEMU's configure no longer has the --disable-sanitizers option, while it is still present in the stable-9.1.0 version. This will cause the command make qemu qemu-coverage in step 2 fail to compile qemu. I am currently trying to confirm the specific commit version where this change was made.

Chan9Yan9 commented 1 month ago

Here are the pictures of the problem i met branch stable-9.1.0 image master image

cyruscyliu commented 1 month ago

Hey did you address this issue? Sorry, I forgot this totally.

Chan9Yan9 commented 1 month ago

Well, I found a commit on GitLab where it seems that QEMU has split --enable-sanitizers into --enable-asan and --enable-ubsan. The specific commit ID is cb771ac1.

Chan9Yan9 commented 1 month ago

Well, I only modified the QEMU configure sanitizer options in a few scripts within the videzzo_qemu folder. I have verified that they can compile the recent QEMU properly, but I'm not sure if this will introduce other issues. Here is my patch: d097238

cyruscyliu commented 1 month ago

Thank you!