Give a proper error message if initdb password file is empty.

hlinnaka · hlinnaka · commit 198cbe0a0c4b · 2014-12-05T14:30:31.000+02:00
Used to say just "could not read password from file "...": Success", which
isn't very informative.

Mats Erik Andersson. Backpatch to all supported versions.
diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.c
@@ -1663,8 +1663,12 @@ get_set_pwd(void)
 		}
 		if (!fgets(pwdbuf, sizeof(pwdbuf), pwf))
 		{
-			fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),
-					progname, pwfilename, strerror(errno));
+			if (ferror(pwf))
+				fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),
+						progname, pwfilename, strerror(errno));
+			else
+				fprintf(stderr, _("%s: password file \"%s\" is empty\n"),
+						progname, pwfilename);
 			exit_nicely();
 		}
 		fclose(pwf);

Original file line number	Diff line number	Diff line change
`@@ -1663,8 +1663,12 @@ get_set_pwd(void)`
`1663`	`1663`	`}`
`1664`	`1664`	`if (!fgets(pwdbuf, sizeof(pwdbuf), pwf))`
`1665`	`1665`	`{`
`1666`		`- fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),`
`1667`		`- progname, pwfilename, strerror(errno));`
	`1666`	`+ if (ferror(pwf))`
	`1667`	`+ fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),`
	`1668`	`+ progname, pwfilename, strerror(errno));`
	`1669`	`+ else`
	`1670`	`+ fprintf(stderr, _("%s: password file \"%s\" is empty\n"),`
	`1671`	`+ progname, pwfilename);`
`1668`	`1672`	`exit_nicely();`
`1669`	`1673`	`}`
`1670`	`1674`	`fclose(pwf);`