diff options
| author |   martijn <martijn@openbsd.org> | 2018-10-01 06:37:37 +0000 |
|---|---|---|
| committer | martijn <martijn@openbsd.org> | 2018-10-01 06:37:37 +0000 |
| commit | 132394c01e18b19a7e7495ae67acb64f9c600cad (patch) | |
| tree | 8b0166da0b527b58d16bec4c339db5eac1391ca7 /lib/libc/string/strchr.c | |
| parent | update currency exchange rates; (diff) | |
| download | wireguard-openbsd-132394c01e18b19a7e7495ae67acb64f9c600cad.tar.xz wireguard-openbsd-132394c01e18b19a7e7495ae67acb64f9c600cad.zip | |
As per POSIX, when str{,r}chr is comparing it should convert c to a char.
The C implementation of str{,r}chr are not linked to the build, because
assembly implementations are used, but change to code for easier reference.
At least the i386 and amd64 are checked and seem to do the correct thing.
Found thanks to the csh any/strchr change.
minor pointers and OK millert@
Diffstat (limited to 'lib/libc/string/strchr.c')
| -rw-r--r-- | lib/libc/string/strchr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/libc/string/strchr.c b/lib/libc/string/strchr.c index b396b45b3b0..8bfa7ac3aa0 100644 --- a/lib/libc/string/strchr.c +++ b/lib/libc/string/strchr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: strchr.c,v 1.3 2015/08/31 02:53:57 guenther Exp $ */ +/* $OpenBSD: strchr.c,v 1.4 2018/10/01 06:37:37 martijn Exp $ */ /*- * Copyright (c) 1990 The Regents of the University of California. * All rights reserved. @@ -36,7 +36,7 @@ char * strchr(const char *p, int ch) { for (;; ++p) { - if (*p == ch) + if (*p == (char) ch) return((char *)p); if (!*p) return((char *)NULL); |