OOM killer leaks messages #19
Comments
|
Another option is generating a unique ID and putting it into the environment (which can be read via /proc/pid/environ), and record that in shared memory so other threads can check it. If there's currently a task with a matching TID but its environ either doesn't have the variable or has a different value, then it's a different process. |
frc971-automation
pushed a commit
that referenced
this issue
Aug 27, 2022
a1b6164ef windows: call to FormatMessage() should _IGNORE_INSERTS (#849) 278ed9689 simplify DLOG macro definition in NDEBUG mode (#830) 6ed0c9e58 added emscripten support (#846) a34226ca9 support ndk r25 (#844) c515e1ae2 remove unused STDC_HEADERS definition (#835) 79d692c64 Strip only the first leading @ 086ce224d bazel: Use glog 0.6.0 in README. b33e3bad4 Merge pull request #812 from google/release-0.6 54b8bcbbf release 0.6 864ef6492 Merge pull request #801 from skeptic-monkey/master c640c2f7c Fix namespace resolution issue in LOG_EVERY_T 5addeedc0 Fix "'GLOG_EXPORT' macro redefined" on clang-cl d153e294b Avoid "GLOG_EXPORT macro redefined" in clang-cl CI c4ca519a8 Add CI job for clang-cl on Windows a8e0007e9 Merge pull request #790 from git-hulk/feature/logs-to-stdout 553ddaea8 Merge pull request #794 from Arfrever/tests-TMPDIR 6d3f70a40 Pass EXIT_SUCCESS or EXIT_FAILURE to exit() and _exit() 612a7d28c Tests: Respect TEST_TMPDIR, TMPDIR, TMP environmental variables 180b700dd Add logtostdout and colorlogtostdout flags for logging to stdout b38ed9d08 cmake_package_config_generate: Fix CMake warnings with CMake 3.23 9f0b7d3bf renamed GOOGLE_GLOG_DLL_DECL to GLOG_EXPORT aa94e6b9a simplify flags export 17269d649 cmake: enable custom prefix by default bc1fada1c libglog.pc: Set Libs.private for static linking 18836106d LogCleaner: avoid scanning logs too frequently b0174b3dd On Solaris, uname() returns non-negative value 696941218 Changed my Email ID 643441014 cmake: require at least gflags 2.2.2 a8cfbe0c0 stdcxx: eliminate excessive use of std::string::c_str() b3abfaa12 use uint32 for --max-log-size 43fc3bf91 Add log_year_in_prefix flag 42d509b1c revert to int atomics 224f66bb6 cmake: do not search for platform specific unwind 4ffa98388 ensure _Unwind_Backtrace to actually be available 087ad5bf3 eliminated clang warnings 9dc1107f8 run clang-tidy dfb9ffc32 provide backward compatible send overload c57511605 ci: removed deprecated msvc runner d09b999f3 LogCleaner: make overdue_days_ unsigned int baa7006e6 ci: enable warnings as errors 660352f02 cmake: require at least version 3.16 (fixes #705) 52354b2ad eliminate msvc warnings ccbda2dfd LogCleaner: Fix relative paths and add a new test ef36f800e Added an API to get GMT offset from LogMessageTime * Added API to get GMT offset * Made LogMessageTime as a memeber of LogMessage * Refactored LogSink::send() method ee6faf13b ci: use codecov token 6e9b2f16f cmake: set CMP0074 policy f4dd77ae6 ci: reworked windows builds 503e3dec8 Add Zhuoran Shen to AUTHORS and CONTRIBUTORS 531659984 Fix log filename format cd8fbd5fb ci: prefix artifact directories c8a7f5abe ci: added checkout action f47582361 ci: install lcov f773eb9b6 ci: unzip artifacts into separate directories f42fd3b15 ci: fixed unzipping multiple archives cb7c19d36 ci: removed non standard --zero option cf4c4699b ci: unzip artifacts individually 5a3bd74cf ci: replaced forEach by a loop 1da31df93 ci: name unpack step and fix indent 6bd5acfb6 ci: download workflow artifacts 8121c0755 ci: aggregate coverage in separate workflow 8b872216e ci: simplify coveralls job id 11f08e7bb Merge pull request #742 from google/protect-vlog 65199ed0b protect VLOG by use_logging b79e18b78 ci: updated coveralls job id 4399e5664 ci: explicitly identify coveralls builds 9206460cd Merge pull request #739 from google/cmake-glog-modules-install 64b4e327f cmake: fixed glog-modules.cmake installation 4df5ae746 ci: added cmake package config test ab5e10929 Revert "cmake: fix incorrect relative path concatenation (#733)" 32522a5f6 ci: notify coveralls on finished parallel builds d581807e3 ci: run coveralls on completed workflow 6c9f5b2ec ci: added jobs to coveralls workflow 0798a37bb ci: added coveralls workflow 33696664c restore the stream format after writing the log prefix (#731) 68964b8db cmake: fix incorrect relative path concatenation (#733) 17e7679fd Merge pull request #654 from aesophor/fix-path c27c3a803 make LogCleaner support relative paths e8e40f76d Merge pull request #732 from google/log-cleaner-tests 3362cc6c7 added log cleaner tests and fixed paths 9cf0eb7c9 Merge pull request #724 from google/mock-log 1398762db added mock log test ce3533613 Merge pull request #723 from google/coverage-use-templates 06e665108 ci: replace generated headers by templates 56d3facee Merge pull request #722 from google/linux-github-action 4a55b1158 ci: added linux github workflow 396558472 support stack unwind on Android acda90348 Merge pull request #720 from google/gmock-support 8474b7854 added gmock support 831a6f823 Add -Werror to Bazel presubmits c34dbe987 Fix `syscall` warning in Bazel Build 56523194b Merge pull request #716 from google/export-os-defines f7407d4d2 export OS_* defines (fixes #715) 2412db315 `raw_logging.h` depends on type definition in `logging.h`. Resolve google/glog#712 3b6b05d70 Use __EMSCRIPTEN__ instead of __USE_POSIX2 de81bb0aa Fix link error for Emscripten 9733727be Merge pull request #706 from dfreese/tsan_annotation ffb40018a Change size type in AnnotateBenignRaceSized to size_t from long 6f9e32a79 Merge pull request #700 from Starsss/master df95749e8 Fix: not implement virtual class when WITH_CUSTOM_PREFIX on 35983c895 Merge pull request #696 from Fettpet/signed_compare 9fce37c46 Add a static cast to compare unsigned with unsigned a063a04ec Merge pull request #694 from google/more-warnings 92cb3c449 fixed additional warnings f8c8e99fd Merge pull request #688 from google/eliminate-warnings 98e0e8c65 cmake: bump .so version 0b83bb203 eliminate warnings 370cb741b Merge pull request #674 from StephLin/readme-fix-header-id 3ce6090d0 override header language 36bcd28c6 Merge pull request #687 from google/exception-spec dfe573088 bump version 717f56983 fixed exception specification mismatch eca3ddef8 Merge pull request #685 from z-aki/master 4a4331f2f Fix syscall deprecation warning on macOS >= 10.12 a01d739ea Merge pull request #684 from sergiud/cmake-namespaced-gflags 8d3ce1c29 Merge pull request #686 from google/ci-more-targets c39fcf6e8 ci: added more targets 47ad26d5c readme: fixed inconsistent formatting 5bf7fdea1 cmake: prefer linking against gflags::gflags (fixes #683) 42ce901f2 Merge pull request #669 from darbitman/log-every-time-period a1374c420 Implement LOG_EVERY_T. Use <chrono> and <atomic> for C++11 or greater. For non-Windows pre-C++11 systems, use <time.h> and built-in atomic operations. For Windows pre-C++11, use the Windows implementations for time and atomic operations. 6e46c125e bazel: add GLOG_CUSTOM_PREFIX_SUPPORT a41555685 Remove ubuntu1604 from presubmit.yml a1fa841f3 readme: fix wrong header id a4a725d54 Merge pull request #671 from sergiud/cmake-define-have-cxx11-atomic 44331dc24 Merge pull request #672 from sergiud/cmake-allow-to-override-fpic f0bac8852 cmake: allow to override `-fPIC` 764162569 Merge pull request #670 from sergiud/cmake-msvc-snprintf d25a1b7bc cmake: export `<atomic>` availability (fixes #667) 9c1a25b93 cmake: fixed msvc snprintf detection (fixes #668) c1499f64a Export COPYING 03f2b3861 Merge pull request #650 from romange/Prod 86fea1ab2 Allow updating vmodule levels after vmodule was parsed, invalidate already cached site flags vmodule flag is relevant for GNUC environments only. fef48c17e Merge pull request #656 from romange/cygwin d2fed749a Fix cmake configuration for cygwin environment 0b3d4cb47 Merge pull request #651 from xkszltl/initapi 81e0d616e Expose `IsGoogleLoggingInitialized()` in public API. Usually library does not have control of the process lifespan. Without this function, it is impossible to init/shutdown reliably. It has been one of the major pain points for years when using glog in libraries. d5c04eec9 bazel: Use glog 0.5.0 in README 8f9ccfe77 Merge pull request #646 from sergiud/release-0.5 36fee386f release 0.5 8d40d7564 Merge pull request #642 from husskade/patch-1 a23e96f04 readme: update CMake source directory option 0efaa59a7 Merge pull request #641 from sergiud/nullptr 230da316c Merge pull request #637 from xiaobfly/master bda40ec00 Merge pull request #640 from sergiud/gflags-windows-linker-warnings 762a2a7ee eliminated gflags related linker warnings (#569) 7d4eeb14f Merge pull request #554 from santigl/santigl/custom-prefixer 0cbc23529 Add support for setting a custom prefix format c7656c3ec CHECK support for nullptr (fixes #341) 663bb2696 Merge pull request #620 from sergiud/cmake-allow-to-disable-symbolize 7ec73b509 Merge pull request #639 from sergiud/fix-windows-builds efc341d76 fixed windows builds a79416bab Merge pull request #638 from NeroBurner/github_actions 63ec71b58 Add windows builds with GitHub Actions 39e83581d cmake: allow to disable symbolize 635a07818 fix LogCleaner::IsLogFromCurrentProject for filename_extension e51790bf3 Merge pull request #625 from sergiud/remove-windows-headers 36e6ed910 Merge pull request #635 from google/win-headers 7d60d6e04 removed windows specific headers 1041a221b Use generated config.h for Windows. 82320a8a8 Merge pull request #631 from xiaobfly/master bf183f44f Merge pull request #632 from sergiud/cmake-default-shared-libs 9ee010d85 cmake: build shared libs by default 4fca1b967 fixed filename extension 30bb9086d bazel: use expand_template instead of genrules d66e59318 Merge pull request #618 from sergiud/ac_cv___attribute___printf_4_5 d5fdcfefc Merge pull request #623 from sergiud/language-version 7f8a48b3b Merge pull request #595 from r-barnes/richard/ctr_type_fix 5abed00e3 Merge pull request #626 from sergiud/use-c++-headers bf3e5a80a Change counter type to uint64 01f3e543e use C++ headers 2f7d608a9 Merge pull request #627 from sergiud/cmake-no-c-compiler 3f4df6aee cmake: do not require a C compiler 7a6e743eb Merge pull request #624 from sergiud/qualify-std-string e27f14119 Merge pull request #622 from sergiud/max_log_size-overflow 0b3c27d04 fully qualify std::string 5c57edfbc readme: document C++ version 20984f973 Merge pull request #621 from sergiud/cmake-use-gtest 3533c22d2 handle --max_log_size overflow be25d94c2 cmake: optionally use gtest 45fdc922d Merge pull request #619 from sergiud/cmake-proper-symbol-export c30896d3d cmake: export symbols selectively b442ef807 build: set ac_cv___attribute___printf_4_5 d4e8ebab7 Add layering_check to Bazel CI. 0863ad797 bazel: declare headers in top-level rule ce50c57b9 Rename BUILD to BUILD.bazel. d824eae3c Merge pull request #613 from sergiud/cmake-unwind-patch-match df3ce126d cmake: fixed unwind patch component match check 246e49ac4 Merge pull request #610 from sergiud/readme-cleanup 9881ea596 Fix race conditions in LOG_EVERY_N (#492) 4cef72332 Merge pull request #611 from sergiud/incorrect-cygwin64-define 544b4a4f5 Merge pull request #609 from sergiud/cmake-unwind-empty-patch-component b9be6bb5d consolidate readme 6d14ac3ef Fix the symlink creation bug if using bazel 795384957 replaced __CYGWIN32__ and __CYGWIN64__ by __CYGWIN__ e370cd516 bump version 3b21da69e cmake: handle empty unwind version patch component 4fa737b62 Merge pull request #400 from guoyuhong/fixWebmailer 4f09bed0b Merge pull request #263 from kennyyu/tsan_benign_race 7bba6030c Merge pull request #592 from aesophor/fix-logcleaner-file-ext 7a8b66aa1 Merge pull request #593 from aesophor/fix-typo 6d5da2154 Merge pull request #608 from mahiuchun/stl 15d51a17c Make stl_logging_unittest more robust. fe1287dd5 src/logging.cc: Fix a typo in comment 431d74c80 src/logging.cc: Make LogCleaner aware of filename extension (#589) c8f8135a5 Merge pull request #502 from aesophor/encapsulate-log-cleaner d82641f70 Merge branch 'master' into encapsulate-log-cleaner cec37e1cf Merge pull request #542 from xulongwu4/master b55437ac1 Fix logmailer problem for dynamic lib. c5dcae830 Merge pull request #467 from sukill/master b30a009fa Merge pull request #585 from sergiud/remove-drop-autoconf-support b539557b3 removed autoconf support d516278b1 Merge pull request #540 from pwnall/fix-uv-warn 6c0a5fcb5 Fix unused parameter warning on gcc. f28ae960c Merge pull request #573 from msamoila/msamoila-use-utc-time fa0d50f9f Merge pull request #570 from jiuqiant/patch-1 fcc9da24a Merge pull request #545 from julianaito/master 40ab82281 Merge pull request #438 from d-uspenskiy/master 4c5a60c3c Merge pull request #551 from aesophor/replace-sprintf 8ee2bb5b3 Merge pull request #553 from huangqinjin/android 024b095d7 Merge pull request #552 from huangqinjin/master d44246069 Merge pull request #579 from arj-fb/safety_check_at_SymbolizeAndDemangle 5bc1e1177 Merge pull request #580 from Yicong-Huang/patch-1 af4df08e5 Merge pull request #576 from santigl/santigl/streambuf-overflow-override 725818984 Add `override` to `LogStreamBuf::overflow()` c652590d8 Update CMakeLists.txt e5ef2728b Added check for if info.dli_sname is NULL 909069ea8 Add FLAGS_log_utc_time; when 'true' the time will be written in log in UTC 64650ef2e Connect glog to Andorid logging API 0a2e5931b Fix build for Emscripten (#546) 0a12a96ce Output to logcat on Android d8998fe4f Make message text null-terminated 2e87f98fe Replace sprintf() with snprintf() (#536) 7001ab9ec OpenBSD/powerpc: clang does not define _CALL_* use proper ifdef 4a82a856d Use NULL to indicate uninitialized VLOG_IS_ON site. 3ba897659 fix bazel build for freebsd ceb716388 Add extra information in log file header (application build/version, actual duration time) 28321d895 Merge pull request #503 from alexsmac/fix-custom-logger-deletion 5b4fb63d2 Prepend the year to each glog line (#516) (#530) 7da49d48a Merge pull request #515 from segfaulthunter/doc 940335d82 Add vcpkg installation instructions (#517) dfc0e441c Merge pull request #523 from edbaunton/cmake-have-symbolize eb13e4f67 Allow CMake user to disable Symbolize functionality 195d416e3 Add target existence checks to Unwind find module (#518) 74076bebc Explain log levels. 130a3e10d Use file offsets in traces if object has no symbols (#513) 445af7ef7 Merge pull request #508 from xiaosuo/close a35e612c5 Don't call close(2) more than once 36fa99ba2 src/logging.cc: check the entire filename format 0a6704b43 Fix missing deletion of custom Logger 7764e4a8b src/logging.cc: match logfile with base_filename 3b5f2807f src/logging.cc: encapsulate log cleaner 1b7d54143 Add note about `namespace` on Win32. 95a365579 Remove unnecessary `defines` c0d2ceade Avoid leaking config.h 925858d99 add windows target to bazel CI 20919b325 add windows support for bazel bazed build 7f732529d apply buildifier on glog.bzl 1863b4228 Merge pull request #368 from UVV-gh/cmake-sdk-fix c0463b17b added cmake package config unit test 9a43cfb8d install unwind find module alongside glog 19570c69c renamed and cleaned up the unwind find module 58d7f873d Added fixed log name support, take 2 (#489) 0bf00f95a Merge pull request #491 from aesophor/update-doc 0f4975a68 doc: add information about log cleaner 5ef521d4a Merge pull request #487 from Telunsu/fix_wrong_assignment e58e596f1 src/logging.cc: fix wrong assignment f811ae9ed Merge pull request #485 from ffontaine/master 3ee89c86d src/symbolize.cc: do not check for HAVE_DLFCN_H for macOS 4d6aff418 Merge pull request #477 from ffontaine/master 742fe94f2 Use libunwind as an imported target 9630e0e84 src/utilities: fix build without pthread fd3dc2c63 Merge pull request #484 from sergiud/remove-executable-flag eec1435d5 removed executable flag from .cmake file e227697b0 Merge pull request #483 from loop0day/enhance 82d0860b7 Support pkg-config e0a2521e3 Merge pull request #482 from google/revert-19-master a6f7be14c Add support for automatic removal of old logs (#432) 15fb5ca5b Revert "Added fixed log name support" 5792d60d6 Merge pull request #441 from asekretenko/useconds 44f6079fa Merge pull request #19 from ezegomez/master 242130d30 Merge pull request #350 from TheModMaker/fixIosSim 1a4c47a74 Merge pull request #88 from kekstee/master f17d11aa8 Merge pull request #161 from mpercy/thread-local-libunwind-2 1227cf4da Merge pull request #312 from ruslo/pr.api21 0f83e53c4 Merge pull request #443 from ammubhave/fix_musl_build e558d50b0 Merge pull request #370 from hemantbits/patch-1 30f7f3ee9 Merge pull request #479 from neheb/patch-2 6f5699a23 Merge pull request #460 from neheb/patch-1 e24276a35 logging: Fix compilation with uClibc++ 1d762fa5b googletest: Switch to nanosleep 1655f4f91 Merge pull request #475 from ffontaine/master 10498b485 src/symbolize.cc: fix build without dlfcn.h 1e6b9f329 declare FLAGS_vmodule 4cc89c9e2 Merge pull request #465 from durswd/support_clang-cl a8fad525e Changed the order of conditions 00cb9f4e6 I supported clang-cl with BUILD_SHARED_LIBS ba8a9f695 fix typo: TOOD -> TODO (#448) 0d0c25431 Don't test on Ubuntu 14.04 (#446) 6ca3d3cf5 Fix stacktrace on bazel build. (#347) 834dd780b Fix symbolize_unittest for musl builds 94a54120d Extended the LogSink interface to be able to pass microseconds. 96a2f23dc Merge pull request #419 from shinh/release-0-4-0 5c576f78c Merge pull request #431 from romkatv/disallow_prefix 29dd5da60 Add Roman Perepelitsa to AUTHORS and CONTRIBUTORS 4db063134 Protect macros from user code to the left of them 460ec1f56 Merge branch 'master' into release-0-4-0 41f4bf9cb Merge pull request #425 from plmuon/remove-time-from-raw-logging 64041397b Use `static` to define CheckStackTrace on non-GNU (#429) 7f91846ca Stop using auto_ptr (#428) e5588de96 remove time from raw logging 7ffca211f fix appveyor link (#424) ed1ef7c4d Apply #397 to glog/logging.h.in 1431a17f3 Merge pull request #420 from kwaegel/patch-1 1d98d00c7 Update ChangeLog to record release dates at least 342fa2674 Use push/pop macro when supressing warning 406b0e702 glog: release 0.4.0 6d6d31872 Run src/windows/preprocess.sh 41a6a7880 Fix warnings a606c3187 Allow getpwuid_r to return missing entry. d064ab8d0 Fix build for Android API < 21 4764ca65f Annotate LOG_EVERY_N macros as a benign race for TSAN f581614a7 Use thread local for libunwind GetStackTrace() 47af307fa Added fixed log name support d68736bc1 Respect configured docdir git-subtree-dir: third_party/google-glog git-subtree-split: a1b6164ef15cf4be3b1914f2df7125fc7ffc7ca8 Signed-off-by: James Kuszmaul <james.kuszmaul@bluerivertech.com> Change-Id: I2286351db32cb46aa8743a28e191dfc9c62783da
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If the OOM killer kills a process, all it's handles get leaked. This is because the OOM killer doesn't run the robust futex cleanup to free the messages.
It actually looks like /proc/pid/stat has a start time for the thread. When opening the queue, we could look for any PIDs which are nonzero, see if they exist, and then check that the start time matches. If it does, it's pretty much guaranteed to be the same process, and is actually running.
The text was updated successfully, but these errors were encountered: