search

decred / dcraddrscanner

ISC License
5 stars 5 forks source link

Cryptographic APIs misuses #32

Open misterAnderson90 opened 2 years ago

misterAnderson90 commented 2 years ago

I'm a PhD student interested in finding security vulnerabilities in open source projects.

We found a total of 7 warnings (indicating potential vulnerabilities) when running the CogniCrypt static analyzer (*) on dcraddrscanner (or its library dependencies). We documented each one of these issues in private gists for the sake of confidentiality (non-disclosure).

Can you please let us know whether we can share these gists with you? We are eager to evaluate the perception of developers (e.g. severity of these warnings) and improve dcraddrscanner's security, and the quality of the reports of static analysis tools. (*) https://github.com/CROSSINGTUD/CryptoAnalysis

JoeGruffins commented 2 years ago

Yes please share.

On Sat, Dec 18, 2021, 6:07 AM Luis Guilherme @.***> wrote:

I'm a PhD student interested in finding security vulnerabilities in open source projects.

We found a total of 7 warnings (indicating potential vulnerabilities) when running the CogniCrypt static analyzer (*) on dcraddrscanner (or its library dependencies). We documented each one of these issues in private gists for the sake of confidentiality (non-disclosure).

Can you please let us know whether we can share these gists with you? We are eager to evaluate the perception of developers (e.g. severity of these warnings) and improve dcraddrscanner's security, and the quality of the reports of static analysis tools. (*) https://github.com/CROSSINGTUD/CryptoAnalysis

— Reply to this email directly, view it on GitHub https://github.com/decred/dcraddrscanner/issues/32, or unsubscribe https://github.com/notifications/unsubscribe-auth/AILARIIFNGOZN2TYZ5VDCU3URORA3ANCNFSM5KJW2V7A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

misterAnderson90 commented 2 years ago

Hello @JoeGruffins,

From the 7 warnings that CogniCrypt reported, we randomly selected 5 gists to share with you. If you are interested, we can share the reports of the remaining warnings.

Gist 01 - MessageDigest Gist 02 - MessageDigest Gist 03 - TrustManagerFactory Gist 04 -SLLContext Gist 05 - SSLContext

For any doubts, you can comment directly in the gists and we can clarify the issues.

Best regards,

misterAnderson90 commented 2 years ago

Hi @JoeGruffins and @dnldd,

How should I proceed with this issue? Are you interested in discussing these warnings? How do you perceive their value?

Thanks a lot for your collaboration.