diff options
| author |   deraadt <deraadt@openbsd.org> | 2017-04-13 14:48:31 +0000 |
|---|---|---|
| committer | deraadt <deraadt@openbsd.org> | 2017-04-13 14:48:31 +0000 |
| commit | dd8c3f8b17e4367358a5e87fd78838b6a63f2986 (patch) | |
| tree | a96ad4185b90ee7c279cbbcbfa74626d1672957a /usr.sbin/ldapd | |
| parent | fix wrongly indented lines (diff) | |
| download | wireguard-openbsd-dd8c3f8b17e4367358a5e87fd78838b6a63f2986.tar.xz wireguard-openbsd-dd8c3f8b17e4367358a5e87fd78838b6a63f2986.zip | |
multi-statement CHECK_RANGE() macro isn't safe for all placements, and
needs to use "do {} while 0" idiom; all callers need repair also.
Discovered by jsg
Diffstat (limited to 'usr.sbin/ldapd')
| -rw-r--r-- | usr.sbin/ldapd/syntax.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/usr.sbin/ldapd/syntax.c b/usr.sbin/ldapd/syntax.c index 27f73527248..fb3a3c0c51e 100644 --- a/usr.sbin/ldapd/syntax.c +++ b/usr.sbin/ldapd/syntax.c @@ -1,4 +1,4 @@ -/* $OpenBSD: syntax.c,v 1.3 2015/10/11 03:23:28 guenther Exp $ */ +/* $OpenBSD: syntax.c,v 1.4 2017/04/13 14:48:31 deraadt Exp $ */ /* * Copyright (c) 2010 Martin Hedenfalk <martin@bzero.se> @@ -266,20 +266,22 @@ syntax_is_time(struct schema *schema, char *value, size_t len, int gen) int n; char *p = value; -#define CHECK_RANGE(min, max) \ +#define CHECK_RANGE(min, max) \ + do { \ if (!isdigit((unsigned char)p[0]) || \ !isdigit((unsigned char)p[1])) \ return 0; \ n = (p[0] - '0') * 10 + (p[1] - '0'); \ if (n < min || n > max) \ return 0; \ - p += 2; + p += 2; \ + } while (0) if (gen) - CHECK_RANGE(0, 99) /* century */ - CHECK_RANGE(0, 99) /* year */ - CHECK_RANGE(1, 12) /* month */ - CHECK_RANGE(1, 31) /* day */ + CHECK_RANGE(0, 99); /* century */ + CHECK_RANGE(0, 99); /* year */ + CHECK_RANGE(1, 12); /* month */ + CHECK_RANGE(1, 31); /* day */ /* FIXME: should check number of days in month */ CHECK_RANGE(0, 23); /* hour */ |