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

Fix MSAN error in rdtime.h #4866

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Fix MSAN error in rdtime.h #4866

wants to merge 6 commits into from
+17 −0

Conversation

fdr400
Copy link

@fdr400 fdr400 commented Oct 8, 2024

Resolves #4865

@fdr400 fdr400 requested a review from a team as a code owner October 8, 2024 13:52
@confluent-cla-assistant
Copy link

confluent-cla-assistant bot commented Oct 8, 2024 •
edited
Loading

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ fdr400
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@cla-assistant CLA Assistant
Copy link

cla-assistant bot commented Oct 8, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

robot-piglet pushed a commit to ytsaurus/ytsaurus that referenced this pull request Oct 9, 2024
@fdr400
import msan fix librdkafka
d095bb7 
Здесь мой пр с подробным описанием (в ишью) фикса, который чинит проблему <confluentinc/librdkafka#4866>

Но, судя по всему, в librdkafka достаточно долго импортят ПР-ы \+ в Аркадии не прошёл импорт новой версии librdkafka, там что-то переломалось из-за nanopb, как я понял

Цель ПР-а в том, чтобы дать возможность librdkafka запускаться под MSAN. Сейчас мы (и все, кто пользуется librdkafka) не можем включить MSAN тесты в userver’е: <HIDDEN_URL>

Поэтому хочется пока добавить этот патч

Вроде как, я правильно сделал, так как реимпорт прошёл. Скажите, если что не так
commit_hash:c5775d53cf9ed820dc44a4e0823574747b26b3e4
robot-piglet pushed a commit to ytsaurus/ytsaurus that referenced this pull request Oct 9, 2024
@fdr400
import msan fix librdkafka
94d2472 
Здесь мой пр с подробным описанием (в ишью) фикса, который чинит проблему <confluentinc/librdkafka#4866>

Но, судя по всему, в librdkafka достаточно долго импортят ПР-ы \+ в Аркадии не прошёл импорт новой версии librdkafka, там что-то переломалось из-за nanopb, как я понял

Цель ПР-а в том, чтобы дать возможность librdkafka запускаться под MSAN. Сейчас мы (и все, кто пользуется librdkafka) не можем включить MSAN тесты в userver’е: <HIDDEN_URL>

Поэтому хочется пока добавить этот патч

Вроде как, я правильно сделал, так как реимпорт прошёл. Скажите, если что не так
commit_hash:c5775d53cf9ed820dc44a4e0823574747b26b3e4
@fdr400
Copy link
Author

fdr400 commented Oct 9, 2024

@emasab Hello! Can you review the changes, please. We already imported the patch into our codebase and it MSAN tests start working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MemorySanitizer: use-of-uninitialized-value in rdtime.h
1 participant
@fdr400