langerhans / dogecoin-wallet-new

Dogecoin Wallet app for your Android device. Standalone Dogecoin node, no centralized backend required. Forked from the popular Bitcoin Wallet.
189 stars 105 forks source link

Cryptographic APIs misuses #120

Closed misterAnderson90 closed 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 54 warnings (indicating potential vulnerabilities) when running the CogniCrypt static analyzer (*) on dogecoin-wallet-new (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 dogecoin-wallet-new's security, and the quality of the reports of static analysis tools. (*) https://github.com/CROSSINGTUD/CryptoAnalysis

misterAnderson90 commented 2 years ago

Hello @langerhans,

Do you want to receive the report with these warnings? Otherwise, should I close this issue?