diff options
author | Paul Moore <paul@paul-moore.com> | 2020-03-19 15:18:46 -0400 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2020-03-19 16:14:21 -0400 |
commit | 4f17cc4d2c74da99ccd619d2e9c8f41ac6ba2e2c (patch) | |
tree | a38542826ba77fa87b743569f4bda33ed82fce68 /tools | |
parent | 60ee8da07e42edbf22a5ecb0e258e04304b397fb (diff) | |
download | libseccomp-4f17cc4d2c74da99ccd619d2e9c8f41ac6ba2e2c.tar.gz |
all: fix all the outstanding lgtm.com alerts
There are no functional changes in this patch, just some minor
changes found by the lgtm.com service:
* four functions in tools/util.c were "hiding" a global variable
with a local variable ("arch")
* src/arch.c had an unnecessary check in an if-condition
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/util.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/tools/util.c b/tools/util.c index 741b2a2..5687b30 100644 --- a/tools/util.c +++ b/tools/util.c @@ -89,15 +89,15 @@ uint32_t arch = ARCH_NATIVE; /** * Convert a 16-bit target integer into the host's endianess - * @param arch the architecture token + * @param arch_token the architecture token * @param val the 16-bit integer * * Convert the endianess of the supplied value and return it to the caller. * */ -uint16_t ttoh16(uint32_t arch, uint16_t val) +uint16_t ttoh16(uint32_t arch_token, uint16_t val) { - if (arch & __AUDIT_ARCH_LE) + if (arch_token & __AUDIT_ARCH_LE) return le16toh(val); else return be16toh(val); @@ -105,15 +105,15 @@ uint16_t ttoh16(uint32_t arch, uint16_t val) /** * Convert a 32-bit target integer into the host's endianess - * @param arch the architecture token + * @param arch_token the architecture token * @param val the 32-bit integer * * Convert the endianess of the supplied value and return it to the caller. * */ -uint32_t ttoh32(uint32_t arch, uint32_t val) +uint32_t ttoh32(uint32_t arch_token, uint32_t val) { - if (arch & __AUDIT_ARCH_LE) + if (arch_token & __AUDIT_ARCH_LE) return le32toh(val); else return be32toh(val); @@ -121,15 +121,15 @@ uint32_t ttoh32(uint32_t arch, uint32_t val) /** * Convert a 32-bit host integer into the target's endianess - * @param arch the architecture token + * @param arch_token the architecture token * @param val the 32-bit integer * * Convert the endianess of the supplied value and return it to the caller. * */ -uint32_t htot32(uint32_t arch, uint32_t val) +uint32_t htot32(uint32_t arch_token, uint32_t val) { - if (arch & __AUDIT_ARCH_LE) + if (arch_token & __AUDIT_ARCH_LE) return htole32(val); else return htobe32(val); @@ -137,15 +137,15 @@ uint32_t htot32(uint32_t arch, uint32_t val) /** * Convert a 64-bit host integer into the target's endianess - * @param arch the architecture token + * @param arch_token the architecture token * @param val the 64-bit integer * * Convert the endianess of the supplied value and return it to the caller. * */ -uint64_t htot64(uint32_t arch, uint64_t val) +uint64_t htot64(uint32_t arch_token, uint64_t val) { - if (arch & __AUDIT_ARCH_LE) + if (arch_token & __AUDIT_ARCH_LE) return htole64(val); else return htobe64(val); |