diff options
| author |   ray <ray@openbsd.org> | 2006-10-24 06:25:10 +0000 |
|---|---|---|
| committer | ray <ray@openbsd.org> | 2006-10-24 06:25:10 +0000 |
| commit | 5ca550024384b00cce7e599416dfc30a519019f0 (patch) | |
| tree | 8f4ee2a22920369cc3594015d6e1005cbba21eb7 | |
| parent | Check that string length != 0 before setting buf[strlen(buf) - 1]. (diff) | |
| download | wireguard-openbsd-5ca550024384b00cce7e599416dfc30a519019f0.tar.xz wireguard-openbsd-5ca550024384b00cce7e599416dfc30a519019f0.zip | |
Reuse len instead of calling strlen again.
OK xsa@.
| -rw-r--r-- | usr.bin/cvs/entries.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/usr.bin/cvs/entries.c b/usr.bin/cvs/entries.c index f51b96695d8..8dbd1f61108 100644 --- a/usr.bin/cvs/entries.c +++ b/usr.bin/cvs/entries.c @@ -1,4 +1,4 @@ -/* $OpenBSD: entries.c,v 1.61 2006/07/09 01:47:20 joris Exp $ */ +/* $OpenBSD: entries.c,v 1.62 2006/10/24 06:25:10 ray Exp $ */ /* * Copyright (c) 2006 Joris Vink <joris@openbsd.org> * @@ -69,8 +69,8 @@ cvs_ent_open(const char *dir) if ((fp = fopen(ep->cef_lpath, "r")) != NULL) { while (fgets(buf, sizeof(buf), fp)) { len = strlen(buf); - if (len > 0 && buf[strlen(buf) - 1] == '\n') - buf[strlen(buf) - 1] = '\0'; + if (len > 0 && buf[len - 1] == '\n') + buf[len - 1] = '\0'; p = &buf[1]; |