BRISE [Bitrise] Token

BRISE.swapETHForTokens(uint256) (Bitrise.sol#454-469) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
BRISE.swapAndSendToMarketing(uint256) (Bitrise.sol#417-425) sends eth to arbitrary user
Dangerous calls:
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#86-102):
External calls:
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#91)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (DividendPayingToken.sol#94)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
State variables written after the call(s):
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- swapping = true (Bitrise.sol#92)
- swapping = false (Bitrise.sol#94)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (Bitrise.sol#397)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- super._transfer(from,to,amount) (Bitrise.sol#400)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- swapping = true (Bitrise.sol#92)
- swapping = false (Bitrise.sol#94)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
State variables written after the call(s):
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- swapping = true (Bitrise.sol#92)
- swapping = false (Bitrise.sol#94)
Apply the check-effects-interactions pattern.

Additional information: link

BRISE._transfer(address,address,uint256).contractTokenBalance (Bitrise.sol#354) is written in both
contractTokenBalance = swapTokensAtAmount (Bitrise.sol#369)
contractTokenBalance = balanceOf(address(this)) (Bitrise.sol#374)
Fix or remove the writes.

Additional information: link

Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains

Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.

Contract ownership is not renounced (belongs to a wallet)

SafeMathInt.MAX_INT256 (SafeMathInt.sol#36) is never used in SafeMathInt (SafeMathInt.sol#34-92)
Remove unused state variables.

Additional information: link

BRISE.setMarketingWallet(address).newWallet (Bitrise.sol#307) lacks a zero-check on :
- _marketingWallet = address(newWallet) (Bitrise.sol#308)
Check that the address is not zero.

Additional information: link

BRISEDividendTracker.getAccount(address) (Bitrise.sol#555-598) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (Bitrise.sol#595-597)
BRISEDividendTracker.canAutoClaim(uint256) (Bitrise.sol#619-625) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (Bitrise.sol#620)
- block.timestamp.sub(lastClaimTime) >= claimWait (Bitrise.sol#624)
Avoid relying on block.timestamp.

Additional information: link

Redundant expression "this (Context.sol#21)" inContext (Context.sol#15-24)
Remove redundant statements if they congest code but offer no value.

Additional information: link

BRISE._transfer(address,address,uint256).iterations (Bitrise.sol#408) is a local variable never initialized
BRISE._transfer(address,address,uint256).lastProcessedIndex (Bitrise.sol#408) is a local variable never initialized
BRISE.addToBlackList(address[]).i (Bitrise.sol#312) is a local variable never initialized
BRISE._transfer(address,address,uint256).claims (Bitrise.sol#408) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

BRISE.claim() (Bitrise.sol#279-281) ignores return value by dividendTracker.processAccount(msg.sender,false) (Bitrise.sol#280)
BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415) ignores return value by dividendTracker.process(gas) (Bitrise.sol#408-413)
Ensure that all the return values of the function calls are used.

Additional information: link

Variable 'BRISE._transfer(address,address,uint256).claims (Bitrise.sol#408)' in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (Bitrise.sol#409)
Variable 'BRISE._transfer(address,address,uint256).lastProcessedIndex (Bitrise.sol#408)' in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (Bitrise.sol#409)
Variable 'BRISE._transfer(address,address,uint256).iterations (Bitrise.sol#408)' in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (Bitrise.sol#409)
Move all variable declarations prior to any usage of the variable, and ensure that reaching a variable declaration does not depend on some conditional if it is used unconditionally.

Additional information: link

Low level call in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#86-102):
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#91)
Low level call in BRISE.swapAndSendDividends(uint256) (Bitrise.sol#483-492):
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- _allowances[owner][spender] = amount (ERC20.sol#287)
- super._transfer(from,address(this),fees) (Bitrise.sol#397)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- super._transfer(from,to,amount) (Bitrise.sol#400)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- swapping = true (Bitrise.sol#92)
- swapping = false (Bitrise.sol#94)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (Bitrise.sol#409)
- SendDividends(tokens,dividends) (Bitrise.sol#490)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (Bitrise.sol#400)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (Bitrise.sol#397)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
State variables written after the call(s):
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- _allowances[owner][spender] = amount (ERC20.sol#287)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- swapping = true (Bitrise.sol#92)
- swapping = false (Bitrise.sol#94)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in BRISE.updateDividendTracker(address) (Bitrise.sol#148-163):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (Bitrise.sol#155)
- newDividendTracker.excludeFromDividends(address(this)) (Bitrise.sol#156)
- newDividendTracker.excludeFromDividends(owner()) (Bitrise.sol#157)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (Bitrise.sol#158)
State variables written after the call(s):
- dividendTracker = newDividendTracker (Bitrise.sol#162)
Apply the check-effects-interactions pattern.

Additional information: link

DividendPayingToken.constructor(string,string)._name (DividendPayingToken.sol#43) shadows:
- ERC20._name (ERC20.sol#43) (state variable)
BRISE._transfer(address,address,uint256).buyBackTokens (Bitrise.sol#376) shadows:
- BRISE.buyBackTokens(uint256) (Bitrise.sol#448-452) (function)
DividendPayingToken.constructor(string,string)._symbol (DividendPayingToken.sol#43) shadows:
- ERC20._symbol (ERC20.sol#44) (state variable)
Rename the local variables that shadow another component.

Additional information: link

get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (IterableMapping.sol#13-15)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (ERC20.sol#151-159)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (IterableMapping.sol#24-26)
isExcludedFromFees(address) should be declared external:
- BRISE.isExcludedFromFees(address) (Bitrise.sol#231-233)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (Ownable.sol#43-46)
getAccountAtIndex(uint256) should be declared external:
- BRISEDividendTracker.getAccountAtIndex(uint256) (Bitrise.sol#600-617)
dividendTokenBalanceOf(address) should be declared external:
- BRISE.dividendTokenBalanceOf(address) (Bitrise.sol#239-241)
updateLiquidityWallet(address) should be declared external:
- BRISE.updateLiquidityWallet(address) (Bitrise.sol#205-210)
process(uint256) should be declared external:
- BRISEDividendTracker.process(uint256) (Bitrise.sol#644-689)
withdrawDividend() should be declared external:
- BRISEDividendTracker.withdrawDividend() (Bitrise.sol#524-526)
- DividendPayingToken.withdrawDividend() (DividendPayingToken.sol#80-82)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (DividendPayingToken.sol#108-110)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (IterableMapping.sol#17-22)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (ERC20.sol#192-195)
symbol() should be declared external:
- ERC20.symbol() (ERC20.sol#71-73)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (ERC20.sol#114-117)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (ERC20.sol#173-176)
withdrawableDividendOf(address) should be declared external:
- BRISE.withdrawableDividendOf(address) (Bitrise.sol#235-237)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- BRISE.excludeMultipleAccountsFromFees(address[],bool) (Bitrise.sol#178-184)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (IterableMapping.sol#30-32)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (Ownable.sol#52-56)
updateUniswapV2Router(address) should be declared external:
- BRISE.updateUniswapV2Router(address) (Bitrise.sol#165-169)
name() should be declared external:
- ERC20.name() (ERC20.sol#63-65)
updateGasForProcessing(uint256) should be declared external:
- BRISE.updateGasForProcessing(uint256) (Bitrise.sol#212-217)
updateDividendTracker(address) should be declared external:
- BRISE.updateDividendTracker(address) (Bitrise.sol#148-163)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (ERC20.sol#133-136)
decimals() should be declared external:
- BRISE.decimals() (Bitrise.sol#144-146)
- ERC20.decimals() (ERC20.sol#88-90)
setBuyBackEnabled(bool) should be declared external:
- BRISE.setBuyBackEnabled(bool) (Bitrise.sol#475-477)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (DividendPayingToken.sol#122-124)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (ERC20.sol#122-124)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- BRISE.setAutomatedMarketMakerPair(address,bool) (Bitrise.sol#187-191)
Use the external attribute for functions never called from the contract.

Additional information: link

Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (DividendPayingToken.sol#87) is too similar to BRISEDividendTracker.getAccount(address).withdrawableDividends (Bitrise.sol#560)
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (IUniswapV2Router.sol#12) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (IUniswapV2Router.sol#13)
Prevent variables from having similar names.

Additional information: link

Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in BRISEDividendTracker.processAccount(address,bool) (Bitrise.sol#691-701):
External calls:
- amount = _withdrawDividendOfUser(account) (Bitrise.sol#692)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#91)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (Bitrise.sol#695)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
State variables written after the call(s):
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
State variables written after the call(s):
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Apply the check-effects-interactions pattern.

Additional information: link

BRISE.setBuybackUpperLimit(uint256) (Bitrise.sol#479-481) should emit an event for:
- buyBackUpperLimit = buyBackLimit * 10 ** 15 (Bitrise.sol#480)
BRISE.setMarketingFee(uint256) (Bitrise.sol#303-305) should emit an event for:
- marketingFee = value (Bitrise.sol#304)
BRISE.setSwapTokensAmt(uint256) (Bitrise.sol#295-297) should emit an event for:
- swapTokensAtAmount = amt (Bitrise.sol#296)
BRISE.setBuyBackFee(uint256) (Bitrise.sol#326-328) should emit an event for:
- buyBackFee = value (Bitrise.sol#327)
BRISE.setMaxSellTxAMount(uint256) (Bitrise.sol#291-293) should emit an event for:
- maxSellTransactionAmount = amount (Bitrise.sol#292)
Emit an event for critical parameter changes.

Additional information: link

DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#86-102) has external calls inside a loop: (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#91)
Favor pull over push strategy for external calls.

Additional information: link

SafeMathInt.div(int256,int256) (SafeMathInt.sol#53-59) is never used and should be removed
SafeMathInt.abs(int256) (SafeMathInt.sol#82-85) is never used and should be removed
SafeMathInt.mul(int256,int256) (SafeMathInt.sol#41-48) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (SafeMath.sol#142-145) is never used and should be removed
Context._msgData() (Context.sol#20-23) is never used and should be removed
SafeMath.mod(uint256,uint256) (SafeMath.sol#126-128) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (DividendPayingToken.sol#142-148) is never used and should be removed
Remove unused functions.

Additional information: link

Reentrancy in BRISE.processDividendTracker(uint256) (Bitrise.sol#274-277):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (Bitrise.sol#275)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (Bitrise.sol#276)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
Reentrancy in BRISE.swapAndSendDividends(uint256) (Bitrise.sol#483-492):
External calls:
- swapTokensForEth(tokens) (Bitrise.sol#485)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
External calls sending eth:
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (Bitrise.sol#490)
Reentrancy in BRISE.swapETHForTokens(uint256) (Bitrise.sol#454-469):
External calls:
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
Event emitted after the call(s):
- SwapETHForTokens(amount,path) (Bitrise.sol#468)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- dividendTracker.setBalance(address(from),balanceOf(from)) (Bitrise.sol#402)
- dividendTracker.setBalance(address(to),balanceOf(to)) (Bitrise.sol#403)
- dividendTracker.process(gas) (Bitrise.sol#408-413)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (Bitrise.sol#409)
Reentrancy in BRISE.updateDividendTracker(address) (Bitrise.sol#148-163):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (Bitrise.sol#155)
- newDividendTracker.excludeFromDividends(address(this)) (Bitrise.sol#156)
- newDividendTracker.excludeFromDividends(owner()) (Bitrise.sol#157)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (Bitrise.sol#158)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (Bitrise.sol#160)
Reentrancy in BRISEDividendTracker.processAccount(address,bool) (Bitrise.sol#691-701):
External calls:
- amount = _withdrawDividendOfUser(account) (Bitrise.sol#692)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (DividendPayingToken.sol#91)
Event emitted after the call(s):
- Claim(account,amount,automatic) (Bitrise.sol#696)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
Reentrancy in BRISE._setAutomatedMarketMakerPair(address,bool) (Bitrise.sol#193-202):
External calls:
- dividendTracker.excludeFromDividends(pair) (Bitrise.sol#198)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (Bitrise.sol#201)
Reentrancy in BRISE._transfer(address,address,uint256) (Bitrise.sol#330-415):
External calls:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapBuyBackTokens(buyBackTokens) (Bitrise.sol#377)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (Bitrise.sol#438-444)
External calls sending eth:
- buyBackTokens(balance.div(100)) (Bitrise.sol#363)
- uniswapV2Router.swapExactETHForTokensSupportingFeeOnTransferTokens{value: amount}(0,path,deadAddress,block.timestamp.add(300)) (Bitrise.sol#461-466)
- swapAndSendToMarketing(swapTokens) (Bitrise.sol#372)
- _marketingWallet.transfer(newBalance) (Bitrise.sol#424)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- (success) = address(dividendTracker).call{value: dividends}() (Bitrise.sol#487)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- SendDividends(tokens,dividends) (Bitrise.sol#490)
- swapAndSendDividends(sellTokens) (Bitrise.sol#380)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (Bitrise.sol#397)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (Bitrise.sol#400)
Apply the check-effects-interactions pattern.

Additional information: link

BRISEDividendTracker.getAccountAtIndex(uint256) (Bitrise.sol#600-617) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (Bitrise.sol#611)
BRISE.updateGasForProcessing(uint256) (Bitrise.sol#212-217) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,BRISE: gasForProcessing must be between 200,000 and 500,000) (Bitrise.sol#213)
BRISE.constructor() (Bitrise.sol#97-138) uses literals with too many digits:
- _mint(owner(),1000000000000000 * (10 ** 9)) (Bitrise.sol#137)
BRISE.slitherConstructorVariables() (Bitrise.sol#14-493) uses literals with too many digits:
- deadAddress = 0x000000000000000000000000000000000000dEaD (Bitrise.sol#19)
BRISE.slitherConstructorVariables() (Bitrise.sol#14-493) uses literals with too many digits:
- swapTokensAtAmount = 100000000000 * (10 ** 9) (Bitrise.sol#31)
BRISE.slitherConstructorVariables() (Bitrise.sol#14-493) uses literals with too many digits:
- maxSellTransactionAmount = 1000000000000000 * (10 ** 9) (Bitrise.sol#30)
BRISE.slitherConstructorVariables() (Bitrise.sol#14-493) uses literals with too many digits:
- gasForProcessing = 300000 (Bitrise.sol#45)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Pragma version^0.6.2 (IUniswapV2Factory.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingToken.sol#3) allows old versions
Pragma version^0.6.2 (IERC20Metadata.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Pair.sol#3) allows old versions
Pragma version^0.6.2 (SafeMathUint.sol#3) allows old versions
Pragma version^0.6.2 (SafeMathInt.sol#28) allows old versions
Pragma version^0.6.2 (IERC20.sol#3) allows old versions
Pragma version^0.6.2 (ERC20.sol#3) allows old versions
Pragma version^0.6.2 (Context.sol#3) allows old versions
Pragma version^0.6.2 (SafeMath.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Router.sol#3) allows old versions
Pragma version^0.6.2 (Bitrise.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingTokenInterface.sol#3) allows old versions
Pragma version^0.6.2 (IterableMapping.sol#2) allows old versions
Pragma version^0.6.2 (DividendPayingTokenOptionalInterface.sol#3) allows old versions
Pragma version^0.6.2 (Ownable.sol#1) allows old versions
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.

Additional information: link

Variable BRISE._marketingWallet (Bitrise.sol#42) is not in mixedCase
Variable BRISE.BNBRewardsFee (Bitrise.sol#33) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (IUniswapV2Pair.sol#21) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (IUniswapV2Pair.sol#38) is not in mixedCase
Constant DividendPayingToken.magnitude (DividendPayingToken.sol#23) is not in UPPER_CASE_WITH_UNDERSCORES
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (IUniswapV2Pair.sol#20) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#108) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#122) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#115) is not in mixedCase
Parameter BRISEDividendTracker.getAccount(address)._account (Bitrise.sol#555) is not in mixedCase
Variable BRISE._isBlacklisted (Bitrise.sol#21) is not in mixedCase
Function IUniswapV2Router01.WETH() (IUniswapV2Router.sol#7) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#132) is not in mixedCase
Parameter BRISE.setBuyBackEnabled(bool)._enabled (Bitrise.sol#475) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Contract has 12% buy tax and 12% sell tax.
Taxes are suspiciously high (over 10%) and contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d PancakeSwap liquidity is less than $100. Token is either dead or inactive.

Average 30d number of PancakeSwap swaps is less than 1. Token is either dead or inactive.