From 60c655413413360b65ee6bec50012d38a97830cb Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Thu, 16 Dec 2021 00:32:24 +0100 Subject: proto: revisit short-circuit loops over upper protocols Move the check for NULL protocol description away from the loop to avoid too long line. Signed-off-by: Pablo Neira Ayuso --- src/proto.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'src/proto.c') diff --git a/src/proto.c b/src/proto.c index 31a2f380..a013a00d 100644 --- a/src/proto.c +++ b/src/proto.c @@ -59,8 +59,9 @@ proto_find_upper(const struct proto_desc *base, unsigned int num) { unsigned int i; - for (i = 0; i < array_size(base->protocols) && base->protocols[i].desc; - i++) { + for (i = 0; i < array_size(base->protocols); i++) { + if (!base->protocols[i].desc) + break; if (base->protocols[i].num == num) return base->protocols[i].desc; } @@ -78,8 +79,9 @@ int proto_find_num(const struct proto_desc *base, { unsigned int i; - for (i = 0; i < array_size(base->protocols) && base->protocols[i].desc; - i++) { + for (i = 0; i < array_size(base->protocols); i++) { + if (!base->protocols[i].desc) + break; if (base->protocols[i].desc == desc) return base->protocols[i].num; } @@ -107,9 +109,9 @@ int proto_dev_type(const struct proto_desc *desc, uint16_t *res) *res = dev_proto_desc[i].type; return 0; } - for (j = 0; j < array_size(base->protocols) && - base->protocols[j].desc; - j++) { + for (j = 0; j < array_size(base->protocols); j++) { + if (!base->protocols[j].desc) + break; if (base->protocols[j].desc == desc) { *res = dev_proto_desc[i].type; return 0; -- cgit v1.2.3