Page MenuHomeFreeBSD
Files F102016377

D44167.diff
No OneTemporary
Actions

D44167.diff
View Options

diff --git a/usr.bin/script/script.c b/usr.bin/script/script.c
--- a/usr.bin/script/script.c
+++ b/usr.bin/script/script.c
@@ -40,6 +40,7 @@
#include <sys/endian.h>
#include <dev/filemon/filemon.h>
+#include <assert.h>
#include <err.h>
#include <errno.h>
#include <fcntl.h>
@@ -114,19 +115,19 @@
usesleep = 1;
rawout = 0;
flushtime = 30;
- fm_fd = -1; /* Shut up stupid "may be used uninitialized" GCC
- warning. (not needed w/clang) */
+ fm_fd = -1;
showexit = 0;
while ((ch = getopt(argc, argv, "adeFfkpqrT:t:")) != -1)
- switch(ch) {
+ switch (ch) {
case 'a':
aflg = 1;
break;
case 'd':
usesleep = 0;
break;
- case 'e': /* Default behavior, accepted for linux compat */
+ case 'e':
+ /* Default behavior, accepted for linux compat. */
break;
case 'F':
Fflg = 1;
@@ -239,6 +240,8 @@
(void)tcsetattr(STDIN_FILENO, TCSAFLUSH, &rtt);
}
+ assert(fflg ? fm_fd >= 0 : fm_fd < 0);
+
child = fork();
if (child < 0) {
warn("fork");
@@ -331,7 +334,7 @@
}
}
if (n > 0 && FD_ISSET(master, &rfd)) {
- cc = read(master, obuf, sizeof (obuf));
+ cc = read(master, obuf, sizeof(obuf));
if (cc <= 0)
break;
(void)write(STDOUT_FILENO, obuf, cc);
@@ -417,7 +420,7 @@
if (showexit)
(void)fprintf(fscript, "\nCommand exit status:"
" %d", eno);
- (void)fprintf(fscript,"\nScript done on %s",
+ (void)fprintf(fscript, "\nScript done on %s",
ctime(&tvec));
}
(void)printf("\nScript done, output file is %s\n", fname);
@@ -459,8 +462,7 @@
if (reg) {
if (fseeko(fp, len, SEEK_CUR) == -1)
err(1, NULL);
- }
- else {
+ } else {
while (len > 0) {
l = MIN(DEF_BUF, len);
if (fread(buf, sizeof(char), l, fp) != l)

File Metadata

Mime Type
text/plain
Expires
Thu, Nov 7, 2:22 PM (22 h, 38 s)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
14514233
Default Alt Text
D44167.diff (1 KB)

Event Timeline