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.

Sign up for GitHub

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

Jump to bottom

Asan: Disable for CentOS7 because LD_PRELOAD issue. #3347

Closed
winlinvip opened this issue Dec 30, 2022 · 0 comments · Fixed by #3352
Closed

Asan: Disable for CentOS7 because LD_PRELOAD issue. #3347

winlinvip opened this issue Dec 30, 2022 · 0 comments · Fixed by #3352
Assignees
@chen-guanghua
Labels
EnglishNative This issue is conveyed exclusively in English. Enhancement Improvement or enhancement.
Milestone
5.0

Comments

@winlinvip
Copy link
Member

winlinvip commented Dec 30, 2022 •
edited
Loading

Description

For CentOS7, we got error as bellow, see google/sanitizers#796 for detail:

==48458==ASan runtime does not come first in initial library list; you should either link runtime to your application or manually preload it with LD_PRELOAD.

  1. SRS Version: 5.0.119

  2. SRS Log:

==48458==ASan runtime does not come first in initial library list; you should either link 
runtime to your application or manually preload it with LD_PRELOAD.
  1. SRS Config:
./objs/srs -c conf/srs.conf

Replay

CentOS 7.

Step 1: Link asan by shared library.

./configure --sanitizer=on --sanitizer-static=off && make

Step 2: Run SRS

./objs/srs -c conf/srs.conf

#==48458==ASan runtime does not come first in initial library list; you should either link 
# runtime to your application or manually preload it with LD_PRELOAD.

There is a workaround:

env LD_PRELOAD=/lib64/libasan.so.5 ./objs/srs -c conf/srs.conf
@winlinvip winlinvip added the Enhancement Improvement or enhancement. label Dec 30, 2022
@winlinvip winlinvip added this to the 5.0 milestone Dec 30, 2022
@chen-guanghua chen-guanghua mentioned this issue Jan 1, 2023
Merged
winlinvip pushed a commit to chen-guanghua/srs that referenced this issue Jan 2, 2023
@chen-guanghua @winlinvip
Asan: Disable asan for CentOS and use statically link if possible. v5…
793f579 
….0.127 (ossrs#3347)

1. Disable asan for all CentOS by default, however user could enable it.
2. Link asan statically if possible.
winlinvip added a commit that referenced this issue Jan 2, 2023
@chen-guanghua @winlinvip
Asan: Disable asan for CentOS and use statically link if possible. v5…
dd0f398 
….0.127 (#3347) (#3352)

* Asan: Disable asan for CentOS and use statically link if possible. v5.0.127 (#3347)

1. Disable asan for all CentOS by default, however user could enable it.
2. Link asan statically if possible.

* Update version to v5.0.127

Co-authored-by: winlin <[email protected]>
winlinvip pushed a commit that referenced this issue Jan 2, 2023
@chen-guanghua @winlinvip
SRS5: Asan: Disable asan for CentOS and use statically link if possib…
e1f6661 
…le. v5.0.127 (#3347) (#3352)

* Asan: Disable asan for CentOS and use statically link if possible. v5.0.127 (#3347)

1. Disable asan for all CentOS by default, however user could enable it.
2. Link asan statically if possible.

* Update version to v5.0.127

Co-authored-by: winlin <[email protected]>

PICK dd0f398
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chen-guanghua chen-guanghua

Labels
EnglishNative This issue is conveyed exclusively in English. Enhancement Improvement or enhancement.
Projects
None yet
Milestone
5.0
Development

Successfully merging a pull request may close this issue.

Asan: Disable asan for CentOS and use statically link if possible. chen-guanghua/srs
2 participants
@winlinvip @chen-guanghua