svn commit: r213300 - head/usr.bin/csup
John Baldwin
jhb at FreeBSD.org
Thu Sep 30 14:28:49 UTC 2010
Author: jhb
Date: Thu Sep 30 14:28:48 2010
New Revision: 213300
URL: http://svn.freebsd.org/changeset/base/213300
Log:
If an RCS file is truncated, rcsfile_getdelta() will return NULL. Instead
of faulting, check for NULL. However, returning an error would cause csup
to just abort the entire update. Instead, break out of the loop and
return ok. The attempts to update the file will trigger a MD5 failure which
will cause csup to download the entire file as a fixup.
Reviewed by: lulf
MFC after: 1 week
Modified:
head/usr.bin/csup/rcsparse.c
Modified: head/usr.bin/csup/rcsparse.c
==============================================================================
--- head/usr.bin/csup/rcsparse.c Thu Sep 30 13:44:55 2010 (r213299)
+++ head/usr.bin/csup/rcsparse.c Thu Sep 30 14:28:48 2010 (r213300)
@@ -318,6 +318,14 @@ parse_deltatexts(struct rcsfile *rf, yys
d = rcsfile_getdelta(rf, revnum);
free(revnum);
+ /*
+ * XXX: The RCS file is corrupt, but lie and say it is ok.
+ * If it is actually broken, then the MD5 mismatch will
+ * trigger a fixup.
+ */
+ if (d == NULL)
+ return (0);
+
/* log string */
asserttoken(sp, KEYWORD);
asserttoken(sp, STRING);
More information about the svn-src-all
mailing list