Recommendations for which sanitizers to enable
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Documentation
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- cpp
- Domaine
- documentation, testing-qa
Piste de recherche
Commencez par comparer les regroupements de sanitizers dans les fichiers Sanitizers.def référencés avec la documentation de AddressSanitizer, ThreadSanitizer, MemorySanitizer, UndefinedBehaviorSanitizer, DataFlowSanitizer, LeakSanitizer, Control Flow Integrity et Safe Stack. Clarifiez les recommandations concernant les tests toujours activés, la CI et les builds de live-testing, y compris les flags, les variables d’environnement et les limites de plateforme pertinentes. Le travail est considéré comme terminé lorsque l’issue contient une référence concise et claire couvrant ces questions.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
As time passes, the g++/clang++ gain more new sanitizers, existing ones get better, and some get ported to new platforms (e.g. recently leak support was added to OS X in clang++ 5).
Similar to #37 for warnings, This ticket can serve as a central place to start documenting a recommended list of sanitizer flags to use. I intend this to be a quick, opinionated reference to check before you would dive deeper into docs like https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html.
The relevant questions I see are:
- 🍇 Which sanitizers should we always test with?
- 🍊 Which additional flags or environment variables should we use?
- 🍏 Which additional flags should be used when testing the sanitizers on CI systems?
- 🍎 Which additional flags should be used when building sanitized binaries for live testing?
Refs:
- source of truth for which sanitizers are in which groupings:
- Docs for HEAD llvm for sanitizers:
- Address: https://clang.llvm.org/docs/AddressSanitizer.html
- Thread: https://clang.llvm.org/docs/ThreadSanitizer.html
- Memory: https://clang.llvm.org/docs/MemorySanitizer.html
- UndefinedBehavior: https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html
- DataFlow: https://clang.llvm.org/docs/DataFlowSanitizer.html
- Leak: https://clang.llvm.org/docs/LeakSanitizer.html
- Control Flow Integrity: https://clang.llvm.org/docs/ControlFlowIntegrity.html
- Safe Stack: https://clang.llvm.org/docs/SafeStack.html
- Langage dominant
- Aucune donnée de langage
- Étoiles
- 110
- Forks
- 17
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de mapbox/cpp
-
glossary
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
-
Docs on ABIs Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
-
clang-tidy v. -Weffc++ Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
-
docs/learning-resources.md blank Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Toutes les issues de mapbox/cpp
Issues similaires
-
The correction-threshold rationale says 17 restraint cases collide; the current corpus has 16 Ouvertearea:dictation documentation P2
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
uttrflow/uttrflow-swift#1180 ·
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
githubnext/gh-aw-workshop#3692 ·
-
agent/guide documentation hive/hosted-available-lke648397-260827-5n31
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
-
kind/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
falcosecurity/falco-website#1601 ·
-
Improve privacy considerations Ouvertespecification
Difficulté 1/5 1-3 heures Accessibilité débutants 78/100
WICG/webmonetization#729 ·