diff options
author | hiago <hiago.dutra@gmail.com> | 2022-04-14 17:09:13 -0300 |
---|---|---|
committer | hiago <hiago.dutra@gmail.com> | 2022-04-22 09:45:12 -0300 |
commit | 172c2333f03aecb4c347c791537e13c296adbde2 (patch) | |
tree | d724e016fbb48e27027084c0dede73b727395bcf /test/lint/lint-assertions.py | |
parent | e14f0fa6a346afecbb1d5470aef5226a8cc33e57 (diff) | |
download | bitcoin-172c2333f03aecb4c347c791537e13c296adbde2.tar.xz |
Porting lint-assertions.sh to lint-assertions.py
Diffstat (limited to 'test/lint/lint-assertions.py')
-rwxr-xr-x | test/lint/lint-assertions.py | 52 |
1 files changed, 52 insertions, 0 deletions
diff --git a/test/lint/lint-assertions.py b/test/lint/lint-assertions.py new file mode 100755 index 0000000000..a8d2b3927c --- /dev/null +++ b/test/lint/lint-assertions.py @@ -0,0 +1,52 @@ +#!/usr/bin/env python3 +# +# Copyright (c) 2018-2022 The Bitcoin Core developers +# Distributed under the MIT software license, see the accompanying +# file COPYING or http://www.opensource.org/licenses/mit-license.php. +# +# Check for assertions with obvious side effects. + +import sys +import subprocess + + +def git_grep(params: [], error_msg: ""): + try: + output = subprocess.check_output(["git", "grep", *params], universal_newlines=True, encoding="utf8") + print(error_msg) + print(output) + return 1 + except subprocess.CalledProcessError as ex1: + if ex1.returncode > 1: + raise ex1 + return 0 + + +def main(): + # PRE31-C (SEI CERT C Coding Standard): + # "Assertions should not contain assignments, increment, or decrement operators." + exit_code = git_grep([ + "-E", + r"[^_]assert\(.*(\+\+|\-\-|[^=!<>]=[^=!<>]).*\);", + "--", + "*.cpp", + "*.h" + ], "Assertions should not have side effects:") + + # Macro CHECK_NONFATAL(condition) should be used instead of assert for RPC code, where it + # is undesirable to crash the whole program. See: src/util/check.h + # src/rpc/server.cpp is excluded from this check since it's mostly meta-code. + exit_code |= git_grep([ + "-nE", + r"\<(A|a)ssert *\(.*\);", + "--", + "src/rpc/", + "src/wallet/rpc*", + ":(exclude)src/rpc/server.cpp" + ], "CHECK_NONFATAL(condition) should be used instead of assert for RPC code.") + + sys.exit(exit_code) + + +if __name__ == "__main__": + main() |