Avoid potential null dereference in ReceiveCoinsDialog constructor #12211

pull ryanofsky wants to merge 1 commits into bitcoin:master from ryanofsky:pr/nullrecv changing 1 files +9 −9

ryanofsky commented at 6:30 PM on January 17, 2018: member

Not a bug in practice because current WalletModel::getDefaultAddressType() implementation does not dereference its this pointer.

Encountered issue while rebasing #10244 after #11991 was merged.

Avoid potential null dereference in ReceiveCoinsDialog constructor

Not a bug in practice because current WalletModel::getDefaultAddressType()
implementation does not dereference its `this` pointer.

cc90a4f46b

promag commented at 10:07 PM on January 17, 2018: member

ACK cc90a4f.
laanwj added the label GUI on Jan 18, 2018
laanwj commented at 9:22 AM on January 18, 2018: member

utACK https://github.com/bitcoin/bitcoin/commit/cc90a4f46bfff17ca358eedf29d2b77117ad4465
jonasschnelli approved
jonasschnelli commented at 7:46 PM on January 18, 2018: contributor

utACK cc90a4f46bfff17ca358eedf29d2b77117ad4465
jonasschnelli merged this on Jan 18, 2018
jonasschnelli closed this on Jan 18, 2018
jonasschnelli referenced this in commit 10d10d7fad on Jan 18, 2018
PastaPastaPasta referenced this in commit 6bfb836bf9 on Jan 17, 2020
PastaPastaPasta referenced this in commit 81e73ad07a on Jan 17, 2020
PastaPastaPasta referenced this in commit 4d560c1f55 on Feb 12, 2020
PastaPastaPasta referenced this in commit 107fffd079 on Feb 12, 2020
DrahtBot locked this on Sep 8, 2021

Contributors

Labels