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

hlinnaka · hlinnaka · commit eadd80c08ddf · 2014-12-05T14:31:54.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
@@ -1656,8 +1656,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
`@@ -1656,8 +1656,12 @@ get_set_pwd(void)`
`1656`	`1656`	`}`
`1657`	`1657`	`if (!fgets(pwdbuf, sizeof(pwdbuf), pwf))`
`1658`	`1658`	`{`
`1659`		`- fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),`
`1660`		`- progname, pwfilename, strerror(errno));`
	`1659`	`+ if (ferror(pwf))`
	`1660`	`+ fprintf(stderr, _("%s: could not read password from file \"%s\": %s\n"),`
	`1661`	`+ progname, pwfilename, strerror(errno));`
	`1662`	`+ else`
	`1663`	`+ fprintf(stderr, _("%s: password file \"%s\" is empty\n"),`
	`1664`	`+ progname, pwfilename);`
`1661`	`1665`	`exit_nicely();`
`1662`	`1666`	`}`
`1663`	`1667`	`fclose(pwf);`