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

test: decrease the source file size from 16MB to 12MB #599

Closed
wants to merge 1 commit into from

Conversation

hzhuang1
Copy link
Collaborator

While decompress a compressed file, it's risky if the plain text file is 16MB. So downgrade it to 12MB.

with text file /tmp/syslog
double free or corruption (!prev)
/home/hzhuang1/test/uadk/test/sanity_test.sh: line 63: 398814 Aborted (core dumped) $@
Fail to run UADK tests with static build.

While decompress a compressed file, it's risky if the plain text
file is 16MB. So downgrade it to 12MB.

with text file /tmp/syslog
double free or corruption (!prev)
/home/hzhuang1/test/uadk/test/sanity_test.sh: line 63: 398814 Aborted                 (core dumped) $@
Fail to run UADK tests with static build.

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
@hzhuang1
Copy link
Collaborator Author

hzhuang1 commented Dec 11, 2023

@Liulongfang @youngersh

Actually it's not a right fix. It should be abandoned.

There're multiple reasons of this "double free" issue.

  1. Because dladdr() is used, wd_get_lib_file_path() can't support to search static library path.
  2. In dynamic library version, it also reports error in fill_zip_comp_sqe() with error code EINVAL.

I attached my script file to reproduce this issue much easier. "cc" was used to invoke either the static library test or dynamic library test. "sanity.patch" was used to apply on $UADK/test/sanity_test.sh. And I also used "syslog.1" as test file. You could find it from "/var/log/syslog.1" from the CI server.

Although the test file is huge, the real test file should be only 16MB. And this issue occurred in "hw_ifl_perf()" with block mode. Each block size is only 8KB. So this issue is not related to 16MB size or 12MB size. I think you guys are much familiar with the hardware. Please help to debug it.

script.tgz

@hzhuang1
Copy link
Collaborator Author

Close it. Use pull request #600 instead.

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

Successfully merging this pull request may close these issues.

None yet

1 participant