Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/make make(1): split local variable in Cmd_Exec into two
details: https://anonhg.NetBSD.org/src/rev/9d4e088af490
branches: trunk
changeset: 974718:9d4e088af490
user: rillig <rillig%NetBSD.org@localhost>
date: Sun Aug 09 09:01:29 2020 +0000
description:
make(1): split local variable in Cmd_Exec into two
This avoids a mismatch between signed and unsigned types.
diffstat:
usr.bin/make/main.c | 34 ++++++++++++++++------------------
1 files changed, 16 insertions(+), 18 deletions(-)
diffs (89 lines):
diff -r 607fb035f708 -r 9d4e088af490 usr.bin/make/main.c
--- a/usr.bin/make/main.c Sun Aug 09 08:49:00 2020 +0000
+++ b/usr.bin/make/main.c Sun Aug 09 09:01:29 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $ */
+/* $NetBSD: main.c,v 1.298 2020/08/09 09:01:29 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -69,7 +69,7 @@
*/
#ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $";
+static char rcsid[] = "$NetBSD: main.c,v 1.298 2020/08/09 09:01:29 rillig Exp $";
#else
#include <sys/cdefs.h>
#ifndef lint
@@ -81,7 +81,7 @@
#if 0
static char sccsid[] = "@(#)main.c 8.3 (Berkeley) 3/19/94";
#else
-__RCSID("$NetBSD: main.c,v 1.297 2020/08/08 18:54:04 rillig Exp $");
+__RCSID("$NetBSD: main.c,v 1.298 2020/08/09 09:01:29 rillig Exp $");
#endif
#endif /* not lint */
#endif
@@ -1599,12 +1599,13 @@
int fds[2]; /* Pipe streams */
int cpid; /* Child PID */
int pid; /* PID from wait() */
- char *res; /* result */
int status; /* command exit status */
Buffer buf; /* buffer to store the result */
+ ssize_t bytes_read;
+ char *res; /* result */
+ size_t res_len;
char *cp;
int savederr; /* saved errno */
- ssize_t cc; /* bytes read, or -1 */
*errfmt = NULL;
@@ -1666,12 +1667,12 @@
/* XXX: split variable cc into 2 */
do {
char result[BUFSIZ];
- cc = read(fds[0], result, sizeof(result));
- if (cc > 0)
- Buf_AddBytes(&buf, result, (size_t)cc);
+ bytes_read = read(fds[0], result, sizeof(result));
+ if (bytes_read > 0)
+ Buf_AddBytes(&buf, result, (size_t)bytes_read);
}
- while (cc > 0 || (cc == -1 && errno == EINTR));
- if (cc == -1)
+ while (bytes_read > 0 || (bytes_read == -1 && errno == EINTR));
+ if (bytes_read == -1)
savederr = errno;
/*
@@ -1686,7 +1687,7 @@
JobReapChild(pid, status, FALSE);
continue;
}
- cc = (/* XXX */ssize_t)Buf_Size(&buf);
+ res_len = Buf_Size(&buf);
res = Buf_Destroy(&buf, FALSE);
if (savederr != 0)
@@ -1697,14 +1698,11 @@
else if (WEXITSTATUS(status) != 0)
*errfmt = "\"%s\" returned non-zero status";
- /*
- * Null-terminate the result, convert newlines to spaces and
- * install it in the variable.
- */
- res[cc] = '\0';
- cp = &res[cc];
+ /* Null-terminate the result and convert newlines to spaces. */
+ res[res_len] = '\0';
+ cp = &res[res_len];
- if (cc > 0 && *--cp == '\n') {
+ if (res_len > 0 && *--cp == '\n') {
/*
* A final newline is just stripped
*/
Home |
Main Index |
Thread Index |
Old Index