fix: annotate static variable in VLOG_IS_ON (#890)

Avoid TSAN issues when multiple threads perform the check.
google · Oct 7, 2023 · e17f932 · e17f932
1 parent f6e4d96
commit e17f932
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/glog/vlog_is_on.h.in b/src/glog/vlog_is_on.h.in
@@ -75,6 +75,8 @@
 #define VLOG_IS_ON(verboselevel)                                \
   __extension__  \
   ({ static @ac_google_namespace@::SiteFlag vlocal__ = {NULL, NULL, 0, NULL};       \
+     GLOG_IFDEF_THREAD_SANITIZER( \
+             AnnotateBenignRaceSized(__FILE__, __LINE__, &vlocal__, sizeof(@ac_google_namespace@::SiteFlag), "")); \
      @ac_google_namespace@::int32 verbose_level__ = (verboselevel);                    \
      (vlocal__.level == NULL ? @ac_google_namespace@::InitVLOG3__(&vlocal__, &FLAGS_v, \
                         __FILE__, verbose_level__) : *vlocal__.level >= verbose_level__); \