commit 8142fcaf9ed8ff66252deecbfd29fc59d5f2df4f
author deraadt@openbsd.org <deraadt@openbsd.org> Wed Jul 03 03:24:02 2019 +0000
committer Damien Miller <djm@mindrot.org> Fri Jul 05 11:15:30 2019 +1000
tree cd3fbb3f81fd3be4b9f9047bed69ccf5df09849e
parent 4d28fa78abce2890e136281950633fae2066cc29

upstream: snprintf/vsnprintf return < 0 on error, rather than -1.

OpenBSD-Commit-ID: a261c421140a0639bb2b66bbceca72bf8239749d

moduli.c

diff --git a/moduli.c b/moduli.c
index 7120415..578fc48 100644
--- a/moduli.c
+++ b/moduli.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: moduli.c,v 1.34 2019/01/23 09:49:00 dtucker Exp $ */
+/* $OpenBSD: moduli.c,v 1.35 2019/07/03 03:24:02 deraadt Exp $ */
 /*
  * Copyright 1994 Phil Karn <karn@qualcomm.com>
  * Copyright 1996-1998, 2003 William Allen Simpson <wsimpson@greendragon.com>
@@ -453,7 +453,7 @@
 	int r;
 
 	r = snprintf(tmp, sizeof(tmp), "%s.XXXXXXXXXX", cpfile);
-	if (r == -1 || r >= PATH_MAX) {
+	if (r < 0 || r >= PATH_MAX) {
 		logit("write_checkpoint: temp pathname too long");
 		return;
 	}