summaryrefslogtreecommitdiff
path: root/usr.bin/unifdef
diff options
context:
space:
mode:
authorTony Finch <fanf@FreeBSD.org>2011-01-18 19:13:03 +0000
committerTony Finch <fanf@FreeBSD.org>2011-01-18 19:13:03 +0000
commit8bd9ecb5690b4f4f9b007942a567ab6fdc0fea29 (patch)
tree57eb1f83ea3de8a98a2d143f019eb9560516a8c8 /usr.bin/unifdef
parented15702f7e5f2e27601dc0d9837b090d659a52ea (diff)
downloadsrc-test2-8bd9ecb5690b4f4f9b007942a567ab6fdc0fea29.tar.gz
src-test2-8bd9ecb5690b4f4f9b007942a567ab6fdc0fea29.zip
Update to upstream version 2.3
Only significant change is to fix a bu when the output file overwrites the input when the input is redirected. Obtained from: http://dotat.at/prog/unifdef
Notes
Notes: svn path=/head/; revision=217550
Diffstat (limited to 'usr.bin/unifdef')
-rw-r--r--usr.bin/unifdef/unifdef.c25
1 files changed, 10 insertions, 15 deletions
diff --git a/usr.bin/unifdef/unifdef.c b/usr.bin/unifdef/unifdef.c
index b8f99f8fb0d8..17a1c192369e 100644
--- a/usr.bin/unifdef/unifdef.c
+++ b/usr.bin/unifdef/unifdef.c
@@ -57,7 +57,7 @@
#include <unistd.h>
const char copyright[] =
- "@(#) $Version: unifdef-2.3 $\n"
+ "@(#) $Version: unifdef-2.5 $\n"
"@(#) $FreeBSD$\n"
"@(#) $Author: Tony Finch (dot@dotat.at) $\n"
"@(#) $URL: http://dotat.at/prog/unifdef $\n"
@@ -329,16 +329,10 @@ main(int argc, char *argv[])
output = stdout;
} else {
struct stat ist, ost;
- memset(&ist, 0, sizeof(ist));
- memset(&ost, 0, sizeof(ost));
-
- if (fstat(fileno(input), &ist) != 0)
- err(2, "can't fstat %s", filename);
- if (stat(ofilename, &ost) != 0 && errno != ENOENT)
- warn("can't stat %s", ofilename);
-
- overwriting = (ist.st_dev == ost.st_dev
- && ist.st_ino == ost.st_ino);
+ if (stat(ofilename, &ost) == 0 &&
+ fstat(fileno(input), &ist) == 0)
+ overwriting = (ist.st_dev == ost.st_dev
+ && ist.st_ino == ost.st_ino);
if (overwriting) {
const char *dirsep;
int ofd;
@@ -402,7 +396,8 @@ usage(void)
* When we have processed a group that starts off with a known-false
* #if/#elif sequence (which has therefore been deleted) followed by a
* #elif that we don't understand and therefore must keep, we edit the
- * latter into a #if to keep the nesting correct.
+ * latter into a #if to keep the nesting correct. We use strncpy() to
+ * overwrite the 4 byte token "elif" with "if " without a '\0' byte.
*
* When we find a true #elif in a group, the following block will
* always be kept and the rest of the sequence after the next #elif or
@@ -455,7 +450,7 @@ static void Oelif (void) { if (!iocccok) Eioccc(); Pelif(); }
static void Idrop (void) { Fdrop(); ignoreon(); }
static void Itrue (void) { Ftrue(); ignoreon(); }
static void Ifalse(void) { Ffalse(); ignoreon(); }
-/* edit this line */
+/* modify this line */
static void Mpass (void) { strncpy(keyword, "if ", 4); Pelif(); }
static void Mtrue (void) { keywordedit("else"); state(IS_TRUE_MIDDLE); }
static void Melif (void) { keywordedit("endif"); state(IS_FALSE_TRAILER); }
@@ -629,10 +624,10 @@ done(void)
if (incomment)
error("EOF in comment");
closeout();
- if (overwriting && rename(tempname, filename) == -1) {
+ if (overwriting && rename(tempname, ofilename) == -1) {
warn("couldn't rename temporary file");
unlink(tempname);
- errx(2, "%s unchanged", filename);
+ errx(2, "%s unchanged", ofilename);
}
exit(exitstat);
}