Skip to content

fix iam monitoring#1

Draft
adameat wants to merge 1 commit into
mainfrom
fix-iam-mon
Draft

fix iam monitoring#1
adameat wants to merge 1 commit into
mainfrom
fix-iam-mon

Conversation

@adameat

@adameat adameat commented Feb 19, 2024

Copy link
Copy Markdown
Owner

fix IAM authorization for monitoring
...

  • Not for changelog (changelog entry is not required)

adameat pushed a commit that referenced this pull request Apr 5, 2024
adameat pushed a commit that referenced this pull request Jul 23, 2024
https://github.com/yandex/toolchain-registry/releases/tag/lld16-v1

Если этот pr вам что-то сломал - смело откатывайте
1e3ff1754d81d2e09aff40ac0b9ca78c5e3f481b
adameat pushed a commit that referenced this pull request Aug 21, 2024
adameat pushed a commit that referenced this pull request Jul 1, 2025
commit_hash:5b246144e81555f1bc12b2c8ca111e351dcd5ed7
adameat pushed a commit that referenced this pull request Jul 30, 2025
adameat pushed a commit that referenced this pull request Sep 29, 2025
```
WARNING: ThreadSanitizer: data race (pid=123179)
  Read of size 8 at 0x0000065aad98 by thread T14:
    #0 NSystemInfo::CachedNumberOfCpus() util/system/info.cpp:241:10 (taxi-uservices-userver-core-tests+0x4155bba)
    #1 engine::(anonymous namespace)::TaskProcessorThreadStartedHook() userver/core/src/engine/task/task_processor.cpp:100:15 (taxi-uservices-userver-core-tests+0x5306bc8)
    ydb-platform#2 engine::TaskProcessor::PrepareWorkerThread(unsigned long) userver/core/src/engine/task/task_processor.cpp:347:5 (taxi-uservices-userver-core-tests+0x5306bc8)
```

The change on a hot path is equivalent to the previously compiler generated x86 code (just a `load` instruction). On other platforms this could actually fix a data race.
commit_hash:acb1fe1b8f3075cc27b18aab4f1bd0b88292c270
adameat pushed a commit that referenced this pull request Jan 12, 2026
commit_hash:7afedb08035fd3cd26143d1448bf94cc6dbc3f8c
adameat pushed a commit that referenced this pull request Jan 27, 2026
adameat pushed a commit that referenced this pull request Apr 14, 2026
adameat pushed a commit that referenced this pull request Apr 14, 2026
discovered by asan (on q-stable-2025-05-15 branch):
==716690==ERROR: AddressSanitizer: heap-use-after-free on address 0x504003ca5868 at pc 0x00005983a31c bp 0x7fd325567890 sp 0x7fd325567888
E   READ of size 8 at 0x504003ca5868 thread T54 (kikimr.User)
E   #0 0x5983a31b in pair<unsigned long &, NYql::TIssues, 0> /-S/contrib/libs/cxxsupp/libcxx/include/__utility/pair.h:163:15
E   #1 0x5983a31b in operator() /-S/contrib/ydb/core/fq/libs/checkpoint_storage/ydb_state_storage.cpp:503:26
...
E   0x504003ca5868 is located 24 bytes inside of 40-byte region [0x504003ca5850,0x504003ca5878)
E   freed by thread T54 (kikimr.User) here:
E   #0 0x262b6fcd in operator delete(void*) /-S/contrib/libs/clang18-rt/lib/asan/asan_new_delete.cpp:143:3
E   #1 0x59839cfa in operator= /-S/contrib/libs/cxxsupp/libcxx/include/__functional/function.h
E   ydb-platform#2 0x59839cfa in operator= /-S/contrib/libs/cxxsupp/libcxx/include/__functional/function.h:969:8
E   ydb-platform#3 0x59839cfa in operator() /-S/contrib/ydb/core/fq/libs/checkpoint_storage/ydb_state_storage.cpp:502:27
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.

2 participants