boostorg / boostorg/interprocess

Security Vulnerability: Security Descriptor NULL DACLs

Aperta
#107 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C++
Stelle
185
Fork
131
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Came across this one while viewing old TRAC bugs. It's pretty recent and I assume the poster didn't know to put it here (and they are anonymous) so... here's the text:

> Our security team has flagged: if(SetSecurityDescriptorDacl(&sd, true, 0, false)) in interprocess\detail\win32_api.hpp as a "high-priority" vulnerability

They then reference some of the text from the [C6248 warning](https://docs.microsoft.com/en-us/cpp/code-quality/c6248?view=msvc-170) which says in part:

>Objects that have null DACLs can have their security descriptors altered by malicious users so that no one has access to the object.
>
>Even if everyone needs access to an object, the object should be secured so that only administrators can alter its security. If only the creator needs access to an object, a DACL should not be set on the object; the system will choose an appropriate default.

Looks like this could be fixed with a little research if someone was available to do the work.

[TRAC 13466](https://svn.boost.org/trac10/ticket/13466)

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Iniziare in interprocess/detail/win32_api.hpp, alla chiamata a SetSecurityDescriptorDacl, quindi leggere l'avviso C6248 e TRAC 13466 collegati per il contesto. Determinare una DACL appropriata non-NULL o un comportamento di sicurezza equivalente per l'oggetto interprocess e verificare il comportamento specifico di Windows con i test esistenti del progetto, se disponibili.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
operating-systems, security
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.