# Chingari Token Audit Notes **Auditor:** Jake Bunce **Client:** Chingari **Commit:** `1a609d9c96a3bb3c84b727836f865eacefe8391b` ## Review of the protocol/implementation ## Best Practices **[1] Lack of docstrings** There is a lack of docstrings on intended function outcomes. **[2] Tests only run on one platform** ``` npm ERR! notsup Valid OS: linux npm ERR! notsup Valid Arch: x64 npm ERR! notsup Actual OS: darwin npm ERR! notsup Actual Arch: arm64 ```
Last changed by  
 
Jake Bunce
182

Read more

LayerZero Re-Re-audit

Pool.sol At the struct ChainPath, we place the bool ready (line 27, line 389) as the first variable to compactly pack the Chainpath. Affects gas utilisation only, but agreed. Router.sol In redeemLocal (line 184), if the redeem amount is 0, it should revert instead of sending the credit out. This is not a bug but will impact UX. This change will not introduce any functional problem to the system

 Agreed

3/11/2022
Nifty Apes

Auditor: Jake Bunce Client: Nifty Apes Commit: 799ca93a8eed0a0750a4245f8f695779077784c4 Review of the protocol/implementation [1] Lack of documentation Severity: High

3/8/2022
LayerZero Audit Notes

Auditor: Jake Bunce Client: LayerZero https://layerzero.network/ https://www.dropbox.com/s/dm8zqpr04sibk7a/LayerZero_Whitepaper_Release.pdf?dl=1 Commit: b387062eed56eb0b74104d9e986779e9d2253585 Review of the protocol/implementation [1] Solidity version and SafeMath

2/3/2022
KeeperDAO-6 Audit Notes

Auditor: Jake Bunce Client: KepperDAO https://www.keeperdao.com/ https://github.com/keeperdao/ Commit: 3eb359bed1d21813124b904c5ca5c54cc1c43373 Review of the protocol/implementation [1] Claimable addition event

1/21/2022
Read more from Jake Bunce
Published on HackMD