From: Marc Dionne <marc.c.dionne@gmail.com>
Date: Tue, 13 Apr 2010 23:35:39 +0000 (-0400)
Subject: Replace unsafe use of gets()
X-Git-Tag: openafs-devel-1_5_74~24
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=ab2005d719d2184a1645fdc33f49e73275d91353;p=packages%2Fo%2Fopenafs.git

Replace unsafe use of gets()

Build tools compain that this is dangerous - replace gets() with a
bounded fgets().

Change-Id: I3bd1854a1dc4a11c801cabb987ab680fa978bd20
Reviewed-on: http://gerrit.openafs.org/1750
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/tviced/state_analyzer.c b/src/tviced/state_analyzer.c
index faf571a80..d3bfd38bc 100644
--- a/src/tviced/state_analyzer.c
+++ b/src/tviced/state_analyzer.c
@@ -368,7 +368,7 @@ prompt(void)
 		fprintf(stderr, "prompt state broken; aborting\n");
 		return;
 	    }
-	    gets(input);
+	    fgets(input, 256, stdin);
 
 	    if (!strcmp(input, "")) {
 		/* repeat last command */